linux-kernel - Re: [PATCH 1/2] ipc/sem.c: Race in sem

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <1379225343.5545.154.camel@marge.simpson.net>
Date:	Sun, 15 Sep 2013 08:09:03 +0200
From:	Mike Galbraith <efault@....de>
To:	Manfred Spraul <manfred@...orfullife.com>
Cc:	Andrew Morton <akpm@...ux-foundation.org>,
	Linus Torvalds <torvalds@...ux-foundation.org>,
	LKML <linux-kernel@...r.kernel.org>,
	Rik van Riel <riel@...hat.com>,
	Davidlohr Bueso <davidlohr.bueso@...com>, hhuang@...hat.com
Subject: Re: [PATCH 1/2] ipc/sem.c: Race in sem_lock()

On Sat, 2013-09-14 at 23:34 +0200, Manfred Spraul wrote:

> The bug is probably also present in 3.10 and 3.11, but for these kernels
> is is probably simpler just to move the test of sma->complex_count after
> the spin_is_locked() test.

IMHO, your 6 patch series should go to stable as well.  Scalability is
still BAD without them.  Now, you've shown the lock split to be buggy.

Logically, the whole thing should be reverted entirely in stable, or
fixed up properly.  Humongous improvements will find their way into
every long term stable kernel on the planet regardless, so..

Stable may as well do the Borg thing, resistance really is futile ;-)

-Mike

--
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/