Re: tipc_init(), WARNING: at arch/x86/mm/highmem_32.c:52, [2.6.24-rc4-git5: Reported regressions from 2.6.23]

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Linus Torvalds <torvalds@...>
Cc: Andrew Morton <akpm@...>, Matt Mackall <mpm@...>, Rafael J. Wysocki <rjw@...>, LKML <linux-kernel@...>, Christoph Lameter <clameter@...>
Date: Saturday, December 8, 2007 - 3:52 pm

* Linus Torvalds <torvalds@linux-foundation.org> wrote:


i dont think the SLUB problem could be explained purely via a double 
memset(). [which ought to be extremely fast anyway] We are talking about 
a 10 times slowdown on a 64-way box of a workload that is fairly 
common-sense. (tasks sending messages to each other via bog standard 
means)

while i dont want to jump to conclusions without looking at some 
profiles, i think the SLUB performance regression is indicative of the 
following fallacy: "SLAB can be done significantly simpler while keeping 
the same performance".

I couldnt point to any particular aspect of SLAB that i could 
characterise as "needless bloat".

the SLUB concept is proudly outlined in init/Kconfig:

  config SLUB
        bool "SLUB (Unqueued Allocator)"
        help
           SLUB is a slab allocator that minimizes cache line usage
           instead of managing queues of cached objects (SLAB approach).
           Per cpu caching is realized using slabs of objects instead
           of queues of objects. SLUB can use memory efficiently
           and has enhanced diagnostics.

but that's not true anymore - the two concepts are pretty much 
equivalent, after all the "performance tuning" that went on in SLUB. 
(read: 'frantically try to catch up with SLAB in benchmarks')

so even today's upstream kernel, which has 'ancient' SLUB code, SLAB and 
SLUB have essentially the same linecount:

  $ wc -l mm/slab.c mm/slub.c
  4478 mm/slab.c
  4125 mm/slub.c

(and while linecount != complexity, there is a strong relationship.)

With SLAB having 10 years more test coverage and tuning.

the messiest and most fragile aspect of SLAB that i can think of is its 
bootstrap hacks - but that is an entirely unimportant detail in my 
opinion. SLAB has been cleaned up significantly in the past few years by 
Pekka Enberg & co, it's pretty pleasant and straightforward code these 
days.

I think we should we make SLAB the default for v2.6.24 ...

	Ingo
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
2.6.24-rc4-git5: Reported regressions from 2.6.23, Rafael J. Wysocki, (Fri Dec 7, 10:40 pm)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Ingo Molnar, (Mon Dec 10, 4:42 pm)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Andrew Morton, (Mon Dec 10, 4:59 pm)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Ingo Molnar, (Mon Dec 10, 6:45 pm)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Ingo Molnar, (Mon Dec 10, 7:04 pm)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Stefano Brivio, (Mon Dec 10, 7:34 pm)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Ingo Molnar, (Tue Dec 11, 5:01 am)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Stefano Brivio, (Tue Dec 11, 5:10 pm)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Guillaume Chazarain, (Mon Dec 10, 7:53 pm)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Ingo Molnar, (Tue Dec 11, 4:48 am)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Arjan van de Ven, (Mon Dec 10, 7:56 pm)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Guillaume Chazarain, (Mon Dec 10, 8:01 pm)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Arjan van de Ven, (Mon Dec 10, 9:06 pm)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Ingo Molnar, (Tue Dec 11, 4:43 am)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Guillaume Chazarain, (Mon Dec 10, 4:57 pm)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Andrew Morton, (Sun Dec 9, 7:54 am)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Rafael J. Wysocki, (Sun Dec 9, 10:24 am)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Fabio Comolli, (Sat Dec 8, 2:53 am)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Andrew Morton, (Sat Dec 8, 5:23 am)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Rafael J. Wysocki, (Sat Dec 8, 6:11 pm)
Re: tipc_init(), WARNING: at arch/x86/mm/highmem_32.c:52, [2..., Christoph Lameter, (Thu Dec 13, 6:03 pm)
Re: tipc_init(), WARNING: at arch/x86/mm/highmem_32.c:52, [2..., Rafael J. Wysocki, (Sun Dec 9, 10:17 am)
Re: tipc_init(), WARNING: at arch/x86/mm/highmem_32.c:52, [2..., Ingo Molnar, (Sat Dec 8, 3:52 pm)
Re: tipc_init(), WARNING: at arch/x86/mm/highmem_32.c:52, [2..., Christoph Lameter, (Fri Dec 14, 12:07 am)
Re: tipc_init(), WARNING: at arch/x86/mm/highmem_32.c:52, [2..., Christoph Lameter, (Thu Dec 13, 6:07 pm)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Andrew Morton, (Sat Dec 8, 5:52 am)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Linus Torvalds, (Sun Dec 9, 2:41 pm)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Linus Torvalds, (Sun Dec 9, 9:57 pm)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Ingo Molnar, (Mon Dec 10, 4:21 am)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Ingo Molnar, (Mon Dec 10, 4:41 am)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Linus Torvalds, (Mon Dec 10, 11:38 am)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Linus Torvalds, (Sun Dec 9, 2:36 pm)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Andrew Morton, (Sat Dec 8, 5:46 am)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Andrew Morton, (Sat Dec 8, 5:42 am)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Roland Dreier, (Sat Dec 8, 2:57 pm)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Theodore Tso, (Sat Dec 8, 3:40 pm)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Rafael J. Wysocki, (Sat Dec 8, 6:30 pm)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Theodore Tso, (Sat Dec 8, 10:15 pm)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Takashi Iwai, (Thu Dec 13, 6:49 am)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Takashi Iwai, (Thu Dec 20, 11:42 am)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Andrew Morton, (Sat Dec 8, 5:36 am)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Rafael J. Wysocki, (Sun Dec 9, 10:20 am)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Andreas Mohr, (Sat Dec 8, 6:12 am)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Andrew Morton, (Sat Dec 8, 6:20 am)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Andreas Mohr, (Sat Dec 8, 6:55 am)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Andreas Mohr, (Sun Dec 9, 3:59 pm)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Matthew Garrett, (Sat Dec 8, 6:28 am)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Andrew Morton, (Sat Dec 8, 5:29 am)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Rafael J. Wysocki, (Sat Dec 8, 6:17 pm)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Richard Purdie, (Sat Dec 8, 6:44 am)
Re: 2.6.24-rc4-git5: Reported regressions from 2.6.23, Rafael J. Wysocki, (Sat Dec 8, 6:32 pm)