[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20100525154352.GB20853@laptop>
Date: Wed, 26 May 2010 01:43:52 +1000
From: Nick Piggin <npiggin@...e.de>
To: Linus Torvalds <torvalds@...ux-foundation.org>
Cc: Pekka Enberg <penberg@...helsinki.fi>,
Christoph Lameter <cl@...ux-foundation.org>,
Christoph Lameter <cl@...ux.com>, linux-mm@...ck.org,
LKML <linux-kernel@...r.kernel.org>,
Andrew Morton <akpm@...ux-foundation.org>,
David Rientjes <rientjes@...gle.com>,
Zhang Yanmin <yanmin_zhang@...ux.intel.com>,
Matthew Wilcox <willy@...ux.intel.com>,
Matt Mackall <mpm@...enic.com>, Mel Gorman <mel@....ul.ie>
Subject: Re: [RFC V2 SLEB 00/14] The Enhanced(hopefully) Slab Allocator
On Tue, May 25, 2010 at 08:13:50AM -0700, Linus Torvalds wrote:
>
>
> On Tue, 25 May 2010, Pekka Enberg wrote:
> >
> > I would have liked to see SLQB merged as well but it just didn't happen.
>
> And it's not going to. I'm not going to merge YASA that will stay around
> for years, not improve on anything, and will just mean that there are some
> bugs that developers don't see because they depend on some subtle
> interaction with the sl*b allocator.
>
> We've got three. That's at least one too many. We're not adding any new
> ones until we've gotten rid of at least one old one.
No agree and realized that a while back (hence stop pushing SLQB).
SLAB is simply a good allocator that is very very hard to beat. The
fact that a lot of places are still using SLAB despite the real
secondary advantages of SLUB (cleaner code, better debugging support)
indicate to me that we should go back and start from there.
What is sad is all this duplicate (and unsynchronized and not always
complete) work implementing things in both the allocators[*] and
split testing base.
As far as I can see, there was never a good reason to replace SLAB
rather than clean up its code and make incremental improvements.
--
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