Update.
authorUlrich Drepper <drepper@redhat.com>
Thu, 19 Jun 2003 19:53:12 +0000 (19:53 +0000)
committerUlrich Drepper <drepper@redhat.com>
Thu, 19 Jun 2003 19:53:12 +0000 (19:53 +0000)
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.

nptl/ChangeLog
nptl/sysdeps/unix/sysv/linux/ia64/sem_post.c [deleted file]
nptl/sysdeps/unix/sysv/linux/powerpc/lowlevellock.h
nptl/sysdeps/unix/sysv/linux/powerpc/sem_post.c [new file with mode: 0644]
nptl/sysdeps/unix/sysv/linux/sem_post.c

index d3221e81f95566ec445f755875aece23bc3b9b28..26b8909e7f728d43021a24780ccbd3d3c035f6f0 100644 (file)
@@ -1,3 +1,17 @@
+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.
+
 2003-06-18  Ulrich Drepper  <drepper@redhat.com>
 
        * 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 (file)
index 0527abe..0000000
+++ /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 <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, write to the Free
-   Software Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA
-   02111-1307 USA.  */
-
-#include <errno.h>
-#include <sysdep.h>
-#include <lowlevellock.h>
-#include <internaltypes.h>
-#include <semaphore.h>
-
-#include <shlib-compat.h>
-
-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
index eba0ade2521f0e67c44b0d4f478bd2f7ed151d22..00ca257ca00cde948864fe392cc85a5008a07c33 100644 (file)
@@ -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 (file)
index 0000000..d9ee5d5
--- /dev/null
@@ -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 <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
+   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 <errno.h>
+#include <sysdep.h>
+#include <lowlevellock.h>
+#include <internaltypes.h>
+#include <semaphore.h>
+
+#include <shlib-compat.h>
+
+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
index 2da6ecd696c0ff2b8f23cf853a2fdb35c3883cbf..b4ee4cfc8ab1d93e617e6097744a0a39c72f572c 100644 (file)
@@ -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)