diff options
author | Christoph Lameter <clameter@engr.sgi.com> | 2006-02-11 20:55:53 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-02-12 00:41:11 -0500 |
commit | 80e4342601abfafacb5f20571e40b56d73d10819 (patch) | |
tree | c03c988860aa8b6a1cc24ba8adff3778c1705928 | |
parent | 643a654540579b0dcc7a206a4a7475276a41aff0 (diff) |
[PATCH] zone reclaim: do not check references to a page during zone reclaim
shrink_list() and refill_inactive() check all ptes pointing to a page for
reference bits in order to decide if the page should be put on the active
list. This is not necessary for zone_reclaim since we are only interested
in removing unmapped pages. Skip the checks in both functions.
Signed-off-by: Christoph Lameter <clameter@sgi.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
-rw-r--r-- | mm/vmscan.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/mm/vmscan.c b/mm/vmscan.c index 5db32fdfaf39..e1c64230ffdd 100644 --- a/mm/vmscan.c +++ b/mm/vmscan.c | |||
@@ -443,6 +443,10 @@ static int shrink_list(struct list_head *page_list, struct scan_control *sc) | |||
443 | BUG_ON(PageActive(page)); | 443 | BUG_ON(PageActive(page)); |
444 | 444 | ||
445 | sc->nr_scanned++; | 445 | sc->nr_scanned++; |
446 | |||
447 | if (!sc->may_swap && page_mapped(page)) | ||
448 | goto keep_locked; | ||
449 | |||
446 | /* Double the slab pressure for mapped and swapcache pages */ | 450 | /* Double the slab pressure for mapped and swapcache pages */ |
447 | if (page_mapped(page) || PageSwapCache(page)) | 451 | if (page_mapped(page) || PageSwapCache(page)) |
448 | sc->nr_scanned++; | 452 | sc->nr_scanned++; |
@@ -1231,7 +1235,7 @@ refill_inactive_zone(struct zone *zone, struct scan_control *sc) | |||
1231 | * Now use this metric to decide whether to start moving mapped memory | 1235 | * Now use this metric to decide whether to start moving mapped memory |
1232 | * onto the inactive list. | 1236 | * onto the inactive list. |
1233 | */ | 1237 | */ |
1234 | if (swap_tendency >= 100) | 1238 | if (swap_tendency >= 100 && sc->may_swap) |
1235 | reclaim_mapped = 1; | 1239 | reclaim_mapped = 1; |
1236 | 1240 | ||
1237 | while (!list_empty(&l_hold)) { | 1241 | while (!list_empty(&l_hold)) { |