diff options
author | Torvald Riegel <triegel@redhat.com> | 2013-05-08 16:35:10 +0200 |
---|---|---|
committer | Torvald Riegel <triegel@redhat.com> | 2014-04-11 15:54:19 +0200 |
commit | 36875b06e0ed7f137190b9228bef553adfc338ba (patch) | |
tree | 347e3826873e48eb6433e6d09666d7cef8904766 /ports | |
parent | 579db35a068e70a4f3112000778138ede6994ac0 (diff) | |
download | glibc-36875b06e0ed7f137190b9228bef553adfc338ba.tar.gz glibc-36875b06e0ed7f137190b9228bef553adfc338ba.tar.xz glibc-36875b06e0ed7f137190b9228bef553adfc338ba.zip |
Fixed and unified pthread_once.
[BZ #15215] This unifies various pthread_once architecture-specific implementations which were using the same algorithm with slightly different implementations. It also adds missing memory barriers that are required for correctness.
Diffstat (limited to 'ports')
-rw-r--r-- | ports/ChangeLog.hppa | 5 | ||||
-rw-r--r-- | ports/sysdeps/unix/sysv/linux/hppa/nptl/pthread_once.c | 93 |
2 files changed, 5 insertions, 93 deletions
diff --git a/ports/ChangeLog.hppa b/ports/ChangeLog.hppa index 39c77018a8..3efd186109 100644 --- a/ports/ChangeLog.hppa +++ b/ports/ChangeLog.hppa @@ -1,3 +1,8 @@ +2014-04-11 Torvald Riegel <triegel@redhat.com> + + [BZ #15215] + * sysdeps/unix/sysv/linux/hppa/nptl/pthread_once.c: Remove file. + 2014-04-09 Adhemerval Zanella <azanella@linux.vnet.ibm.com> * ports/sysdeps/unix/sysv/linux/hppa/nptl/bits/pthreadtypes.h diff --git a/ports/sysdeps/unix/sysv/linux/hppa/nptl/pthread_once.c b/ports/sysdeps/unix/sysv/linux/hppa/nptl/pthread_once.c deleted file mode 100644 index ee6b496e4f..0000000000 --- a/ports/sysdeps/unix/sysv/linux/hppa/nptl/pthread_once.c +++ /dev/null @@ -1,93 +0,0 @@ -/* Copyright (C) 2003-2014 Free Software Foundation, Inc. - This file is part of the GNU C Library. - Contributed by Jakub Jelinek <jakub@redhat.com>, 2003. - - The GNU C Library is free software; you can redistribute it and/or - modify it under the terms of the GNU Lesser General Public - License as published by the Free Software Foundation; either - version 2.1 of the License, or (at your option) any later version. - - The GNU C Library is distributed in the hope that it will be useful, - but WITHOUT ANY WARRANTY; without even the implied warranty of - MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU - Lesser General Public License for more details. - - You should have received a copy of the GNU Lesser General Public - License along with the GNU C Library. If not, see - <http://www.gnu.org/licenses/>. */ - -#include "pthreadP.h" -#include <lowlevellock.h> - - -unsigned long int __fork_generation attribute_hidden; - - -static void -clear_once_control (void *arg) -{ - pthread_once_t *once_control = (pthread_once_t *) arg; - - *once_control = 0; - lll_private_futex_wake (once_control, INT_MAX); -} - - -int -__pthread_once (once_control, init_routine) - pthread_once_t *once_control; - void (*init_routine) (void); -{ - while (1) - { - int oldval, val, newval; - - val = *once_control; - do - { - /* Check if the initialized has already been done. */ - if ((val & 2) != 0) - return 0; - - oldval = val; - newval = (oldval & 3) | __fork_generation | 1; - val = atomic_compare_and_exchange_val_acq (once_control, newval, - oldval); - } - while (__builtin_expect (val != oldval, 0)); - - /* Check if another thread already runs the initializer. */ - if ((oldval & 1) != 0) - { - /* Check whether the initializer execution was interrupted - by a fork. */ - if (((oldval ^ newval) & -4) == 0) - { - /* Same generation, some other thread was faster. Wait. */ - lll_private_futex_wait (once_control, newval); - continue; - } - } - - /* This thread is the first here. Do the initialization. - Register a cleanup handler so that in case the thread gets - interrupted the initialization can be restarted. */ - pthread_cleanup_push (clear_once_control, once_control); - - init_routine (); - - pthread_cleanup_pop (0); - - - /* Add one to *once_control. */ - atomic_increment (once_control); - - /* Wake up all other threads. */ - lll_private_futex_wake (once_control, INT_MAX); - break; - } - - return 0; -} -weak_alias (__pthread_once, pthread_once) -hidden_def (__pthread_once) |