login
Header Space

 
 

Re: [patch 09/10] Remove the SLOB allocator for 2.6.23

Score:
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Ingo Molnar <mingo@...>
Cc: Christoph Lameter <clameter@...>, <linux-kernel@...>, <linux-mm@...>, <suresh.b.siddha@...>, <corey.d.gough@...>, Pekka Enberg <penberg@...>, Matt Mackall <mpm@...>
Date: Sunday, July 8, 2007 - 2:02 pm

On Sun, 8 Jul 2007 09:51:19 +0200 Ingo Molnar <mingo@elte.hu> wrote:


I don't think a saving of a few k of text would justify slob's retention.

A reason for retaining slob would be that it has some O(n) memory saving
due to better packing, etc.  Indeed that was the reason for merging it in
the first place.  If slob no longer retains that advantage (wrt slub) then
we no longer need it.


Guys, look at this the other way.  Suppose we only had slub, and someone
came along and said "here's a whole new allocator which saves 4.5k of
text", would we merge it on that basis?  Hell no, it's not worth it.  What
we might do is to get motivated to see if we can make slub less porky under
appropriate config settings.


Let's not get sentimental about these things: in general, if there's any
reasonable way in which we can rid ourselves of any code at all, we should
do so, no?
-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[patch 09/10] Remove the SLOB allocator for 2.6.23, Christoph Lameter, (Sat Jul 7, 11:50 pm)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Matt Mackall, (Mon Jul 9, 4:52 pm)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Christoph Lameter, (Mon Jul 9, 12:00 pm)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Andrew Morton, (Sun Jul 8, 2:02 pm)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Matt Mackall, (Mon Jul 9, 5:57 pm)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Nick Piggin, (Sun Jul 8, 10:57 pm)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Christoph Lameter, (Mon Jul 9, 12:06 pm)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Christoph Lameter, (Mon Jul 9, 9:51 pm)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Matt Mackall, (Mon Jul 9, 10:32 pm)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Matt Mackall, (Tue Jul 10, 2:22 am)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Nick Piggin, (Tue Jul 10, 3:03 am)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Andrew Morton, (Mon Jul 9, 12:51 pm)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Matt Mackall, (Mon Jul 9, 7:09 pm)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Christoph Lameter, (Mon Jul 9, 1:26 pm)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Christoph Lameter, (Mon Jul 9, 9:56 pm)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Nick Piggin, (Mon Jul 9, 10:02 pm)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Christoph Lameter, (Mon Jul 9, 10:11 pm)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Matt Mackall, (Tue Jul 10, 4:32 am)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Christoph Lameter, (Tue Jul 10, 9:37 pm)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Matt Mackall, (Tue Jul 10, 10:06 pm)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Christoph Lameter, (Wed Jul 11, 2:06 pm)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Pekka J Enberg, (Wed Jul 11, 2:25 pm)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Nick Piggin, (Wed Jul 11, 8:33 pm)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Christoph Lameter, (Wed Jul 11, 2:33 pm)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Pekka J Enberg, (Wed Jul 11, 2:36 pm)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Håvard Skinnemoen, (Tue Jul 10, 5:01 am)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Nick Piggin, (Tue Jul 10, 5:11 am)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Håvard Skinnemoen, (Tue Jul 10, 5:21 am)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Nick Piggin, (Tue Jul 10, 3:09 am)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Christoph Lameter, (Tue Jul 10, 6:09 pm)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Matt Mackall, (Tue Jul 10, 7:12 pm)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Andrew Morton, (Mon Jul 9, 2:00 pm)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Pekka Enberg, (Mon Jul 9, 7:04 am)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Christoph Lameter, (Mon Jul 9, 12:08 pm)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Pekka J Enberg, (Tue Jul 10, 4:17 am)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Nick Piggin, (Tue Jul 10, 4:27 am)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Pekka Enberg, (Tue Jul 10, 5:31 am)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Matt Mackall, (Tue Jul 10, 8:02 am)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Christoph Lameter, (Tue Jul 10, 6:12 pm)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Matt Mackall, (Tue Jul 10, 6:40 pm)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Christoph Lameter, (Tue Jul 10, 6:50 pm)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Pekka J Enberg, (Tue Jul 10, 8:57 am)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Nick Piggin, (Tue Jul 10, 6:09 am)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Pekka J Enberg, (Mon Jul 9, 9:46 am)
Re: [patch 09/10] Remove the SLOB allocator for 2.6.23, Pekka Enberg, (Mon Jul 9, 8:47 am)
speck-geostationary