On Wed, May 22, 2024 at 04:05:12PM +0200, David Hildenbrand wrote: > On 21.05.24 14:57, Brendan Jackman wrote: > > + return zone->zone_start_pfn + READ_ONCE(zone->spanned_pages); > > It's weird to apply that logic only to spanned_pages, whereby zone_start_pfn > can (and will) similarly change when onlining/offlining memory. > Oh, yep. For some reason I had decided that zone_start_pfn was fixed but that is (actually very obviously) not true! Will take a closer look and extend v2 to cover that too, unless someone finds a reason this whole patch is nonsense. Thanks for the review.
