[PATCH] D13502: Reduce overhead of OMPT

John Mellor-Crummey via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 7 02:40:45 PDT 2015


jmellorcrummey created this revision.
jmellorcrummey added reviewers: jlpeyton, Hahnfeld.
jmellorcrummey added subscribers: openmp-commits, llvm-commits.

Avoid computing state needed only by OMPT unless the ompt_enabled flag is set.

Properly handle a corner case in OMPT where team == NULL.

http://reviews.llvm.org/D13502

Files:
  runtime/src/kmp_csupport.c
  runtime/src/kmp_sched.cpp
  runtime/src/ompt-specific.c

Index: runtime/src/ompt-specific.c
===================================================================
--- runtime/src/ompt-specific.c
+++ runtime/src/ompt-specific.c
@@ -50,6 +50,8 @@
 
     if (thr) {
         kmp_team *team = thr->th.th_team;
+        if (team == NULL) return NULL;
+
         ompt_lw_taskteam_t *lwt = LWT_FROM_TEAM(team);
 
         while(depth > 0) {
Index: runtime/src/kmp_sched.cpp
===================================================================
--- runtime/src/kmp_sched.cpp
+++ runtime/src/kmp_sched.cpp
@@ -97,8 +97,14 @@
     register kmp_info_t *th = __kmp_threads[ gtid ];
 
 #if OMPT_SUPPORT && OMPT_TRACE
-    ompt_team_info_t *team_info = __ompt_get_teaminfo(0, NULL);
-    ompt_task_info_t *task_info = __ompt_get_taskinfo(0);
+    ompt_team_info_t *team_info = NULL; 
+    ompt_task_info_t *task_info = NULL; 
+
+    if (ompt_enabled) {
+        // Only fully initialize variables needed by OMPT if OMPT is enabled.
+        team_info = __ompt_get_teaminfo(0, NULL);
+        task_info = __ompt_get_taskinfo(0);
+    }
 #endif
 
     KMP_DEBUG_ASSERT( plastiter && plower && pupper && pstride );
Index: runtime/src/kmp_csupport.c
===================================================================
--- runtime/src/kmp_csupport.c
+++ runtime/src/kmp_csupport.c
@@ -1485,12 +1485,12 @@
     KE_TRACE( 10, ("__kmpc_for_static_fini called T#%d\n", global_tid));
 
 #if OMPT_SUPPORT && OMPT_TRACE
-    kmp_info_t *this_thr        = __kmp_threads[ global_tid ];
-    kmp_team_t *team            = this_thr -> th.th_team;
-    int tid = __kmp_tid_from_gtid( global_tid );
-
     if (ompt_enabled &&
         ompt_callbacks.ompt_callback(ompt_event_loop_end)) {
+        kmp_info_t *this_thr = __kmp_threads[ global_tid ];
+        kmp_team_t *team     = this_thr -> th.th_team;
+        int tid = __kmp_tid_from_gtid( global_tid );
+
         ompt_callbacks.ompt_callback(ompt_event_loop_end)(
             team->t.ompt_team_info.parallel_id,
             team->t.t_implicit_task_taskdata[tid].ompt_task_info.task_id);


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D13502.36719.patch
Type: text/x-patch
Size: 2052 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151007/335c99df/attachment.bin>


More information about the llvm-commits mailing list