On Tue, 20 Mar 2012, Christoph Lameter wrote: > > diff --git a/mm/slub.c b/mm/slub.c > > index 4907563..01cf99d 100644 > > --- a/mm/slub.c > > +++ b/mm/slub.c > > @@ -3238,7 +3238,7 @@ static struct kmem_cache *__init create_kmalloc_cache(const char *name, > > * This function is called with IRQs disabled during early-boot on > > * single CPU so there's no need to take slub_lock here. > > */ > > - if (!kmem_cache_open(s, name, size, ARCH_KMALLOC_MINALIGN, > > + if (!kmem_cache_open(s, name, size, ALIGN_OF_LAST_BIT(size), > > flags, NULL)) > > goto panic; > > Why does the alignment of struct kmem_cache change? I'd rather have a > __alignof__(struct kmem_cache) here with alignment specified with the > struct definition. Ok this aligns the data not the cache . Ok I see what is going on here. So the kmalloc array now has a higher alignment. That means you can align up to that limit within the structure. -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: <a href=mailto:"dont@xxxxxxxxx"> email@xxxxxxxxx </a>