Messages in this thread | | | Date | Tue, 1 Nov 2011 18:47:59 +0000 | From | Mel Gorman <> | Subject | Re: [PATCH 2/9] mm: alloc_contig_freed_pages() added |
| |
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
| |