From 9bdabb94dac4be59ab69b3330457861ae997c940 Mon Sep 17 00:00:00 2001 From: Ulrich Drepper Date: Thu, 19 Jun 2003 19:53:12 +0000 Subject: Update. 2003-06-19 Jakub Jelinek * sysdeps/unix/sysv/linux/ia64/sem_post.c: Move to... * sysdeps/unix/sysv/linux/sem_post.c: ...here. * sysdeps/unix/sysv/linux/sem_post.c: Move to... * sysdeps/unix/sysv/linux/powerpc/sem_post.c: ... here. Pass nr + 1 instead of nr to lll_futex_wake. Only set errno and return -1 if err < 0. * sysdeps/unix/sysv/linux/powerpc/lowlevellock.h (lll_futex_wait, lll_futex_timed_wait, lll_futex_wake, lll_futex_requeue): On success return actual return value from the syscall, not 0. compat_timer_getoverrun. --- nptl/ChangeLog | 16 ++++++- nptl/sysdeps/unix/sysv/linux/ia64/sem_post.c | 48 --------------------- .../sysdeps/unix/sysv/linux/powerpc/lowlevellock.h | 8 ++-- nptl/sysdeps/unix/sysv/linux/powerpc/sem_post.c | 49 ++++++++++++++++++++++ nptl/sysdeps/unix/sysv/linux/sem_post.c | 15 +++---- 5 files changed, 76 insertions(+), 60 deletions(-) delete mode 100644 nptl/sysdeps/unix/sysv/linux/ia64/sem_post.c create mode 100644 nptl/sysdeps/unix/sysv/linux/powerpc/sem_post.c diff --git a/nptl/ChangeLog b/nptl/ChangeLog index d3221e81f9..26b8909e7f 100644 --- a/nptl/ChangeLog +++ b/nptl/ChangeLog @@ -1,3 +1,17 @@ +2003-06-19 Jakub Jelinek + + * sysdeps/unix/sysv/linux/ia64/sem_post.c: Move to... + * sysdeps/unix/sysv/linux/sem_post.c: ...here. + + * sysdeps/unix/sysv/linux/sem_post.c: Move to... + * sysdeps/unix/sysv/linux/powerpc/sem_post.c: ... here. Pass nr + 1 + instead of nr to lll_futex_wake. Only set errno and return -1 + if err < 0. + + * sysdeps/unix/sysv/linux/powerpc/lowlevellock.h (lll_futex_wait, + lll_futex_timed_wait, lll_futex_wake, lll_futex_requeue): On success + return actual return value from the syscall, not 0. + 2003-06-18 Ulrich Drepper * tst-cancel4.c (tf_msgsnd): Don't always use 100 as the type, @@ -10,7 +24,7 @@ * sysdeps/unix/sysv/linux/timer_gettime.c: Add prototype for compat_timer_gettime. * sysdeps/unix/sysv/linux/timer_getoverr.c: Add prototype for - compat_timer_getoverrun.o + compat_timer_getoverrun. * sysdeps/unix/sysv/linux/timer_delete.c: Add prototype for compat_timer_delete. diff --git a/nptl/sysdeps/unix/sysv/linux/ia64/sem_post.c b/nptl/sysdeps/unix/sysv/linux/ia64/sem_post.c deleted file mode 100644 index 0527abeba6..0000000000 --- a/nptl/sysdeps/unix/sysv/linux/ia64/sem_post.c +++ /dev/null @@ -1,48 +0,0 @@ -/* sem_post -- post to a POSIX semaphore. IA-64 version. - Copyright (C) 2003 Free Software Foundation, Inc. - This file is part of the GNU C Library. - Contributed by Jakub Jelinek , 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, write to the Free - Software Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA - 02111-1307 USA. */ - -#include -#include -#include -#include -#include - -#include - -int -__new_sem_post (sem_t *sem) -{ - int *futex = (int *) sem; - int err, nr; - - nr = atomic_exchange_and_add (futex, 1); - err = lll_futex_wake (futex, nr + 1); - if (__builtin_expect (err, 0) < 0) - { - __set_errno (-err); - return -1; - } - return 0; -} -versioned_symbol (libpthread, __new_sem_post, sem_post, GLIBC_2_1); -#if SHLIB_COMPAT (libpthread, GLIBC_2_0, GLIBC_2_1) -strong_alias (__new_sem_post, __old_sem_post) -compat_symbol (libpthread, __old_sem_post, sem_post, GLIBC_2_0); -#endif diff --git a/nptl/sysdeps/unix/sysv/linux/powerpc/lowlevellock.h b/nptl/sysdeps/unix/sysv/linux/powerpc/lowlevellock.h index eba0ade252..00ca257ca0 100644 --- a/nptl/sysdeps/unix/sysv/linux/powerpc/lowlevellock.h +++ b/nptl/sysdeps/unix/sysv/linux/powerpc/lowlevellock.h @@ -41,7 +41,7 @@ \ __ret = INTERNAL_SYSCALL (futex, __err, 4, \ (futexp), FUTEX_WAIT, (val), 0); \ - INTERNAL_SYSCALL_ERROR_P (__ret, __err)? -__ret: 0; \ + INTERNAL_SYSCALL_ERROR_P (__ret, __err) ? -__ret : __ret; \ }) #define lll_futex_timed_wait(futexp, val, timespec) \ @@ -51,7 +51,7 @@ \ __ret = INTERNAL_SYSCALL (futex, __err, 4, \ (futexp), FUTEX_WAIT, (val), (timespec)); \ - INTERNAL_SYSCALL_ERROR_P (__ret, __err)? -__ret: 0; \ + INTERNAL_SYSCALL_ERROR_P (__ret, __err) ? -__ret : __ret; \ }) #define lll_futex_wake(futexp, nr) \ @@ -61,7 +61,7 @@ \ __ret = INTERNAL_SYSCALL (futex, __err, 4, \ (futexp), FUTEX_WAKE, (nr), 0); \ - INTERNAL_SYSCALL_ERROR_P (__ret, __err)? -__ret: 0; \ + INTERNAL_SYSCALL_ERROR_P (__ret, __err) ? -__ret : __ret; \ }) #define lll_futex_requeue(futexp, nr_wake, nr_move, mutex) \ @@ -72,7 +72,7 @@ __ret = INTERNAL_SYSCALL (futex, __err, 5, \ (futexp), FUTEX_REQUEUE, (nr_wake), (nr_move), \ (mutex)); \ - INTERNAL_SYSCALL_ERROR_P (__ret, __err)? -__ret: 0; \ + INTERNAL_SYSCALL_ERROR_P (__ret, __err) ? -__ret : __ret; \ }) #ifdef UP diff --git a/nptl/sysdeps/unix/sysv/linux/powerpc/sem_post.c b/nptl/sysdeps/unix/sysv/linux/powerpc/sem_post.c new file mode 100644 index 0000000000..d9ee5d50b7 --- /dev/null +++ b/nptl/sysdeps/unix/sysv/linux/powerpc/sem_post.c @@ -0,0 +1,49 @@ +/* sem_post -- post to a POSIX semaphore. Powerpc version. + Copyright (C) 2003 Free Software Foundation, Inc. + This file is part of the GNU C Library. + Contributed by Paul Mackerras , 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, write to the Free + Software Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA + 02111-1307 USA. */ + +#include +#include +#include +#include +#include + +#include + +int +__new_sem_post (sem_t *sem) +{ + int *futex = (int *) sem; + int err, nr; + + __asm __volatile (__lll_rel_instr ::: "memory"); + nr = atomic_exchange_and_add (futex, 1); + err = lll_futex_wake (futex, nr + 1); + if (__builtin_expect (err, 0) < 0) + { + __set_errno (-err); + return -1; + } + return 0; +} +versioned_symbol (libpthread, __new_sem_post, sem_post, GLIBC_2_1); +#if SHLIB_COMPAT (libpthread, GLIBC_2_0, GLIBC_2_1) +strong_alias (__new_sem_post, __old_sem_post) +compat_symbol (libpthread, __old_sem_post, sem_post, GLIBC_2_0); +#endif diff --git a/nptl/sysdeps/unix/sysv/linux/sem_post.c b/nptl/sysdeps/unix/sysv/linux/sem_post.c index 2da6ecd696..b4ee4cfc8a 100644 --- a/nptl/sysdeps/unix/sysv/linux/sem_post.c +++ b/nptl/sysdeps/unix/sysv/linux/sem_post.c @@ -1,7 +1,7 @@ /* sem_post -- post to a POSIX semaphore. Generic futex-using version. Copyright (C) 2003 Free Software Foundation, Inc. This file is part of the GNU C Library. - Contributed by Paul Mackerras , 2003. + Contributed by Jakub Jelinek , 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 @@ -32,13 +32,14 @@ __new_sem_post (sem_t *sem) int *futex = (int *) sem; int err, nr; - __asm __volatile (__lll_rel_instr ::: "memory"); nr = atomic_exchange_and_add (futex, 1); - err = lll_futex_wake (futex, nr); - if (err == 0) - return 0; - __set_errno (-err); - return -1; + err = lll_futex_wake (futex, nr + 1); + if (__builtin_expect (err, 0) < 0) + { + __set_errno (-err); + return -1; + } + return 0; } versioned_symbol (libpthread, __new_sem_post, sem_post, GLIBC_2_1); #if SHLIB_COMPAT (libpthread, GLIBC_2_0, GLIBC_2_1) -- cgit 1.4.1