Re: [PATCH 2/9] mm: alloc_contig_freed_pages() added

From: Mel Gorman
Date: Tue Nov 01 2011 - 14:48:12 EST


On Tue, Nov 01, 2011 at 07:06:56PM +0100, Michal Nazarewicz wrote:
> >page_isolation.c may also be a better fit than page_alloc.c
>
> Since isolate_freepages_block() is the only user of split_free_page(),
> would it make sense to move split_free_page() to page_isolation.c as
> well? I sort of like the idea of making it static and removing from
> header file.
>

I see no problem with that. It'll be separate from split_page() but that
is not earth shattering.

Thanks.

--
Mel Gorman
SUSE Labs
--
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/