summaryrefslogtreecommitdiff
path: root/src/thread
diff options
context:
space:
mode:
authorRich Felker <dalias@aerifal.cx>2011-03-17 12:21:32 -0400
committerRich Felker <dalias@aerifal.cx>2011-03-17 12:21:32 -0400
commitb1c43161c2b37e35eae60fa61621a24a5d820d83 (patch)
treeb6c532f959850f42ad219d87b721a587973c6654 /src/thread
parente914f8b7ec79f622fa3b017af434642f61d45ce8 (diff)
downloadmusl-b1c43161c2b37e35eae60fa61621a24a5d820d83.tar.gz
unify lock and owner fields of mutex structure
this change is necessary to free up one slot in the mutex structure so that we can use doubly-linked lists in the implementation of robust mutexes.
Diffstat (limited to 'src/thread')
-rw-r--r--src/thread/pthread_mutex_lock.c5
-rw-r--r--src/thread/pthread_mutex_trylock.c5
-rw-r--r--src/thread/pthread_mutex_unlock.c3
3 files changed, 6 insertions, 7 deletions
diff --git a/src/thread/pthread_mutex_lock.c b/src/thread/pthread_mutex_lock.c
index 82556141..56111ec8 100644
--- a/src/thread/pthread_mutex_lock.c
+++ b/src/thread/pthread_mutex_lock.c
@@ -4,10 +4,11 @@ int pthread_mutex_lock(pthread_mutex_t *m)
{
int r;
while ((r=pthread_mutex_trylock(m)) == EBUSY) {
+ if (!(r=m->_m_lock)) continue;
if (m->_m_type == PTHREAD_MUTEX_ERRORCHECK
- && m->_m_owner == pthread_self()->tid)
+ && r == pthread_self()->tid)
return EDEADLK;
- __wait(&m->_m_lock, &m->_m_waiters, 1, 0);
+ __wait(&m->_m_lock, &m->_m_waiters, r, 0);
}
return r;
}
diff --git a/src/thread/pthread_mutex_trylock.c b/src/thread/pthread_mutex_trylock.c
index 6fc604fe..de57ff9e 100644
--- a/src/thread/pthread_mutex_trylock.c
+++ b/src/thread/pthread_mutex_trylock.c
@@ -9,14 +9,13 @@ int pthread_mutex_trylock(pthread_mutex_t *m)
tid = pthread_self()->tid;
- if (m->_m_owner == tid && m->_m_type == PTHREAD_MUTEX_RECURSIVE) {
+ if (m->_m_lock == tid && m->_m_type == PTHREAD_MUTEX_RECURSIVE) {
if ((unsigned)m->_m_count >= INT_MAX) return EAGAIN;
m->_m_count++;
return 0;
}
- if (m->_m_owner || a_xchg(&m->_m_lock, 1)) return EBUSY;
- m->_m_owner = tid;
+ if (m->_m_lock || a_cas(&m->_m_lock, 0, tid)) return EBUSY;
m->_m_count = 1;
return 0;
}
diff --git a/src/thread/pthread_mutex_unlock.c b/src/thread/pthread_mutex_unlock.c
index 39cea4b6..61a2b947 100644
--- a/src/thread/pthread_mutex_unlock.c
+++ b/src/thread/pthread_mutex_unlock.c
@@ -3,13 +3,12 @@
int pthread_mutex_unlock(pthread_mutex_t *m)
{
if (m->_m_type != PTHREAD_MUTEX_NORMAL) {
- if (m->_m_owner != pthread_self()->tid)
+ if (m->_m_lock != pthread_self()->tid)
return EPERM;
if (m->_m_type == PTHREAD_MUTEX_RECURSIVE && --m->_m_count)
return 0;
}
- m->_m_owner = 0;
m->_m_lock = 0;
if (m->_m_waiters) __wake(&m->_m_lock, 1, 0);
return 0;