diff options
author | Wei Yang <richard.weiyang@gmail.com> | 2017-05-13 11:02:30 +1000 |
---|---|---|
committer | Stephen Rothwell <sfr@canb.auug.org.au> | 2017-05-14 10:20:55 +1000 |
commit | dc154149c62d29d88cc408cde8c7b59c5c13dab2 (patch) | |
tree | e823bbb27d6525be0ad263c3fdfb59ca593d014b | |
parent | fb63adc505c3b011a298a91ea87e7099c91b14fa (diff) | |
download | linux-dc154149c62d29d88cc408cde8c7b59c5c13dab2.tar.gz linux-dc154149c62d29d88cc408cde8c7b59c5c13dab2.tar.xz |
mm/page_alloc: return 0 in case this node has no page within the zone
The whole memory space is divided into several zones and nodes may have no
page in some zones. In this case, the __absent_pages_in_range() would
return 0, since the range it is searching for is an empty range.
Also this happens more often to those nodes with higher memory range when
there are more nodes, which is a trend for future architectures.
This patch checks the zone range after clamp and adjustment, return 0 if
the range is an empty range.
Link: http://lkml.kernel.org/r/20170206154314.15705-1-richard.weiyang@gmail.com
Signed-off-by: Wei Yang <richard.weiyang@gmail.com>
Cc: Vlastimil Babka <vbabka@suse.cz>
Cc: Mel Gorman <mgorman@techsingularity.net>
Cc: Michal Hocko <mhocko@suse.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
-rw-r--r-- | mm/page_alloc.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/mm/page_alloc.c b/mm/page_alloc.c index f9e450c6b6e4..7f7b50e03d4f 100644 --- a/mm/page_alloc.c +++ b/mm/page_alloc.c @@ -5781,6 +5781,11 @@ static unsigned long __meminit zone_absent_pages_in_node(int nid, adjust_zone_range_for_zone_movable(nid, zone_type, node_start_pfn, node_end_pfn, &zone_start_pfn, &zone_end_pfn); + + /* If this node has no page within this zone, return 0. */ + if (zone_start_pfn == zone_end_pfn) + return 0; + nr_absent = __absent_pages_in_range(nid, zone_start_pfn, zone_end_pfn); /* |