diff options
author | KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com> | 2010-10-26 17:21:43 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2010-10-26 19:52:07 -0400 |
commit | 47185052165a4c5de0a461018238375dd982c2ec (patch) | |
tree | 4425005eb24efdd66d0d9d4f95a11fdad04843b1 /mm | |
parent | 7d3579e8e61937cbba268ea9b218d006b6d64221 (diff) |
vmscan: remove dead code in shrink_inactive_list()
After synchrounous lumpy reclaim, the page_list is guaranteed to not have
active pages as page activation in shrink_page_list() disables lumpy
reclaim. Remove the dead code.
Signed-off-by: KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com>
Signed-off-by: Mel Gorman <mel@csn.ul.ie>
Reviewed-by: Minchan Kim <minchan.kim@gmail.com>
Cc: Johannes Weiner <hannes@cmpxchg.org>
Cc: Wu Fengguang <fengguang.wu@intel.com>
Cc: Rik van Riel <riel@redhat.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')
-rw-r--r-- | mm/vmscan.c | 8 |
1 files changed, 0 insertions, 8 deletions
diff --git a/mm/vmscan.c b/mm/vmscan.c index d9fc2dce93af..cec8081bbd95 100644 --- a/mm/vmscan.c +++ b/mm/vmscan.c | |||
@@ -1331,7 +1331,6 @@ shrink_inactive_list(unsigned long nr_to_scan, struct zone *zone, | |||
1331 | unsigned long nr_scanned; | 1331 | unsigned long nr_scanned; |
1332 | unsigned long nr_reclaimed = 0; | 1332 | unsigned long nr_reclaimed = 0; |
1333 | unsigned long nr_taken; | 1333 | unsigned long nr_taken; |
1334 | unsigned long nr_active; | ||
1335 | unsigned long nr_anon; | 1334 | unsigned long nr_anon; |
1336 | unsigned long nr_file; | 1335 | unsigned long nr_file; |
1337 | 1336 | ||
@@ -1386,13 +1385,6 @@ shrink_inactive_list(unsigned long nr_to_scan, struct zone *zone, | |||
1386 | 1385 | ||
1387 | /* Check if we should syncronously wait for writeback */ | 1386 | /* Check if we should syncronously wait for writeback */ |
1388 | if (should_reclaim_stall(nr_taken, nr_reclaimed, priority, sc)) { | 1387 | if (should_reclaim_stall(nr_taken, nr_reclaimed, priority, sc)) { |
1389 | /* | ||
1390 | * The attempt at page out may have made some | ||
1391 | * of the pages active, mark them inactive again. | ||
1392 | */ | ||
1393 | nr_active = clear_active_flags(&page_list, NULL); | ||
1394 | count_vm_events(PGDEACTIVATE, nr_active); | ||
1395 | |||
1396 | set_lumpy_reclaim_mode(priority, sc, true); | 1388 | set_lumpy_reclaim_mode(priority, sc, true); |
1397 | nr_reclaimed += shrink_page_list(&page_list, sc); | 1389 | nr_reclaimed += shrink_page_list(&page_list, sc); |
1398 | } | 1390 | } |