From: KAMEZAWA Hiroyuki
Date: Wed Jul 16 2008 - 23:22:45 EST

Under memcg, active anon tend not to go to inactive anon.
This will cause OOM in memcg easily when tons of anon was used at once.
This check was lacked in split-lru.

This patch is a fix agaisnt

Changelog: v1 -> v2:
- avoid adding "else".

Signed-off-by:KAMEZAWA Hiroyuki <kamezawa.hiroyu@xxxxxxxxxxxxxx>
Acked-by: Rik van Riel <riel@xxxxxxxxxx>

mm/vmscan.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

Index: mmtom-stamp-2008-07-15-15-39/mm/vmscan.c
--- mmtom-stamp-2008-07-15-15-39.orig/mm/vmscan.c
+++ mmtom-stamp-2008-07-15-15-39/mm/vmscan.c
@@ -1351,7 +1351,7 @@ static unsigned long shrink_zone(int pri
* Even if we did not try to evict anon pages at all, we want to
* rebalance the anon lru active/inactive ratio.
- if (scan_global_lru(sc) && inactive_anon_is_low(zone))
+ if (!scan_global_lru(sc) || inactive_anon_is_low(zone))
shrink_active_list(SWAP_CLUSTER_MAX, zone, sc, priority, 0);


To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/