Re: [PATCH 1/1] arch/x86/mm/numa: Do not initialize nodes twice

[Oscar Salvador]

On Mon, Feb 21, 2022 at 10:20:02AM +0100, Michal Hocko wrote:
> On Fri 18-02-22 23:43:02, Oscar Salvador wrote:
> > Why setting the node online saves us then? Well, simply because
> > __try_online_node() backs off when the node is online, meaning
> > we do not end up calling register_one_node() in the first place.
> 
> This is really a mess and a house built on sand. Thanks for looking into
> it and hopefully this can get cleaned up to a saner state.

Yes, I am willing to have a deep look into that and see how we can
improve the situation.

> This sha1 is from linux-next very likely so it won't be persistent.
> Please drop it.

Yes, it is. I guess it is fine to not have a "Fixes" tag here, so I will
remove it then.

> I would stick a TODO here.
> 			/*
> 			 * Exclude this node from 
> 			 * bringup_nonboot_cpus
> 			 *  cpu_up
> 			 *    __try_online_node
> 			 *      register_one_node
> 			 * because node_subsys is not initialized yet
> 			 * TODO remove dependency on node_online()
> 			 */

Sure, will do.

Thanks!

-- 
Oscar Salvador
SUSE Labs