The patch titled Subject: mm/mm_init.c: get the highest zone directly has been added to the -mm mm-unstable branch. Its filename is mm-mm_initc-get-the-highest-zone-directly.patch This patch will shortly appear at https://git.kernel.org/pub/scm/linux/kernel/git/akpm/25-new.git/tree/patches/mm-mm_initc-get-the-highest-zone-directly.patch This patch will later appear in the mm-unstable branch at git://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm Before you just go and hit "reply", please: a) Consider who else should be cc'ed b) Prefer to cc a suitable mailing list as well c) Ideally: find the original patch on the mailing list and do a reply-to-all to that, adding suitable additional cc's *** Remember to use Documentation/process/submit-checklist.rst when testing your code *** The -mm tree is included into linux-next via the mm-everything branch at git://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm and is updated there every 2-3 working days ------------------------------------------------------ From: Wei Yang <richard.weiyang@xxxxxxxxx> Subject: mm/mm_init.c: get the highest zone directly Date: Fri, 31 May 2024 00:26:11 +0000 We have recorded nr_zones in pgdat, just get it directly. Link: https://lkml.kernel.org/r/20240531002613.5231-1-richard.weiyang@xxxxxxxxx Signed-off-by: Wei Yang <richard.weiyang@xxxxxxxxx> Cc: Mike Rapoport (IBM) <rppt@xxxxxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- mm/mm_init.c | 12 ++++-------- 1 file changed, 4 insertions(+), 8 deletions(-) --- a/mm/mm_init.c~mm-mm_initc-get-the-highest-zone-directly +++ a/mm/mm_init.c @@ -2138,7 +2138,7 @@ static int __init deferred_init_memmap(v unsigned long first_init_pfn, flags; unsigned long start = jiffies; struct zone *zone; - int zid, max_threads; + int max_threads; u64 i; /* Bind memory initialisation thread to a local node if possible */ @@ -2165,12 +2165,8 @@ static int __init deferred_init_memmap(v */ pgdat_resize_unlock(pgdat, &flags); - /* Only the highest zone is deferred so find it */ - for (zid = 0; zid < MAX_NR_ZONES; zid++) { - zone = pgdat->node_zones + zid; - if (first_init_pfn < zone_end_pfn(zone)) - break; - } + /* Only the highest zone is deferred */ + zone = pgdat->node_zones + pgdat->nr_zones - 1; /* If the zone is empty somebody else may have cleared out the zone */ if (!deferred_init_mem_pfn_range_in_zone(&i, zone, &spfn, &epfn, @@ -2198,7 +2194,7 @@ static int __init deferred_init_memmap(v } zone_empty: /* Sanity check that the next zone really is unpopulated */ - WARN_ON(++zid < MAX_NR_ZONES && populated_zone(++zone)); + WARN_ON(pgdat->nr_zones < MAX_NR_ZONES && populated_zone(++zone)); pr_info("node %d deferred pages initialised in %ums\n", pgdat->node_id, jiffies_to_msecs(jiffies - start)); _ Patches currently in -mm which might be from richard.weiyang@xxxxxxxxx are mm-mm_initc-get-the-highest-zone-directly.patch mm-mm_initc-use-deferred_init_mem_pfn_range_in_zone-to-decide-loop-condition.patch mm-mm_initc-not-always-search-next-deferred_init_pfn-from-very-beginning.patch