lkml.org 
[lkml]   [2010]   [Feb]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] [3/4] SLAB: Set up the l3 lists for the memory of freshly added memory v2
On Mon, 15 Feb 2010, Nick Piggin wrote:

> > > @@ -1577,6 +1595,8 @@ void __init kmem_cache_init_late(void)
> > > */
> > > register_cpu_notifier(&cpucache_notifier);
> > >
> > > + hotplug_memory_notifier(slab_memory_callback, SLAB_CALLBACK_PRI);
> > > +
> >
> > Only needed for CONFIG_NUMA, but there's no side-effects for UMA kernels
> > since status_change_nid will always be -1.
>
> Compiler doesn't know that, though.
>

Right, setting up a memory hotplug callback for UMA kernels here isn't
necessary although slab_node_prepare() would have to be defined
unconditionally. I made this suggestion in my review of the patchset's
initial version but it was left unchanged, so I'd rather see it included
than otherwise stall out. This could always be enclosed in
#ifdef CONFIG_NUMA later just like the callback in slub does.


\
 
 \ /
  Last update: 2010-02-15 22:49    [W:0.368 / U:1.348 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site