On 04/12/2007, Nick Piggin <npiggin@xxxxxxx> wrote: > + gfp_flags = GFP_NOIO | __GFP_ZERO; > +#ifndef CONFIG_BLK_DEV_XIP > + gfp_flags |= __GFP_HIGHMEM; > +#endif > page = alloc_page(GFP_NOIO | __GFP_HIGHMEM | __GFP_ZERO); I think that should be alloc_page(gfp_flags), no? Cheers, Duane. -- "I never could learn to drink that blood and call it wine" - Bob Dylan - To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html