Skip to content

Commit

Permalink
Remove named mutex workaround for Linux arm/arm64 (#70381)
Browse files Browse the repository at this point in the history
The workaround is no longer needed as the underlying issue in
glibc was fixed in glibc 2.24 and the minimum glibc we now
support is 2.27.
  • Loading branch information
janvorli authored Jun 7, 2022
1 parent 7066ca9 commit 31cd7eb
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions src/coreclr/pal/src/include/pal/mutex.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -120,15 +120,13 @@ Miscellaneous
existing shared memory, naming, and waiting infrastructure is not suitable for this purpose, and is not used.
*/

// - Temporarily disabling usage of pthread process-shared mutexes on ARM/ARM64 due to functional issues that cannot easily be
// detected with code due to hangs. See https://github.com/dotnet/runtime/issues/6014.
// - On FreeBSD, pthread process-shared robust mutexes cannot be placed in shared memory mapped independently by the processes
// involved. See https://github.com/dotnet/runtime/issues/10519.
// - On OSX, pthread robust mutexes were/are not available at the time of this writing. In case they are made available in the
// future, their use is disabled for compatibility.
#if HAVE_FULLY_FEATURED_PTHREAD_MUTEXES && \
HAVE_FUNCTIONAL_PTHREAD_ROBUST_MUTEXES && \
!(defined(HOST_ARM) || defined(HOST_ARM64) || defined(__FreeBSD__) || defined(TARGET_OSX))
!(defined(__FreeBSD__) || defined(TARGET_OSX))

#define NAMED_MUTEX_USE_PTHREAD_MUTEX 1
#else
Expand Down

0 comments on commit 31cd7eb

Please sign in to comment.