From: Alan Modra Date: Fri, 2 Dec 2011 10:54:02 +0000 (+1030) Subject: affinity.c: Use atomic rather than sync builtin. X-Git-Tag: releases/gcc-4.7.0~1827 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=45608a43c4716d018288df65f56dacd931f7e74f;p=thirdparty%2Fgcc.git affinity.c: Use atomic rather than sync builtin. * config/linux/affinity.c: Use atomic rather than sync builtin. * config/linux/lock.c: Likewise. * config/linux/ptrlock.h: Likewise. * config/linux/ptrlock.c: Likewise. * config/linux/ptrlock.h (gomp_ptrlock_set): Always write here.. * config/linux/ptrlock.c (gomp_ptrlock_set_slow): ..not here. * config/linux/futex.h (atomic_write_barrier): Delete unused function. * config/linux/alpha/futex.h (atomic_write_barrier): Likewise. * config/linux/ia64/futex.h (atomic_write_barrier): Likewise. * config/linux/mips/futex.h (atomic_write_barrier): Likewise. * config/linux/powerpc/futex.h (atomic_write_barrier): Likewise. * config/linux/s390/futex.h (atomic_write_barrier): Likewise. * config/linux/sparc/futex.h (atomic_write_barrier): Likewise. * config/linux/x86/futex.h (atomic_write_barrier): Likewise. From-SVN: r181906 --- diff --git a/libgomp/ChangeLog b/libgomp/ChangeLog index f6278b771591..11c96dd34857 100644 --- a/libgomp/ChangeLog +++ b/libgomp/ChangeLog @@ -1,3 +1,20 @@ +2011-12-02 Alan Modra + + * config/linux/affinity.c: Use atomic rather than sync builtin. + * config/linux/lock.c: Likewise. + * config/linux/ptrlock.h: Likewise. + * config/linux/ptrlock.c: Likewise. + * config/linux/ptrlock.h (gomp_ptrlock_set): Always write here.. + * config/linux/ptrlock.c (gomp_ptrlock_set_slow): ..not here. + * config/linux/futex.h (atomic_write_barrier): Delete unused function. + * config/linux/alpha/futex.h (atomic_write_barrier): Likewise. + * config/linux/ia64/futex.h (atomic_write_barrier): Likewise. + * config/linux/mips/futex.h (atomic_write_barrier): Likewise. + * config/linux/powerpc/futex.h (atomic_write_barrier): Likewise. + * config/linux/s390/futex.h (atomic_write_barrier): Likewise. + * config/linux/sparc/futex.h (atomic_write_barrier): Likewise. + * config/linux/x86/futex.h (atomic_write_barrier): Likewise. + 2011-11-30 Alan Modra PR libgomp/51298 diff --git a/libgomp/config/linux/affinity.c b/libgomp/config/linux/affinity.c index 8896a6f8ba7e..7a904df9492f 100644 --- a/libgomp/config/linux/affinity.c +++ b/libgomp/config/linux/affinity.c @@ -109,7 +109,7 @@ gomp_init_thread_affinity (pthread_attr_t *attr) unsigned int cpu; cpu_set_t cpuset; - cpu = __sync_fetch_and_add (&affinity_counter, 1); + cpu = __atomic_fetch_add (&affinity_counter, 1, MEMMODEL_RELAXED); cpu %= gomp_cpu_affinity_len; CPU_ZERO (&cpuset); CPU_SET (gomp_cpu_affinity[cpu], &cpuset); diff --git a/libgomp/config/linux/alpha/futex.h b/libgomp/config/linux/alpha/futex.h index a594d1663e6d..c8076bd299ee 100644 --- a/libgomp/config/linux/alpha/futex.h +++ b/libgomp/config/linux/alpha/futex.h @@ -101,9 +101,3 @@ cpu_relax (void) { __asm volatile ("" : : : "memory"); } - -static inline void -atomic_write_barrier (void) -{ - __asm volatile ("wmb" : : : "memory"); -} diff --git a/libgomp/config/linux/futex.h b/libgomp/config/linux/futex.h index bab088a558b6..d0136f50ac2e 100644 --- a/libgomp/config/linux/futex.h +++ b/libgomp/config/linux/futex.h @@ -67,9 +67,3 @@ cpu_relax (void) { __asm volatile ("" : : : "memory"); } - -static inline void -atomic_write_barrier (void) -{ - __sync_synchronize (); -} diff --git a/libgomp/config/linux/ia64/futex.h b/libgomp/config/linux/ia64/futex.h index e5e9aac39967..85cd02fa15c6 100644 --- a/libgomp/config/linux/ia64/futex.h +++ b/libgomp/config/linux/ia64/futex.h @@ -86,9 +86,3 @@ cpu_relax (void) { __asm volatile ("hint @pause" : : : "memory"); } - -static inline void -atomic_write_barrier (void) -{ - __sync_synchronize (); -} diff --git a/libgomp/config/linux/lock.c b/libgomp/config/linux/lock.c index 405460780a94..60693439422f 100644 --- a/libgomp/config/linux/lock.c +++ b/libgomp/config/linux/lock.c @@ -1,4 +1,4 @@ -/* Copyright (C) 2005, 2008, 2009 Free Software Foundation, Inc. +/* Copyright (C) 2005, 2008, 2009, 2011 Free Software Foundation, Inc. Contributed by Richard Henderson . This file is part of the GNU OpenMP Library (libgomp). @@ -62,7 +62,10 @@ gomp_unset_lock_30 (omp_lock_t *lock) int gomp_test_lock_30 (omp_lock_t *lock) { - return __sync_bool_compare_and_swap (lock, 0, 1); + int oldval = 0; + + return __atomic_compare_exchange_n (lock, &oldval, 1, false, + MEMMODEL_ACQUIRE, MEMMODEL_RELAXED); } void @@ -104,11 +107,14 @@ int gomp_test_nest_lock_30 (omp_nest_lock_t *lock) { void *me = gomp_icv (true); + int oldval; if (lock->owner == me) return ++lock->count; - if (__sync_bool_compare_and_swap (&lock->lock, 0, 1)) + oldval = 0; + if (__atomic_compare_exchange_n (&lock->lock, &oldval, 1, false, + MEMMODEL_ACQUIRE, MEMMODEL_RELAXED)) { lock->owner = me; lock->count = 1; @@ -184,8 +190,9 @@ gomp_set_nest_lock_25 (omp_nest_lock_25_t *lock) while (1) { - otid = __sync_val_compare_and_swap (&lock->owner, 0, tid); - if (otid == 0) + otid = 0; + if (__atomic_compare_exchange_n (&lock->owner, &otid, tid, false, + MEMMODEL_ACQUIRE, MEMMODEL_RELAXED)) { lock->count = 1; return; @@ -207,7 +214,7 @@ gomp_unset_nest_lock_25 (omp_nest_lock_25_t *lock) if (--lock->count == 0) { - __sync_lock_release (&lock->owner); + __atomic_store_n (&lock->owner, 0, MEMMODEL_RELEASE); futex_wake (&lock->owner, 1); } } @@ -217,8 +224,9 @@ gomp_test_nest_lock_25 (omp_nest_lock_25_t *lock) { int otid, tid = gomp_tid (); - otid = __sync_val_compare_and_swap (&lock->owner, 0, tid); - if (otid == 0) + otid = 0; + if (__atomic_compare_exchange_n (&lock->owner, &otid, tid, false, + MEMMODEL_ACQUIRE, MEMMODEL_RELAXED)) { lock->count = 1; return 1; diff --git a/libgomp/config/linux/mips/futex.h b/libgomp/config/linux/mips/futex.h index 576ada452ee3..93655d3114c1 100644 --- a/libgomp/config/linux/mips/futex.h +++ b/libgomp/config/linux/mips/futex.h @@ -64,9 +64,3 @@ cpu_relax (void) { __asm volatile ("" : : : "memory"); } - -static inline void -atomic_write_barrier (void) -{ - __sync_synchronize (); -} diff --git a/libgomp/config/linux/powerpc/futex.h b/libgomp/config/linux/powerpc/futex.h index efc05c4163f1..9d928e20441f 100644 --- a/libgomp/config/linux/powerpc/futex.h +++ b/libgomp/config/linux/powerpc/futex.h @@ -84,9 +84,3 @@ cpu_relax (void) { __asm volatile ("" : : : "memory"); } - -static inline void -atomic_write_barrier (void) -{ - __asm volatile ("eieio" : : : "memory"); -} diff --git a/libgomp/config/linux/ptrlock.c b/libgomp/config/linux/ptrlock.c index 7c2ad619282d..5896f424a329 100644 --- a/libgomp/config/linux/ptrlock.c +++ b/libgomp/config/linux/ptrlock.c @@ -1,4 +1,4 @@ -/* Copyright (C) 2008, 2009 Free Software Foundation, Inc. +/* Copyright (C) 2008, 2009, 2011 Free Software Foundation, Inc. Contributed by Jakub Jelinek . This file is part of the GNU OpenMP Library (libgomp). @@ -34,7 +34,10 @@ void * gomp_ptrlock_get_slow (gomp_ptrlock_t *ptrlock) { int *intptr; - __sync_bool_compare_and_swap (ptrlock, 1, 2); + uintptr_t oldval = 1; + + __atomic_compare_exchange_n (ptrlock, &oldval, 2, false, + MEMMODEL_RELAXED, MEMMODEL_RELAXED); /* futex works on ints, not pointers. But a valid work share pointer will be at least @@ -53,11 +56,10 @@ gomp_ptrlock_get_slow (gomp_ptrlock_t *ptrlock) } void -gomp_ptrlock_set_slow (gomp_ptrlock_t *ptrlock, void *ptr) +gomp_ptrlock_set_slow (gomp_ptrlock_t *ptrlock) { int *intptr; - *ptrlock = ptr; __asm volatile ("" : "=r" (intptr) : "0" (ptrlock)); #if __BYTE_ORDER == __BIG_ENDIAN if (sizeof (*ptrlock) > sizeof (int)) diff --git a/libgomp/config/linux/ptrlock.h b/libgomp/config/linux/ptrlock.h index 97a3a1ad9e5a..2f17e09cc7c2 100644 --- a/libgomp/config/linux/ptrlock.h +++ b/libgomp/config/linux/ptrlock.h @@ -1,4 +1,4 @@ -/* Copyright (C) 2008, 2009 Free Software Foundation, Inc. +/* Copyright (C) 2008, 2009, 2011 Free Software Foundation, Inc. Contributed by Jakub Jelinek . This file is part of the GNU OpenMP Library (libgomp). @@ -24,7 +24,14 @@ /* This is a Linux specific implementation of a mutex synchronization mechanism for libgomp. This type is private to the library. This - implementation uses atomic instructions and the futex syscall. */ + implementation uses atomic instructions and the futex syscall. + + A ptrlock has four states: + 0/NULL Initial + 1 Owned by me, I get to write a pointer to ptrlock. + 2 Some thread is waiting on the ptrlock. + >2 Ptrlock contains a valid pointer. + It is not valid to gain the ptrlock and then write a NULL to it. */ #ifndef GOMP_PTRLOCK_H #define GOMP_PTRLOCK_H 1 @@ -39,20 +46,25 @@ static inline void gomp_ptrlock_init (gomp_ptrlock_t *ptrlock, void *ptr) extern void *gomp_ptrlock_get_slow (gomp_ptrlock_t *ptrlock); static inline void *gomp_ptrlock_get (gomp_ptrlock_t *ptrlock) { + uintptr_t oldval; + if ((uintptr_t) *ptrlock > 2) return *ptrlock; - if (__sync_bool_compare_and_swap (ptrlock, NULL, (uintptr_t) 1)) + oldval = 0; + if (__atomic_compare_exchange_n (ptrlock, &oldval, 1, false, + MEMMODEL_ACQUIRE, MEMMODEL_ACQUIRE)) return NULL; return gomp_ptrlock_get_slow (ptrlock); } -extern void gomp_ptrlock_set_slow (gomp_ptrlock_t *ptrlock, void *ptr); +extern void gomp_ptrlock_set_slow (gomp_ptrlock_t *ptrlock); static inline void gomp_ptrlock_set (gomp_ptrlock_t *ptrlock, void *ptr) { - if (!__sync_bool_compare_and_swap (ptrlock, (uintptr_t) 1, ptr)) - gomp_ptrlock_set_slow (ptrlock, ptr); + void *wait = __atomic_exchange_n (ptrlock, ptr, MEMMODEL_RELEASE); + if ((uintptr_t) wait != 1) + gomp_ptrlock_set_slow (ptrlock); } static inline void gomp_ptrlock_destroy (gomp_ptrlock_t *ptrlock) diff --git a/libgomp/config/linux/s390/futex.h b/libgomp/config/linux/s390/futex.h index 060032d73d89..e015c921de88 100644 --- a/libgomp/config/linux/s390/futex.h +++ b/libgomp/config/linux/s390/futex.h @@ -76,9 +76,3 @@ cpu_relax (void) { __asm volatile ("" : : : "memory"); } - -static inline void -atomic_write_barrier (void) -{ - __sync_synchronize (); -} diff --git a/libgomp/config/linux/sparc/futex.h b/libgomp/config/linux/sparc/futex.h index 214b786f7f80..6c600446eb9c 100644 --- a/libgomp/config/linux/sparc/futex.h +++ b/libgomp/config/linux/sparc/futex.h @@ -96,13 +96,3 @@ cpu_relax (void) __asm volatile ("" : : : "memory"); #endif } - -static inline void -atomic_write_barrier (void) -{ -#if defined __arch64__ || defined __sparc_v9__ - __asm volatile ("membar #StoreStore" : : : "memory"); -#else - __sync_synchronize (); -#endif -} diff --git a/libgomp/config/linux/x86/futex.h b/libgomp/config/linux/x86/futex.h index 419f4d981c66..f6d26dc4918b 100644 --- a/libgomp/config/linux/x86/futex.h +++ b/libgomp/config/linux/x86/futex.h @@ -145,9 +145,3 @@ cpu_relax (void) { __asm volatile ("rep; nop" : : : "memory"); } - -static inline void -atomic_write_barrier (void) -{ - __sync_synchronize (); -}