about summary refs log tree commit diff
path: root/src/thread/sem_trywait.c
diff options
context:
space:
mode:
authorRich Felker <dalias@aerifal.cx>2011-04-06 12:24:34 -0400
committerRich Felker <dalias@aerifal.cx>2011-04-06 12:24:34 -0400
commita113434cd68ce30642c4995b1caadcd084be6f09 (patch)
treeb26daaeda2c9e2caf1605615c497a4a84191e01d /src/thread/sem_trywait.c
parentcd3bb38412cfcc3bc47985ba25287e0af463609a (diff)
downloadmusl-a113434cd68ce30642c4995b1caadcd084be6f09.tar.gz
musl-a113434cd68ce30642c4995b1caadcd084be6f09.tar.xz
musl-a113434cd68ce30642c4995b1caadcd084be6f09.zip
major semaphore improvements (performance and correctness)
1. make sem_[timed]wait interruptible by signals, per POSIX
2. keep a waiter count in order to avoid unnecessary futex wake syscalls
Diffstat (limited to 'src/thread/sem_trywait.c')
-rw-r--r--src/thread/sem_trywait.c5
1 files changed, 2 insertions, 3 deletions
diff --git a/src/thread/sem_trywait.c b/src/thread/sem_trywait.c
index c5370c70..beb7aa5d 100644
--- a/src/thread/sem_trywait.c
+++ b/src/thread/sem_trywait.c
@@ -3,9 +3,8 @@
 
 int sem_trywait(sem_t *sem)
 {
-	int val = a_fetch_add(sem->__val, -1);
-	if (val > 0) return 0;
-	if (!a_fetch_add(sem->__val, 1))
+	if (a_fetch_add(sem->__val, -1) > 0) return 0;
+	if (!a_fetch_add(sem->__val, 1) && sem->__val[1])
 		__wake(sem->__val, 1, 0);
 	errno = EAGAIN;
 	return -1;