diff options
author | Mel Gorman <mel@csn.ul.ie> | 2007-10-16 04:25:55 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-10-16 12:43:00 -0400 |
commit | 46dafbca2bba811665b01d8cedf911204820623c (patch) | |
tree | c0dbc78b3da9749f257fa8756cd3133f39cd4f88 | |
parent | 5adc5be7cd1bcef6bb64f5255d2a33f20a3cf5be (diff) |
Be more agressive about stealing when MIGRATE_RECLAIMABLE allocations fallback
MIGRATE_RECLAIMABLE allocations tend to be very bursty in nature like when
updatedb starts. It is likely this will occur in situations where MAX_ORDER
blocks of pages are not free. This means that updatedb can scatter
MIGRATE_RECLAIMABLE pages throughout the address space. This patch is more
agressive about stealing blocks of pages for MIGRATE_RECLAIMABLE.
Signed-off-by: Mel Gorman <mel@csn.ul.ie>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | mm/page_alloc.c | 18 |
1 files changed, 15 insertions, 3 deletions
diff --git a/mm/page_alloc.c b/mm/page_alloc.c index e1d87ee1d9c6..b864584c92b4 100644 --- a/mm/page_alloc.c +++ b/mm/page_alloc.c | |||
@@ -821,11 +821,23 @@ retry: | |||
821 | 821 | ||
822 | /* | 822 | /* |
823 | * If breaking a large block of pages, move all free | 823 | * If breaking a large block of pages, move all free |
824 | * pages to the preferred allocation list | 824 | * pages to the preferred allocation list. If falling |
825 | * back for a reclaimable kernel allocation, be more | ||
826 | * agressive about taking ownership of free pages | ||
825 | */ | 827 | */ |
826 | if (unlikely(current_order >= MAX_ORDER / 2)) { | 828 | if (unlikely(current_order >= MAX_ORDER / 2) || |
829 | start_migratetype == MIGRATE_RECLAIMABLE) { | ||
830 | unsigned long pages; | ||
831 | pages = move_freepages_block(zone, page, | ||
832 | start_migratetype); | ||
833 | |||
834 | /* Claim the whole block if over half of it is free */ | ||
835 | if ((pages << current_order) >= (1 << (MAX_ORDER-2)) && | ||
836 | migratetype != MIGRATE_HIGHATOMIC) | ||
837 | set_pageblock_migratetype(page, | ||
838 | start_migratetype); | ||
839 | |||
827 | migratetype = start_migratetype; | 840 | migratetype = start_migratetype; |
828 | move_freepages_block(zone, page, migratetype); | ||
829 | } | 841 | } |
830 | 842 | ||
831 | /* Remove the page from the freelists */ | 843 | /* Remove the page from the freelists */ |