Re: [PATCH,RFC] Add __GFP_ZERO to GFP_LEVEL_MASK

From: Andrew Morton
Date: Fri Oct 13 2006 - 16:46:58 EST


On Fri, 13 Oct 2006 14:25:16 -0600
Jonathan Corbet <corbet@xxxxxxx> wrote:

> There is a very helpful comment in <linux/gfp.h>:
>
> /* if you forget to add the bitmask here kernel will crash, period */
>
> Well, my kernel has been crashing (period) at the BUG() in cache_grow();
> the offending flag is __GFP_ZERO. I think it needs to be in
> GFP_LEVEL_MASK. Anybody know a good reason why it's not there now?
>

It would be a bit odd to pass __GFP_ZERO into the slab allocator. Slab
doesn't need that hint: it has its own ways of initialising the memory.

What is the callsite?

Thanks.

-
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/