Re: i386 spinlock fairness: bizarre test results
From: Linus Torvalds (torvalds_at_osdl.org)
Date: 10/11/05
- Previous message: Jiri Slaby: "Re: sys_sendfile oops in 2.6.13?"
- In reply to: Alan Cox: "Re: i386 spinlock fairness: bizarre test results"
- Next in thread: Eric Dumazet: "[PATCH] i386 spinlocks should use the full 32 bits, not only 8 bits"
- Reply: Eric Dumazet: "[PATCH] i386 spinlocks should use the full 32 bits, not only 8 bits"
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Date: Tue, 11 Oct 2005 07:44:43 -0700 (PDT) To: Alan Cox <alan@lxorguk.ukuu.org.uk>
On Tue, 11 Oct 2005, Alan Cox wrote:
> On Maw, 2005-10-11 at 00:04 -0400, Chuck Ebbert wrote:
> > That test machine was a dual 350MHz Pentium II Xeon; on a dual 333MHz Pentium II
> > Overdrive (with very slow Socket 8 bus) I could not reproduce those results.
> > However, on that machine the 'xchg' instruction made the test run almost 20%
> > _faster_ than using 'mov'.
> >
> > So I think the i386 spinlock code should be changed to always use 'xchg' to do
> > spin_unlock.
>
>
> Using xchg on the spin unlock path is expensive. Really expensive on P4
> compared to movb. It also doesn't guarantee anything either way around
> especially as you go to four cores or change CPU (or in some cases quite
> likely even chipset).
Indeed.
I suspect that the behaviour Chuck saw is (a) only present under
contention and (b) very much dependent on other timing issues.
(a) is the wrong thing to optimize for, and (b) means that Chuck's numbers
aren't reliable anyway (as shown by the fact that things like instruction
alignment matters, and by Eric's numbers on other machines).
We want the spinlocks to behave well when they are _not_ under heavy
contention. If a spinlock gets so much contention that it starts having
these kinds of issues, then there's something wrong at higher levels, and
the fix is to use a different algorithm, or use a different kind of lock.
Spinlocks by definition are the _simplest_ locks there are. Not the
smartest or most fair. Trying to make them anything else is kind of
missing the whole point of them.
Linus
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
- Previous message: Jiri Slaby: "Re: sys_sendfile oops in 2.6.13?"
- In reply to: Alan Cox: "Re: i386 spinlock fairness: bizarre test results"
- Next in thread: Eric Dumazet: "[PATCH] i386 spinlocks should use the full 32 bits, not only 8 bits"
- Reply: Eric Dumazet: "[PATCH] i386 spinlocks should use the full 32 bits, not only 8 bits"
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Relevant Pages
|
