[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <1185307985.6586.50.camel@localhost>
Date: Tue, 24 Jul 2007 16:13:05 -0400
From: Trond Myklebust <trond.myklebust@....uio.no>
To: Chris Mason <chris.mason@...cle.com>
Cc: Nick Piggin <npiggin@...e.de>,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
Linux Memory Management List <linux-mm@...ck.org>,
linux-fsdevel@...r.kernel.org
Subject: Re: [PATCH RFC] extent mapped page cache
On Tue, 2007-07-24 at 16:00 -0400, Chris Mason wrote:
> On Tue, 10 Jul 2007 17:03:26 -0400
> Chris Mason <chris.mason@...cle.com> wrote:
>
> > This patch aims to demonstrate one way to replace buffer heads with a
> > few extent trees. Buffer heads provide a few different features:
> >
> > 1) Mapping of logical file offset to blocks on disk
> > 2) Recording state (dirty, locked etc)
> > 3) Providing a mechanism to access sub-page sized blocks.
> >
> > This patch covers #1 and #2, I'll start on #3 a little later next
> > week.
> >
> Well, almost. I decided to try out an rbtree instead of the radix,
> which turned out to be much faster. Even though individual operations
> are slower, the rbtree was able to do many fewer ops to accomplish the
> same thing, especially for merging extents together. It also uses much
> less ram.
The problem with an rbtree is that you can't use it together with RCU to
do lockless lookups. You can probably modify it to allocate nodes
dynamically (like the radix tree does) and thus make it RCU-compatible,
but then you risk losing the two main benefits that you list above.
Trond
-
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