diff options
author | Michal Nazarewicz <[email protected]> | 2012-01-30 13:23:47 +0100 |
---|---|---|
committer | Marek Szyprowski <[email protected]> | 2012-05-21 15:09:28 +0200 |
commit | 03d44192f69a45d780ba124f691e76020a44ebae (patch) | |
tree | 81b43ba18b0fa70280a20fa534d37b9d0c5e5478 | |
parent | 2fe86e0004076128f05d5a774b5c9c03d9dc3de2 (diff) |
mm: compaction: introduce map_pages()
This commit creates a map_pages() function which map pages freed
using split_free_pages(). This merely moves some code from
isolate_freepages() so that it can be reused in other places.
Signed-off-by: Michal Nazarewicz <[email protected]>
Signed-off-by: Marek Szyprowski <[email protected]>
Acked-by: Mel Gorman <[email protected]>
Reviewed-by: KAMEZAWA Hiroyuki <[email protected]>
Tested-by: Robert Nelson <[email protected]>
Tested-by: Barry Song <[email protected]>
-rw-r--r-- | mm/compaction.c | 15 |
1 files changed, 11 insertions, 4 deletions
diff --git a/mm/compaction.c b/mm/compaction.c index ee20fc044b91..d9d7b35d3933 100644 --- a/mm/compaction.c +++ b/mm/compaction.c @@ -127,6 +127,16 @@ static bool suitable_migration_target(struct page *page) return false; } +static void map_pages(struct list_head *list) +{ + struct page *page; + + list_for_each_entry(page, list, lru) { + arch_alloc_page(page, 0); + kernel_map_pages(page, 1, 1); + } +} + /* * Based on information in the current compact_control, find blocks * suitable for isolating free pages from and then isolate them. @@ -206,10 +216,7 @@ static void isolate_freepages(struct zone *zone, } /* split_free_page does not map the pages */ - list_for_each_entry(page, freelist, lru) { - arch_alloc_page(page, 0); - kernel_map_pages(page, 1, 1); - } + map_pages(freelist); cc->free_pfn = high_pfn; cc->nr_freepages = nr_freepages; |