diff options
| author | Ulrich Drepper <drepper@redhat.com> | 2003-06-19 19:53:12 +0000 |
|---|---|---|
| committer | Ulrich Drepper <drepper@redhat.com> | 2003-06-19 19:53:12 +0000 |
| commit | 9bdabb94dac4be59ab69b3330457861ae997c940 (patch) | |
| tree | 0d1ae3209cc907c1468d55788142191527e1054e /nptl/sysdeps/unix/sysv/linux | |
| parent | 4e502c601d4522c846b779e82a1786469ea0a02d (diff) | |
| download | glibc-9bdabb94dac4be59ab69b3330457861ae997c940.tar.gz glibc-9bdabb94dac4be59ab69b3330457861ae997c940.tar.xz glibc-9bdabb94dac4be59ab69b3330457861ae997c940.zip | |
Update.
2003-06-19 Jakub Jelinek <jakub@redhat.com> * 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.
Diffstat (limited to 'nptl/sysdeps/unix/sysv/linux')
| -rw-r--r-- | nptl/sysdeps/unix/sysv/linux/powerpc/lowlevellock.h | 8 | ||||
| -rw-r--r-- | nptl/sysdeps/unix/sysv/linux/powerpc/sem_post.c (renamed from nptl/sysdeps/unix/sysv/linux/ia64/sem_post.c) | 5 | ||||
| -rw-r--r-- | nptl/sysdeps/unix/sysv/linux/sem_post.c | 15 |
3 files changed, 15 insertions, 13 deletions
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/ia64/sem_post.c b/nptl/sysdeps/unix/sysv/linux/powerpc/sem_post.c index 0527abeba6..d9ee5d50b7 100644 --- a/nptl/sysdeps/unix/sysv/linux/ia64/sem_post.c +++ b/nptl/sysdeps/unix/sysv/linux/powerpc/sem_post.c @@ -1,7 +1,7 @@ -/* sem_post -- post to a POSIX semaphore. IA-64 version. +/* 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 Jakub Jelinek <jakub@redhat.com>, 2003. + Contributed by Paul Mackerras <paulus@au.ibm.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 @@ -32,6 +32,7 @@ __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) 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 <paulus@au.ibm.com>, 2003. + 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 @@ -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) |