[Openmp-commits] [PATCH] D14544: [OMPT] Fix the lock id provided for ordered events

Joachim Protze via Openmp-commits openmp-commits at lists.llvm.org
Tue Nov 10 09:32:18 PST 2015


protze.joachim created this revision.
protze.joachim added reviewers: jmellorcrummey, jlpeyton.
protze.joachim added a subscriber: openmp-commits.

With the current implementation, ompt_thread_info.wait_id is set to 0 once the thread got the lock. This value is used to provide the lock id for the event calls.

With this patch, the lock id as provided by the function argument is used. Now all event callbacks provide a consistent lock id.

http://reviews.llvm.org/D14544

Files:
  runtime/src/kmp_csupport.c

Index: runtime/src/kmp_csupport.c
===================================================================
--- runtime/src/kmp_csupport.c
+++ runtime/src/kmp_csupport.c
@@ -868,7 +868,7 @@
         /* OMPT event callback */
         if (ompt_callbacks.ompt_callback(ompt_event_acquired_ordered)) {
             ompt_callbacks.ompt_callback(ompt_event_acquired_ordered)(
-                th->th.ompt_thread_info.wait_id);
+                (uint64_t) loc);
         }
     }
 #endif
@@ -909,7 +909,7 @@
     if (ompt_enabled &&
         ompt_callbacks.ompt_callback(ompt_event_release_ordered)) {
         ompt_callbacks.ompt_callback(ompt_event_release_ordered)(
-            th->th.ompt_thread_info.wait_id);
+            (uint64_t) loc);
     }
 #endif
 }


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D14544.39815.patch
Type: text/x-patch
Size: 752 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/openmp-commits/attachments/20151110/b5d6e054/attachment.bin>


More information about the Openmp-commits mailing list