[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20090107145621.GB23455@elte.hu>
Date: Wed, 7 Jan 2009 15:56:21 +0100
From: Ingo Molnar <mingo@...e.hu>
To: Matthew Wilcox <matthew@....cx>
Cc: Chris Mason <chris.mason@...cle.com>,
Nick Piggin <nickpiggin@...oo.com.au>,
Peter Zijlstra <peterz@...radead.org>,
Andi Kleen <andi@...stfloor.org>,
Andrew Morton <akpm@...ux-foundation.org>,
linux-kernel@...r.kernel.org,
linux-fsdevel <linux-fsdevel@...r.kernel.org>,
linux-btrfs <linux-btrfs@...r.kernel.org>,
Thomas Gleixner <tglx@...utronix.de>,
Steven Rostedt <rostedt@...dmis.org>,
Gregory Haskins <ghaskins@...ell.com>
Subject: Re: Btrfs for mainline
* Matthew Wilcox <matthew@....cx> wrote:
> On Wed, Jan 07, 2009 at 02:07:42PM +0100, Ingo Molnar wrote:
> > * Chris Mason <chris.mason@...cle.com> wrote:
> > > All of this is a long way of saying the btrfs locking scheme is far from
> > > perfect. I'll look harder at the loop and ways to get rid of it.
> >
> > <ob'plug>
> >
> > adaptive spinning mutexes perhaps? Such as:
>
> Um, I don't know how your mail client does threading, but mine shows
> Peter's message introducing the adaptive spinning mutexes as a reply to
> one of Chris' messages in the btrfs thread.
>
> Chris is just saying he'll look at other ways to not need the spinning
> mutexes.
But those are not the same spinning mutexes. Chris wrote his mail on Jan
05, Peter his first mail about spin-mutexes on Jan 06, as a reaction to
Chris's mail.
My reply links back the discussion to the original analysis from Chris
pointing out that it would be nice to try BTRFS with plain mutexes plus
Peter's patch - instead of throwing away BTRFS's locking design or
anything intrusive like that.
Where's the problem? :)
Ingo
--
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/
Powered by blists - more mailing lists