]> CyberLeo.Net >> Repos - FreeBSD/FreeBSD.git/commit
Prevent leak of URWLOCK_READ_WAITERS flag for urwlocks.
authorkib <kib@FreeBSD.org>
Thu, 25 Aug 2016 16:35:42 +0000 (16:35 +0000)
committerkib <kib@FreeBSD.org>
Thu, 25 Aug 2016 16:35:42 +0000 (16:35 +0000)
commitc0cdc81b9591bcdaf2d4f2c06315f27547df8eb9
tree7a35619a5c9a9ce1a044883c2e2adda3e211df4c
parent5dd28c7f858f7baeb809b4d3727b622671dc76a0
Prevent leak of URWLOCK_READ_WAITERS flag for urwlocks.

If there was some error, e.g. the sleep was interrupted, as in the
referenced PR, do_rw_rdlock() did not cleared URWLOCK_READ_WAITERS.
Since unlock only wakes up write waiters when there is no read
waiters, for URWLOCK_PREFER_READER kind of locks, the result was
missed wakeups for writers.

In particular, the most visible victims are ld-elf.so locks in
processes which loaded libthr, because rtld locks are urwlocks in
prefer-reader mode.  Normal rwlocks fall into prefer-reader mode only
if thread already owns rw lock in read mode, which is not typical and
correspondingly less visible.  In the PR, unowned rtld bind lock was
waited for in the process where only one thread was left alive.

Note that do_rw_wrlock() correctly clears URWLOCK_WRITE_WAITERS in
case of errors.

Reported and tested by: longwitz@incore.de
PR: 211947
Sponsored by: The FreeBSD Foundation
MFC after: 1 week
sys/kern/kern_umtx.c