Re: [PATCH] slab: Ignore internal flags in cache creation

From: David Rientjes
Date: Tue Sep 25 2012 - 20:46:52 EST


On Tue, 25 Sep 2012, Christoph Lameter wrote:

> > No cache should ever pass those as a creation flags. We can just ignore
> > this bit if it happens to be passed (such as when duplicating a cache in
> > the kmem memcg patches)
>
> Acked-by: Christoph Lameter <cl@xxxxxxxxx>
>

Nack, this is already handled by CREATE_MASK in the mm/slab.c allocator;
the flag extensions beyond those defined in the generic slab.h header are
implementation defined. It may be true that SLAB uses a bit only
internally (and already protects it with a BUG_ON() in
__kmem_cache_create()) but that doesn't mean other implementations can't
use such a flag that would be a no-op on another allocator.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/