summaryrefslogtreecommitdiff
path: root/src/thread/pthread_mutex_timedlock.c
diff options
context:
space:
mode:
authorRich Felker <dalias@aerifal.cx>2014-08-25 20:24:07 -0400
committerRich Felker <dalias@aerifal.cx>2014-08-25 20:24:07 -0400
commitf5fb20b0e934770c37093105524ea644dcaba5e2 (patch)
treedcf0555f52553e2ef67b6aef1d8de05b5c89bd8a /src/thread/pthread_mutex_timedlock.c
parent97a7512b6819014d15c679c8998caa0006d13c29 (diff)
downloadmusl-f5fb20b0e934770c37093105524ea644dcaba5e2.tar.gz
refrain from spinning on locks when there is already a waiter
if there is already a waiter for a lock, spinning on the lock is essentially an attempt to steal it from whichever waiter would obtain it via any priority rules in place, and is therefore undesirable. in the current implementation, there is always an inherent race window at unlock during which a newly-arriving thread may steal the lock from the existing waiters, but we should aim to keep this window minimal rather than enlarging it.
Diffstat (limited to 'src/thread/pthread_mutex_timedlock.c')
-rw-r--r--src/thread/pthread_mutex_timedlock.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/thread/pthread_mutex_timedlock.c b/src/thread/pthread_mutex_timedlock.c
index 116a8b7b..ae883f90 100644
--- a/src/thread/pthread_mutex_timedlock.c
+++ b/src/thread/pthread_mutex_timedlock.c
@@ -12,7 +12,7 @@ int pthread_mutex_timedlock(pthread_mutex_t *restrict m, const struct timespec *
if (r != EBUSY) return r;
int spins = 100;
- while (spins-- && m->_m_lock) a_spin();
+ while (spins-- && m->_m_lock && !m->_m_waiters) a_spin();
while ((r=pthread_mutex_trylock(m)) == EBUSY) {
if (!(r=m->_m_lock) || ((r&0x40000000) && (m->_m_type&4)))