From mboxrd@z Thu Jan 1 00:00:00 1970 From: Linus Torvalds Subject: Re: [PATCH -v7][RFC]: mutex: implement adaptive spinning Date: Thu, 8 Jan 2009 11:13:29 -0800 (PST) Message-ID: References: <1231441350.14304.48.camel@think.oraclecorp.com> Mime-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Cc: Steven Rostedt , Peter Zijlstra , Ingo Molnar , paulmck@linux.vnet.ibm.com, Gregory Haskins , Matthew Wilcox , Andi Kleen , Andrew Morton , Linux Kernel Mailing List , linux-fsdevel , linux-btrfs , Thomas Gleixner , Nick Piggin , Peter Morreale , Sven Dietrich To: Chris Mason Return-path: In-Reply-To: <1231441350.14304.48.camel@think.oraclecorp.com> List-ID: On Thu, 8 Jan 2009, Chris Mason wrote: > > It is less fair though, the 50 proc parallel creates had a much bigger > span between the first and last proc's exit time. This isn't a huge > shock, I think it shows the hot path is closer to a real spin lock. Actually, the real spin locks are now fair. We use ticket locks on x86. Well, at least we do unless you enable that broken paravirt support. I'm not at all clear on why CONFIG_PARAVIRT wants to use inferior locks, but I don't much care. We _could_ certainly aim for using ticket locks for mutexes too, that might be quite nice. But yes, from a throughput standpoint fairness is almost always a bad thing, so your numbers could easily go down if we did. Linus