[patch 1/2]vmscan: correct all_unreclaimable for zone without lru pages

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



I saw DMA zone always has ->all_unreclaimable set. The reason is the high zones
are big, so zone_watermark_ok/_safe() will always return false with a high
classzone_idx for DMA zone, because DMA zone's lowmem_reserve is big for a high
classzone_idx. When kswapd runs into DMA zone, it doesn't scan/reclaim any
pages(no pages in lru), but mark the zone as all_unreclaimable. This can
happen in other low zones too.
This is confusing and can potentially cause oom. Say a low zone has
all_unreclaimable when high zone hasn't enough memory. Then allocating
some pages in low zone(for example reading blkdev with highmem support),
then run into direct reclaim. Since the zone has all_unreclaimable set,
direct reclaim might reclaim nothing and an oom reported. If
all_unreclaimable is unset, the zone can actually reclaim some pages.
If all_unreclaimable is unset, in the inner loop of balance_pgdat we always have
all_zones_ok 0 when checking a low zone's watermark. If high zone watermark isn't
good, there is no problem. Otherwise, we might loop one more time in the outer
loop, but since high zone watermark is ok, the end_zone will be lower, then low
zone's watermark check will be ok and the outer loop will break. So looks this
doesn't bring any problem.

Signed-off-by: Shaohua Li <shaohua.li@xxxxxxxxx>

---
 mm/vmscan.c |    4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

Index: linux/mm/vmscan.c
===================================================================
--- linux.orig/mm/vmscan.c	2011-09-27 13:46:31.000000000 +0800
+++ linux/mm/vmscan.c	2011-09-27 15:09:29.000000000 +0800
@@ -2565,7 +2565,9 @@ loop_again:
 				sc.nr_reclaimed += reclaim_state->reclaimed_slab;
 				total_scanned += sc.nr_scanned;
 
-				if (nr_slab == 0 && !zone_reclaimable(zone))
+				if (nr_slab == 0 && !zone_reclaimable(zone) &&
+				    !zone_watermark_ok_safe(zone, order,
+				    high_wmark_pages(zone) + balance_gap, 0, 0))
 					zone->all_unreclaimable = 1;
 			}
 


--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@xxxxxxxxx.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/
Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>


[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]