If multiple simple decrements on the same semaphore are pending, then the
commitfd2f5f5f0f085a5ac55a369961a9e6b7fab27812
authorManfred Spraul <manfred@colorfullife.com>
Tue, 6 Oct 2009 21:46:41 +0000 (6 23:46 +0200)
committerjames toy <toyj@union.edu>
Tue, 6 Oct 2009 21:46:41 +0000 (6 23:46 +0200)
tree3206f148fcdbfd8e3ce8c41004214af2e1e33c22
parenta4c44b9bd856d625684e1da898cc6fe16a51c261
If multiple simple decrements on the same semaphore are pending, then the
current code scans all decrement operations, even if the semaphore value
is already 0.

The patch optimizes that: if the semaphore value is 0, then there is no
need to scan the q->alter entries.

Note that this is a common case: It happens if 100 decrements by one are
pending and now an increment by one increases the semaphore value from 0
to 1.  Without this patch, all 100 entries are scanned.  With the patch,
only one entry is scanned, then woken up.  Then the new rule triggers and
the scanning is aborted, without looking at the remaining 99 tasks.

With this patch, single sop increment/decrement by 1 are now O(1).
(same as with Nick's patch)

Signed-off-by: Manfred Spraul <manfred@colorfullife.com>
Cc: Nick Piggin <npiggin@suse.de>
Cc: Pierre Peiffer <peifferp@gmail.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
ipc/sem.c