diff options
Diffstat (limited to 'linuxthreads')
| -rw-r--r-- | linuxthreads/ChangeLog | 3 | ||||
| -rw-r--r-- | linuxthreads/spinlock.h | 23 |
2 files changed, 20 insertions, 6 deletions
diff --git a/linuxthreads/ChangeLog b/linuxthreads/ChangeLog index daa6f06..6d22d9b 100644 --- a/linuxthreads/ChangeLog +++ b/linuxthreads/ChangeLog @@ -1,5 +1,8 @@ 2000-09-26 Ulrich Drepper <drepper@redhat.com> + * spinlock.h (__pthread_set_own_extricate_if): Optimize a bit. + Patch by Kaz Kylheku <kaz@ashi.footprints.net>. + * sysdeps/unix/sysv/linux/i386/bits/posix_opt.h: Remove _POSIX_MONOTONIC_CLOCK. diff --git a/linuxthreads/spinlock.h b/linuxthreads/spinlock.h index c194d79..2d3e9bf 100644 --- a/linuxthreads/spinlock.h +++ b/linuxthreads/spinlock.h @@ -212,11 +212,22 @@ static inline long atomic_decrement(struct pthread_atomic *pa) static inline void -__pthread_set_own_extricate_if(pthread_descr self, pthread_extricate_if *peif) +__pthread_set_own_extricate_if (pthread_descr self, pthread_extricate_if *peif) { - /* The locks here are not ensuring an atomic update of the p_extricate - pointer. They protect users of the pointer from using stale memory. */ - __pthread_lock(THREAD_GETMEM(self, p_lock), self); - THREAD_SETMEM(self, p_extricate, peif); - __pthread_unlock(THREAD_GETMEM (self, p_lock)); + /* Only store a non-null peif if the thread has cancellation enabled. + Otherwise pthread_cancel will unconditionally call the extricate handler, + and restart the thread giving rise to forbidden spurious wakeups. */ + if (peif == NULL + || THREAD_GETMEM(self, p_cancelstate) == PTHREAD_CANCEL_ENABLE) + { + /* If we are removing the extricate interface, we need to synchronize + against pthread_cancel so that it does not continue with a pointer + to a deallocated pthread_extricate_if struct! The thread lock + is (ab)used for this synchronization purpose. */ + if (peif == NULL) + __pthread_lock (THREAD_GETMEM(self, p_lock), self); + THREAD_SETMEM(self, p_extricate, peif); + if (peif == NULL) + __pthread_unlock (THREAD_GETMEM(self, p_lock)); + } } |