RE: [PATCH] mm/memory hotplug/unplug: Optimize zone contiguous check when changing pfn range

["Liu, Yuan1" <yuan1.liu@intel.com>]

> -----Original Message-----
> From: David Hildenbrand (Arm) <david@kernel.org>
> Sent: Thursday, March 26, 2026 5:23 PM
> To: Liu, Yuan1 <yuan1.liu@intel.com>; Oscar Salvador <osalvador@suse.de>;
> Mike Rapoport <rppt@kernel.org>; Wei Yang <richard.weiyang@gmail.com>
> Cc: linux-mm@kvack.org; Hu, Yong <yong.hu@intel.com>; Zou, Nanhai
> <nanhai.zou@intel.com>; Tim Chen <tim.c.chen@linux.intel.com>; Zhuo, Qiuxu
> <qiuxu.zhuo@intel.com>; Chen, Yu C <yu.c.chen@intel.com>; Deng, Pan
> <pan.deng@intel.com>; Li, Tianyou <tianyou.li@intel.com>; Chen Zhang
> <zhangchen.kidd@jd.com>; linux-kernel@vger.kernel.org
> Subject: Re: [PATCH] mm/memory hotplug/unplug: Optimize zone contiguous
> check when changing pfn range
> 
> 
> > About AI feedback below
> >
> > init_unavailable_range() uses for_each_valid_pfn() and skips unmapped
> > sections that lack a mem_map. If the full hole size is added anyway,
> > pages_with_online_memmap could artificially match spanned_pages for
> > zones with missing sections.
> >
> > Could this make zone_is_contiguous() incorrectly return true, bypassing
> > pfn_valid() checks and potentially dereferencing missing sections?
> > unsinged long zone_hole_pages
> >
> > I think if I can update the pages_with_online_memmap in the
> > init_unavailable_range directly.
> 
> See how I handled it: I simply return the number of initialized pages
> from the function. I think that should do.

Yes, I will follow that and thanks for your comments.

> --
> Cheers,
> 
> David