diff options
author | Hugh Dickins <hughd@google.com> | 2012-03-23 05:57:31 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2012-03-23 11:34:40 -0400 |
commit | 643ac9fc5429e85b8b7f534544b80bcc4f34c367 (patch) | |
tree | f6db7d7e8a08761119d478ef7adf5bef4402f786 /mm/vmscan.c | |
parent | 437538267b672f9320833907f1b5acbb2605f4be (diff) |
mm: fix testorder interaction between two kswapd patches
Adjusting cc715d99e529 "mm: vmscan: forcibly scan highmem if there are
too many buffer_heads pinning highmem" for -stable reveals that it was
slightly wrong once on top of fe2c2a106663 "vmscan: reclaim at order 0
when compaction is enabled", which specifically adds testorder for the
zone_watermark_ok_safe() test.
Signed-off-by: Hugh Dickins <hughd@google.com>
Acked-by: Mel Gorman <mel@csn.ul.ie>
Acked-by: Rik van Riel <riel@redhat.com>
Cc: 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 | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/mm/vmscan.c b/mm/vmscan.c index 49f15ef0a99a..7658fd6536dd 100644 --- a/mm/vmscan.c +++ b/mm/vmscan.c | |||
@@ -2817,7 +2817,7 @@ loop_again: | |||
2817 | testorder = 0; | 2817 | testorder = 0; |
2818 | 2818 | ||
2819 | if ((buffer_heads_over_limit && is_highmem_idx(i)) || | 2819 | if ((buffer_heads_over_limit && is_highmem_idx(i)) || |
2820 | !zone_watermark_ok_safe(zone, order, | 2820 | !zone_watermark_ok_safe(zone, testorder, |
2821 | high_wmark_pages(zone) + balance_gap, | 2821 | high_wmark_pages(zone) + balance_gap, |
2822 | end_zone, 0)) { | 2822 | end_zone, 0)) { |
2823 | shrink_zone(priority, zone, &sc); | 2823 | shrink_zone(priority, zone, &sc); |