Re: [PATCH v2 1/3] mm,page_alloc: Use {get,put}_online_mems() to get stable zone's values

From: Oscar Salvador
Date: Wed Jun 02 2021 - 15:46:02 EST


On 2021-06-02 20:37, David Hildenbrand wrote:
Long story short, using get_online_mems() looks wrong.

Maybe the current lightweight reader/writer protection does serve a purpose?

It was too nice and easy to be true I guess.
Yeah, I missed the point that blocking might be an issue here, and hotplug operations can take really long, so not an option.
I must have switched my brain off back there, because now it is just too obvious.

Then I guwmess that seqlock must stay and the only thing than can go is the pgdat resize lock from the hotplug code.

Meh.

--
Oscar Salvador
SUSE L3