Re: [PATCH] vmap_pages() (4th resend)

[Christoph Hellwig <hch@lst.de>]

On Mon, Jul 29, 2002 at 12:25:24PM -0700, Linus Torvalds wrote:
> > The old vmalloc_area_pages is renamed to __vmap_area_pages and
> > vmalloc_area_pages is a small wrapper around it, passing in an NULL page
> > array.  Similarly __vmalloc is renamed to vmap_pages and a small wrapper
> > is added.
> 
> I don't like the NULL page array.
> 
> I think vmalloc() should just allocate the pages and create the page 
> array, and vmap_pages() should never check for NULL. Ok?

I looked into implementing your suggestion the last, but the problem is
that we need to pass in a page array.  To do so we could either allocate
all the pages as high-order allocation or kmalloc() the array where we
place the pages.  Both doesn't seem very practical to me..

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