about summary refs log tree commit diff
path: root/nptl/sysdeps/unix/sysv/linux/powerpc/sem_post.c
diff options
context:
space:
mode:
authorUlrich Drepper <drepper@redhat.com>2007-05-15 06:24:31 +0000
committerUlrich Drepper <drepper@redhat.com>2007-05-15 06:24:31 +0000
commit83d87915da34fc02d170ff8c356406040da2823f (patch)
tree7d76dcc1f12bd5dd7d65b329a3cd37422be12a82 /nptl/sysdeps/unix/sysv/linux/powerpc/sem_post.c
parent1d47e92f71c36165364c8807e52d0d0dc0f66142 (diff)
downloadglibc-83d87915da34fc02d170ff8c356406040da2823f.tar.gz
glibc-83d87915da34fc02d170ff8c356406040da2823f.tar.xz
glibc-83d87915da34fc02d170ff8c356406040da2823f.zip
* sysdeps/unix/sysv/linux/sem_post.c: Only wake threads if old
	value of semaphore was zero.
	* sysdeps/unix/sysv/linux/i386/i486/sem_post.S: Likewise.
	* sysdeps/unix/sysv/linux/powerpc/sem_post.c: Likewise.
	* sysdeps/unix/sysv/linux/sparc/sparc32/sem_post.c: Likewise.
	* sysdeps/unix/sysv/linux/x86_64/sem_post.S: Likewise.

	* sysdeps/unix/sysv/linux/x86_64/sem_wait.S: Remove unnecessary
	extra cancellation test.
	* sysdeps/unix/sysv/linux/x86_64/sem_timedwait.S: Likewise.
Diffstat (limited to 'nptl/sysdeps/unix/sysv/linux/powerpc/sem_post.c')
-rw-r--r--nptl/sysdeps/unix/sysv/linux/powerpc/sem_post.c14
1 files changed, 8 insertions, 6 deletions
diff --git a/nptl/sysdeps/unix/sysv/linux/powerpc/sem_post.c b/nptl/sysdeps/unix/sysv/linux/powerpc/sem_post.c
index 91b9955181..86dd0ebb3b 100644
--- a/nptl/sysdeps/unix/sysv/linux/powerpc/sem_post.c
+++ b/nptl/sysdeps/unix/sysv/linux/powerpc/sem_post.c
@@ -1,5 +1,5 @@
 /* sem_post -- post to a POSIX semaphore.  Powerpc version.
-   Copyright (C) 2003, 2004 Free Software Foundation, Inc.
+   Copyright (C) 2003, 2004, 2007 Free Software Foundation, Inc.
    This file is part of the GNU C Library.
    Contributed by Paul Mackerras <paulus@au.ibm.com>, 2003.
 
@@ -32,12 +32,14 @@ __new_sem_post (sem_t *sem)
   int *futex = (int *) sem;
 
   __asm __volatile (__lll_rel_instr ::: "memory");
-  int nr = atomic_increment_val (futex);
-  int err = lll_futex_wake (futex, nr);
-  if (__builtin_expect (err, 0) < 0)
+  if (atomic_increment_val (futex) == 1)
     {
-      __set_errno (-err);
-      return -1;
+      int err = lll_futex_wake (futex, 1);
+      if (__builtin_expect (err, 0) < 0)
+	{
+	  __set_errno (-err);
+	  return -1;
+	}
     }
   return 0;
 }