[Openmp-commits] [PATCH] D21196: Excluded untied tasks from task stealing constraint

[Jonathan Peyton via Openmp-commits]

jlpeyton added inline comments.


================
Comment at: openmp/trunk/runtime/src/kmp_tasking.c:1754
         }
-        if ( parent != current ) {
+        if ( parent != current && (taskdata->td_flags.tiedness == TASK_TIED) ) { // untied is always allowed to be stolen
             // If the tail task is not a child, then no other childs can appear in the deque (?).
----------------
pawosm01 wrote:
> This line causes race condition which leads to occasional livelock in strassen_taskdep benchmark from Kastors suite (a benchmark that heavily uses task dependencies). Could be replicated on AArch64 and x86_64, typically working on odd number of cores (5, 7, etc.).
> 
Can you give the exact libomp revision you're using, how you are calling strassen_taskdep, operating system, compiler?


Repository:
  rL LLVM

https://reviews.llvm.org/D21196