Re: [PATCH] don't use ld.bias

From: Jesse Barnes <jbarnes_at_sgi.com>
Date: 2003-07-22 23:11:54
On Mon, Jul 21, 2003 at 05:15:39PM -0700, Chen, Kenneth W wrote:
> > Patch to fix spin_lock to not use ld.bias since it causes a bunch of
> bus
> > activity that hurts scalability when a lock is contended.
> 
> It's dangerous to say something without knowing the whole context, but I
> can't resist asking why not fix the contended lock area instead?  Sounds
> like you have some serious contention.

Maybe I shouldn't have said 'contended'.  This causes problems even when
a lock is nearly uncontended.  I agree that fixing lock contention is a
good idea, but as I read this code, it'll only make the very best case
(totally uncontended) only a little better than if the ld.bias were just
an ld and it makes the average and contended cases _much_ worse.

I can get some data if you're not convinced by reading the code :)

Thanks,
Jesse
-
To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Received on Tue Jul 22 09:12:26 2003

This archive was generated by hypermail 2.1.8 : 2005-08-02 09:20:16 EST