From cbed6a607d6b42b0cb264d2bee1bd784c8504d60 Mon Sep 17 00:00:00 2001 From: Ulrich Drepper Date: Fri, 9 Nov 2007 09:09:07 +0000 Subject: [BZ #5240] * sysdeps/unix/sysv/linux/lowlevellock.c (__lll_timedlock_wait): If we time out, try one last time to lock the futex to avoid losing a wakeup signal. * sysdeps/unix/sysv/linux/i386/i486/lowlevellock.S: Likewise. * sysdeps/unix/sysv/linux/x86_64/lowlevellock.S: Likewise. --- nptl/sysdeps/unix/sysv/linux/lowlevellock.c | 23 +++++++++++++---------- 1 file changed, 13 insertions(+), 10 deletions(-) (limited to 'nptl/sysdeps/unix/sysv/linux/lowlevellock.c') diff --git a/nptl/sysdeps/unix/sysv/linux/lowlevellock.c b/nptl/sysdeps/unix/sysv/linux/lowlevellock.c index f0e42957c2..c2cdf3a1e4 100644 --- a/nptl/sysdeps/unix/sysv/linux/lowlevellock.c +++ b/nptl/sysdeps/unix/sysv/linux/lowlevellock.c @@ -59,10 +59,10 @@ __lll_timedlock_wait (int *futex, const struct timespec *abstime, int private) if (abstime->tv_nsec < 0 || abstime->tv_nsec >= 1000000000) return EINVAL; + struct timespec rt; do { struct timeval tv; - struct timespec rt; /* Get the current time. */ (void) __gettimeofday (&tv, NULL); @@ -76,18 +76,21 @@ __lll_timedlock_wait (int *futex, const struct timespec *abstime, int private) --rt.tv_sec; } - /* Already timed out? */ - if (rt.tv_sec < 0) - return ETIMEDOUT; + /* If timed out do not go to sleep. */ + if (__builtin_expect (rt.tv_sec >= 0, 1)) + { + /* Wait. */ + int oldval = atomic_compare_and_exchange_val_acq (futex, 2, 1); + if (oldval != 0) + lll_futex_timed_wait (futex, 2, &rt, private); + } - /* Wait. */ - int oldval = atomic_compare_and_exchange_val_acq (futex, 2, 1); - if (oldval != 0) - lll_futex_timed_wait (futex, 2, &rt, private); + if (atomic_compare_and_exchange_bool_acq (futex, 2, 0) == 0) + return 0; } - while (atomic_compare_and_exchange_bool_acq (futex, 2, 0) != 0); + while (rt.tv_sec >= 0); - return 0; + return ETIMEDOUT; } -- cgit 1.4.1