diff options
author | Johannes Weiner <hannes@cmpxchg.org> | 2009-03-31 18:19:38 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2009-04-01 11:59:12 -0400 |
commit | bd2f6199cf9af472aeefa1b642c9f504f19e6008 (patch) | |
tree | ba8cd5e7d940bcac161e28872998f86a2910362b /mm/vmscan.c | |
parent | bd775c42ea5f7c766d03a287083837cf05e7e738 (diff) |
vmscan: respect higher order in zone_reclaim()
During page allocation, there are two stages of direct reclaim that are
applied to each zone in the preferred list. The first stage using
zone_reclaim() reclaims unmapped file backed pages and slab pages if over
defined limits as these are cheaper to reclaim. The caller specifies the
order of the target allocation but the scan control is not being correctly
initialised.
The impact is that the correct number of pages are being reclaimed but
that lumpy reclaim is not being applied. This increases the chances of a
full direct reclaim via try_to_free_pages() is required.
This patch initialises the order field of the scan control as requested by
the caller.
[mel@csn.ul.ie: rewrote changelog]
Signed-off-by: Johannes Weiner <hannes@cmpxchg.org>
Acked-by: Mel Gorman <mel@csn.ul.ie>
Cc: Rik van Riel <riel@redhat.com>
Cc: Andy Whitcroft <apw@shadowen.org>
Cc: KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com>
Cc: KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'mm/vmscan.c')
-rw-r--r-- | mm/vmscan.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/mm/vmscan.c b/mm/vmscan.c index 9578063cd943..51f2df04d7cf 100644 --- a/mm/vmscan.c +++ b/mm/vmscan.c | |||
@@ -2295,6 +2295,7 @@ static int __zone_reclaim(struct zone *zone, gfp_t gfp_mask, unsigned int order) | |||
2295 | SWAP_CLUSTER_MAX), | 2295 | SWAP_CLUSTER_MAX), |
2296 | .gfp_mask = gfp_mask, | 2296 | .gfp_mask = gfp_mask, |
2297 | .swappiness = vm_swappiness, | 2297 | .swappiness = vm_swappiness, |
2298 | .order = order, | ||
2298 | .isolate_pages = isolate_pages_global, | 2299 | .isolate_pages = isolate_pages_global, |
2299 | }; | 2300 | }; |
2300 | unsigned long slab_reclaimable; | 2301 | unsigned long slab_reclaimable; |