clean up access to mutex type in pthread_mutex_trylock
authorRich Felker <dalias@aerifal.cx>
Fri, 29 Mar 2019 19:49:14 +0000 (15:49 -0400)
committerRich Felker <dalias@aerifal.cx>
Fri, 29 Mar 2019 19:49:14 +0000 (15:49 -0400)
there was no point in masking off the pshared bit when first loading
the type, since every subsequent access involves a mask anyway. not
masking it may avoid a subsequent load to check the pshared flag, and
it's just simpler.

src/thread/pthread_mutex_trylock.c

index 3fe591276f0dfd8403b4e46358f61c02f6df30a6..29622ff9b0350ef6e704fe7a2ff728f90928c9ed 100644 (file)
@@ -3,7 +3,7 @@
 int __pthread_mutex_trylock_owner(pthread_mutex_t *m)
 {
        int old, own;
-       int type = m->_m_type & 15;
+       int type = m->_m_type;
        pthread_t self = __pthread_self();
        int tid = self->tid;
 
@@ -17,7 +17,7 @@ int __pthread_mutex_trylock_owner(pthread_mutex_t *m)
        if (own == 0x3fffffff) return ENOTRECOVERABLE;
        if (own || (old && !(type & 4))) return EBUSY;
 
-       if (m->_m_type & 128) {
+       if (type & 128) {
                if (!self->robust_list.off) {
                        self->robust_list.off = (char*)&m->_m_lock-(char *)&m->_m_next;
                        __syscall(SYS_set_robust_list, &self->robust_list, 3*sizeof(long));