From mboxrd@z Thu Jan 1 00:00:00 1970 Date: Tue, 24 Jul 2007 00:09:47 -0700 (PDT) From: Christoph Lameter Subject: Re: [PATCH] add __GFP_ZERO to GFP_LEVEL_MASK In-Reply-To: <1185256869.8197.27.camel@twins> Message-ID: References: <1185185020.8197.11.camel@twins> <20070723112143.GB19437@skynet.ie> <1185190711.8197.15.camel@twins> <1185256869.8197.27.camel@twins> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-linux-mm@kvack.org Return-Path: To: Peter Zijlstra Cc: Mel Gorman , Andrew Morton , Linus Torvalds , linux-kernel , Daniel Phillips , linux-mm List-ID: On Tue, 24 Jul 2007, Peter Zijlstra wrote: > Then we can either fixup the slab allocators to mask out __GFP_ZERO, or > do something like the below. > > Personally I like the consistency of adding __GFP_ZERO here (removes > this odd exception) and just masking it in the sl[aou]b thingies. There is another exception for __GFP_DMA. > Anybody else got a preference? > #define __GFP_BITS_MASK ((__force gfp_t)((1 << __GFP_BITS_SHIFT) - 1)) > > -/* if you forget to add the bitmask here kernel will crash, period */ > +/* > + * If you forget to add the bitmask here kernel will crash, period! > + * > + * GFP_LEVEL_MASK is used to filter out the flags that are to be passed to the > + * page allocator. > + * GFP_LEVEL_MASK is also used in mm/vmalloc.c. We need a definition that goes beyond slab allocators. -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org