Re: [patch 1/4] x86: FIFO ticket spinlocks

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Gregory Haskins <gregory.haskins.ml@...>
Cc: Nick Piggin <npiggin@...>, Linux Kernel Mailing List <linux-kernel@...>, Andi Kleen <ak@...>, Ingo Molnar <mingo@...>
Date: Thursday, November 1, 2007 - 12:38 pm

On Thu, 1 Nov 2007, Gregory Haskins wrote:

Can you test under interesting loads?

We're interested in:
 - is the unfairness fix really noticeable (or does it just move the 
   problem somewhere else, and there is no real change in behaviour)
 - what is the performance impact?

In particular, unfair spinlocks have the potential to perform much better. 
Not so much because the spinlock itself acts all that differently, but 
because being unfair also fundmanetally tends to keep the data structures 
that are *protected* by the spinlock on just one CPU.

So "unfair" is obviously always bad. Except when it isn't. I'd personally 
like to merge the ticket spinlocks, but I'd really like to have people who 
have real loads where they matter actually also do some performance 
testing. Because I do think it will potentially be a performance downer. 

(I obviously hope it won't be, but..)

		Linus
-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[patch 0/4] ticket spinlocks for x86, Nick Piggin, (Thu Nov 1, 10:01 am)
Re: [patch 0/4] ticket spinlocks for x86, Jeremy Fitzhardinge, (Sat Nov 3, 6:36 pm)
[patch 4/4] x86: spinlock.h merge, Nick Piggin, (Thu Nov 1, 10:05 am)
[patch 3/4] x86: spinlock.h merge prep, Nick Piggin, (Thu Nov 1, 10:04 am)
[patch 1/4] x86: FIFO ticket spinlocks, Nick Piggin, (Thu Nov 1, 10:03 am)
Re: [patch 1/4] x86: FIFO ticket spinlocks, Chuck Ebbert, (Fri Nov 2, 12:22 pm)
Re: [patch 1/4] x86: FIFO ticket spinlocks, Linus Torvalds, (Fri Nov 2, 12:51 pm)
Re: [patch 1/4] x86: FIFO ticket spinlocks, Nick Piggin, (Fri Nov 2, 7:01 pm)
Re: [patch 1/4] x86: FIFO ticket spinlocks, Chuck Ebbert, (Fri Nov 2, 8:56 pm)
Re: [patch 1/4] x86: FIFO ticket spinlocks, Nick Piggin, (Fri Nov 2, 11:41 pm)
Re: [patch 1/4] x86: FIFO ticket spinlocks, Chuck Ebbert, (Thu Nov 1, 4:01 pm)
Re: [patch 1/4] x86: FIFO ticket spinlocks, Nick Piggin, (Thu Nov 1, 8:00 pm)
Re: [patch 1/4] x86: FIFO ticket spinlocks, Gregory Haskins, (Thu Nov 1, 10:40 am)
Re: [patch 1/4] x86: FIFO ticket spinlocks, Linus Torvalds, (Thu Nov 1, 12:38 pm)
Re: [patch 1/4] x86: FIFO ticket spinlocks, Gregory Haskins, (Fri Nov 2, 10:24 am)
Re: [patch 1/4] x86: FIFO ticket spinlocks, Rik van Riel, (Thu Nov 1, 8:35 pm)
Re: [patch 1/4] x86: FIFO ticket spinlocks, Linus Torvalds, (Thu Nov 1, 9:19 pm)
Re: [patch 1/4] x86: FIFO ticket spinlocks, Nick Piggin, (Fri Nov 2, 2:42 am)
Re: [patch 1/4] x86: FIFO ticket spinlocks, Ingo Molnar, (Fri Nov 2, 11:33 am)
Re: [patch 1/4] x86: FIFO ticket spinlocks, Nick Piggin, (Wed Nov 7, 4:46 am)
Re: [patch 1/4] x86: FIFO ticket spinlocks, Rik van Riel, (Fri Nov 2, 10:05 am)
Re: [patch 1/4] x86: FIFO ticket spinlocks, Nick Piggin, (Fri Nov 2, 6:37 pm)
Re: [patch 1/4] x86: FIFO ticket spinlocks, Rik van Riel, (Thu Nov 1, 10:01 pm)
[patch 1/4] spinlock: lockbreak cleanup, Nick Piggin, (Thu Nov 1, 10:02 am)
Re: [patch 1/4] spinlock: lockbreak cleanup, Peter Zijlstra, (Thu Nov 1, 10:06 am)
Re: [patch 1/4] spinlock: lockbreak cleanup, Nick Piggin, (Thu Nov 1, 10:29 am)
Re: [patch 1/4] spinlock: lockbreak cleanup, Lee Schermerhorn, (Thu Nov 1, 11:39 am)
Re: [patch 1/4] spinlock: lockbreak cleanup, Ingo Molnar, (Thu Nov 1, 11:46 am)
Re: [patch 1/4] spinlock: lockbreak cleanup, Nick Piggin, (Thu Nov 1, 11:53 am)