]> git.ipfire.org Git - thirdparty/glibc.git/commit
Fix blocking pthread_join. [BZ #23137]
authorStefan Liebler <stli@linux.vnet.ibm.com>
Fri, 4 May 2018 08:00:59 +0000 (10:00 +0200)
committerFangrui Song <i@maskray.me>
Fri, 27 Aug 2021 23:22:12 +0000 (16:22 -0700)
commitb3356fb4a1be19767e98d4f9ce7411e86dcd5149
tree45af57e3ae6e4d0b88259dd47ca3a31caf225b1f
parent1ab675ca63987193ea159d4d75043d3812b54d6f
Fix blocking pthread_join. [BZ #23137]

On s390 (31bit) if glibc is build with -Os, pthread_join sometimes
blocks indefinitely. This is e.g. observable with
testcase intl/tst-gettext6.

pthread_join is calling lll_wait_tid(tid), which performs the futex-wait
syscall in a loop as long as tid != 0 (thread is alive).

On s390 (and build with -Os), tid is loaded from memory before
comparing against zero and then the tid is loaded a second time
in order to pass it to the futex-wait-syscall.
If the thread exits in between, then the futex-wait-syscall is
called with the value zero and it waits until a futex-wake occurs.
As the thread is already exited, there won't be a futex-wake.

In lll_wait_tid, the tid is stored to the local variable __tid,
which is then used as argument for the futex-wait-syscall.
But unfortunately the compiler is allowed to reload the value
from memory.

With this patch, the tid is loaded with atomic_load_acquire.
Then the compiler is not allowed to reload the value for __tid from memory.

ChangeLog:

[BZ #23137]
* sysdeps/nptl/lowlevellock.h (lll_wait_tid):
Use atomic_load_acquire to load __tid.

(cherry picked from commit 1660901840dfc9fde6c5720a32f901af6f08f00a)
ChangeLog
sysdeps/nptl/lowlevellock.h