diff options
author | Mike Frysinger <vapier@gentoo.org> | 2015-07-28 02:19:49 -0400 |
---|---|---|
committer | Mike Frysinger <vapier@gentoo.org> | 2015-07-28 05:03:27 -0400 |
commit | 978908245b2c0f759100708a9966649f7b273664 (patch) | |
tree | a2a8dacbc615d6051fe0d07c4ec3afdd156dec74 /crypt/Banner | |
parent | 8a4e5cc6574a419d5724bef6be98d705d58db48d (diff) | |
download | glibc-978908245b2c0f759100708a9966649f7b273664.tar.gz glibc-978908245b2c0f759100708a9966649f7b273664.tar.xz glibc-978908245b2c0f759100708a9966649f7b273664.zip |
ia64: atomic.h: fix atomic_exchange_and_add 64bit handling
Way back in 2005 the atomic_exchange_and_add function was cleaned up to avoid the explicit size checking and instead let gcc handle things itself. Unfortunately that change ended up leaving beyond a cast to int, even when the incoming value was a long. This has flown under the radar for a long time due to the function not being heavily used in the tree (especially as a full 64bit field), but a recent change to semaphores made some nptl tests fail reliably. This is due to the code packing two 32bit values into one 64bit variable (where the high 32bits contained the number of waiters), and then the whole variable being atomically updated between threads. On ia64, that meant we never atomically updated the count, so sometimes the sem_post would not wake up the waiters. (cherry picked from commit cf31a2c79957936b60de34ea1e718e892baf669c)
Diffstat (limited to 'crypt/Banner')
0 files changed, 0 insertions, 0 deletions