[all-commits] [llvm/llvm-project] 961589: [OpenMP] libomp: change shm name to include UID, c...

[terdner via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 9615890db576721fbd73ae77d81d39435e83b4b4
      https://github.com/llvm/llvm-project/commit/9615890db576721fbd73ae77d81d39435e83b4b4
  Author: Todd Erdner <todd.erdner at intel.com>
  Date:   2020-12-01 (Tue, 01 Dec 2020)

  Changed paths:
    M openmp/runtime/src/kmp.h
    M openmp/runtime/src/kmp_runtime.cpp
    M openmp/runtime/src/z_Linux_util.cpp

  Log Message:
  -----------
  [OpenMP] libomp: change shm name to include UID, call unregister_lib on SIGTERM

With the change to using shared memory, there were a few problems that need to be fixed.
- The previous filename that was used for SHM only used process id. Given that process is
  usually based on 16bit number, this was causing some conflicts on machines. Thus we add
  UID to the name to prevent this.
- It appears under some conditions (SIGTERM, etc) the shared memory files were not getting
  cleaned up. Added a call to clean up the shm files under those conditions. For this user
  needs to set envirable KMP_HANDLE_SIGNALS to true.

Patch by Erdner, Todd <todd.erdner at intel.com>

Differential Revision: https://reviews.llvm.org/D91869