diff options
| author | Jonathan Peyton <jonathan.l.peyton@intel.com> | 2018-08-24 18:07:35 +0000 |
|---|---|---|
| committer | Jonathan Peyton <jonathan.l.peyton@intel.com> | 2018-08-24 18:07:35 +0000 |
| commit | a4a9c48c784cb01a36dad9b6aa9e828334df31f6 (patch) | |
| tree | 398684486812b86cef3d1190bb93ffe5248c57b1 /openmp | |
| parent | 4058e29e7d86c266b9acd50c3b97715ca564a0b0 (diff) | |
| download | bcm5719-llvm-a4a9c48c784cb01a36dad9b6aa9e828334df31f6.tar.gz bcm5719-llvm-a4a9c48c784cb01a36dad9b6aa9e828334df31f6.zip | |
[OpenMP] Fix tasking bug for decreasing hot team nthreads
The __kmp_execute_tasks_template() function reads the task_team and
current_task from the thread structure. There appears to be a pathological
timing where the number of threads in the hot team decreases and so a
thread is put in the pool via __kmp_free_thread(). It could be the case that:
1) A thread reads th_task_team into task_team local variables
and is then interrupted by the OS
2) Master frees the thread and sets current task and task team to NULL
3) The thread reads current_task as NULL
When this happens, current_task is dereferenced and a segfault occurs.
This patch just checks for current_task to not be NULL as well.
Differential Revision: https://reviews.llvm.org/D50651
llvm-svn: 340632
Diffstat (limited to 'openmp')
| -rw-r--r-- | openmp/runtime/src/kmp_tasking.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/openmp/runtime/src/kmp_tasking.cpp b/openmp/runtime/src/kmp_tasking.cpp index d9220c5a2dc..e1b03f780b5 100644 --- a/openmp/runtime/src/kmp_tasking.cpp +++ b/openmp/runtime/src/kmp_tasking.cpp @@ -2469,7 +2469,7 @@ static inline int __kmp_execute_tasks_template( KMP_DEBUG_ASSERT(__kmp_tasking_mode != tskm_immediate_exec); KMP_DEBUG_ASSERT(thread == __kmp_threads[gtid]); - if (task_team == NULL) + if (task_team == NULL || current_task == NULL) return FALSE; KA_TRACE(15, ("__kmp_execute_tasks_template(enter): T#%d final_spin=%d " |
