Commit 32aaf055 authored by Pengfei Li's avatar Pengfei Li Committed by Linus Torvalds

mm/compaction.c: remove unnecessary zone parameter in isolate_migratepages()

Like commit 40cacbcb ("mm, compaction: remove unnecessary zone
parameter in some instances"), remove unnecessary zone parameter.

No functional change.

Link: http://lkml.kernel.org/r/20190806151616.21107-1-lpf.vector@gmail.comSigned-off-by: default avatarPengfei Li <lpf.vector@gmail.com>
Reviewed-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Acked-by: default avatarVlastimil Babka <vbabka@suse.cz>
Cc: Mel Gorman <mgorman@techsingularity.net>
Cc: Qian Cai <cai@lca.pw>
Cc: Andrey Ryabinin <aryabinin@virtuozzo.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent a94b5252
...@@ -1737,8 +1737,7 @@ static unsigned long fast_find_migrateblock(struct compact_control *cc) ...@@ -1737,8 +1737,7 @@ static unsigned long fast_find_migrateblock(struct compact_control *cc)
* starting at the block pointed to by the migrate scanner pfn within * starting at the block pointed to by the migrate scanner pfn within
* compact_control. * compact_control.
*/ */
static isolate_migrate_t isolate_migratepages(struct zone *zone, static isolate_migrate_t isolate_migratepages(struct compact_control *cc)
struct compact_control *cc)
{ {
unsigned long block_start_pfn; unsigned long block_start_pfn;
unsigned long block_end_pfn; unsigned long block_end_pfn;
...@@ -1756,8 +1755,8 @@ static isolate_migrate_t isolate_migratepages(struct zone *zone, ...@@ -1756,8 +1755,8 @@ static isolate_migrate_t isolate_migratepages(struct zone *zone,
*/ */
low_pfn = fast_find_migrateblock(cc); low_pfn = fast_find_migrateblock(cc);
block_start_pfn = pageblock_start_pfn(low_pfn); block_start_pfn = pageblock_start_pfn(low_pfn);
if (block_start_pfn < zone->zone_start_pfn) if (block_start_pfn < cc->zone->zone_start_pfn)
block_start_pfn = zone->zone_start_pfn; block_start_pfn = cc->zone->zone_start_pfn;
/* /*
* fast_find_migrateblock marks a pageblock skipped so to avoid * fast_find_migrateblock marks a pageblock skipped so to avoid
...@@ -1787,8 +1786,8 @@ static isolate_migrate_t isolate_migratepages(struct zone *zone, ...@@ -1787,8 +1786,8 @@ static isolate_migrate_t isolate_migratepages(struct zone *zone,
if (!(low_pfn % (SWAP_CLUSTER_MAX * pageblock_nr_pages))) if (!(low_pfn % (SWAP_CLUSTER_MAX * pageblock_nr_pages)))
cond_resched(); cond_resched();
page = pageblock_pfn_to_page(block_start_pfn, block_end_pfn, page = pageblock_pfn_to_page(block_start_pfn,
zone); block_end_pfn, cc->zone);
if (!page) if (!page)
continue; continue;
...@@ -2169,7 +2168,7 @@ compact_zone(struct compact_control *cc, struct capture_control *capc) ...@@ -2169,7 +2168,7 @@ compact_zone(struct compact_control *cc, struct capture_control *capc)
cc->rescan = true; cc->rescan = true;
} }
switch (isolate_migratepages(cc->zone, cc)) { switch (isolate_migratepages(cc)) {
case ISOLATE_ABORT: case ISOLATE_ABORT:
ret = COMPACT_CONTENDED; ret = COMPACT_CONTENDED;
putback_movable_pages(&cc->migratepages); putback_movable_pages(&cc->migratepages);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment