| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-Id: <200908160049.n7G0n1Sg023832@mail.q-ag.de> Date: Sat, 15 Aug 2009 23:57:12 +0200 From: Manfred Spraul <manfred@...orfullife.com> To: linux-kernel@...r.kernel.org Cc: Pierre Peiffer <peifferp@...il.com> Subject: [PATCH 1/4] ipc/sem.c: optimize if semops fail This is the first of four patches that reduce the amount of scanning of the list of pending semaphore operations: Part 1: if try_atomic_semop failed, then no changes were applied. Thus no need to restart. Additionally, this patch correct an incorrect comment. The changes are from Nick Piggin, the patch is just splitted differently. Signed-off-by: Manfred Spraul <manfred@...orfullife.com> --- ipc/sem.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/ipc/sem.c b/ipc/sem.c index 3629ef8..9425817 100644 --- a/ipc/sem.c +++ b/ipc/sem.c @@ -444,13 +444,13 @@ again: * of the completed operation: * - if the operation modified the array, then restart from the * head of the queue and check for threads that might be - * waiting for semaphore values to become 0. + * waiting for the new semaphore values. * - if the operation didn't modify the array, then just * continue. */ alter = q->alter; wake_up_sem_queue(q, error); - if (alter) + if (alter && !error) goto again; } } -- 1.6.2.5 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists