linux-kernel - Re: BUG in 2.6.20-rt8

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]

Date:	Sun, 25 Feb 2007 18:09:27 -0800 (PST)
From:	David Miller <davem@...emloft.net>
To:	paulmck@...ux.vnet.ibm.com
Cc:	mingo@...e.hu, tglx@...utronix.de, linux-kernel@...r.kernel.org,
	linux-rt-users@...r.kernel.org
Subject: Re: BUG in 2.6.20-rt8

From: "Paul E. McKenney" <paulmck@...ux.vnet.ibm.com>
Date: Sun, 25 Feb 2007 17:52:30 -0800

> Why doesn't the traditional hash table of locks work here?  Use the
> cache-line address as input to the hash function, take the corresponding
> lock, do the compare-and-exchange by hand, and then release the lock.
> What am I missing here?  Address aliasing do to memory being mapped into
> multiple locations or something?  (In that case, use only the portion
> of the address within the page, right?)

It doesn't protect against pure stores.

Even ignoring that you'll also need to disable interrupts, always,
which therefore will always perform very poorly compared to an
open-coded spinlock variant.

There are zero gains to cmpxchg(), in my opinion, you dirty and
grab exclusive access to a cacheline with cmpxchg() just the
same as a spinlock, so the real cost is the same.

There is zero justification for using this primitive in generic
code.
-
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/