From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wr0-f198.google.com (mail-wr0-f198.google.com [209.85.128.198]) by kanga.kvack.org (Postfix) with ESMTP id 72EB96B03C1 for ; Thu, 13 Apr 2017 10:13:20 -0400 (EDT) Received: by mail-wr0-f198.google.com with SMTP id 18so6439199wrz.4 for ; Thu, 13 Apr 2017 07:13:20 -0700 (PDT) Received: from mx1.suse.de (mx2.suse.de. [195.135.220.15]) by mx.google.com with ESMTPS id e140si13279580wmd.56.2017.04.13.07.13.19 for (version=TLS1 cipher=AES128-SHA bits=128/128); Thu, 13 Apr 2017 07:13:19 -0700 (PDT) Date: Thu, 13 Apr 2017 16:13:16 +0200 From: Michal Hocko Subject: Re: [PATCH 5/9] mm, memory_hotplug: split up register_one_node Message-ID: <20170413141316.GC11795@dhcp22.suse.cz> References: <20170410110351.12215-1-mhocko@kernel.org> <20170410110351.12215-6-mhocko@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: owner-linux-mm@kvack.org List-ID: To: Vlastimil Babka Cc: linux-mm@kvack.org, Andrew Morton , Mel Gorman , Andrea Arcangeli , Jerome Glisse , Reza Arbab , Yasuaki Ishimatsu , qiuxishi@huawei.com, Kani Toshimitsu , slaoub@gmail.com, Joonsoo Kim , Andi Kleen , David Rientjes , Daniel Kiper , Igor Mammedov , Vitaly Kuznetsov , LKML On Thu 13-04-17 16:05:17, Vlastimil Babka wrote: > On 04/10/2017 01:03 PM, Michal Hocko wrote: > > From: Michal Hocko > > > > Memory hotplug (add_memory_resource) has to reinitialize node > > infrastructure if the node is offline (one which went through the > > complete add_memory(); remove_memory() cycle). That involves node > > registration to the kobj infrastructure (register_node), the proper > > association with cpus (register_cpu_under_node) and finally creation of > > node<->memblock symlinks (link_mem_sections). > > > > The last part requires to know node_start_pfn and node_spanned_pages > > which we currently have but a leter patch will postpone this > > initialization to the onlining phase which happens later. In fact we do > > not need to rely on the early pgdat initialization even now because the > > currently hot added pfn range is currently known. > > > > Split register_one_node into core which does all the common work for > > the boot time NUMA initialization and the hotplug (__register_one_node). > > register_one_node keeps the full initialization while hotplug calls > > __register_one_node and manually calls link_mem_sections for the proper > > range. > > > > This shouldn't introduce any functional change. > > > > Signed-off-by: Michal Hocko > > Acked-by: Vlastimil Babka Thanks! > nit: > > @@ -1387,7 +1387,22 @@ int __ref add_memory_resource(int nid, struct resource *res, bool online) > > node_set_online(nid); > > > > if (new_node) { > > - ret = register_one_node(nid); > > + unsigned long start_pfn = start >> PAGE_SHIFT; > > + unsigned long nr_pages = size >> PAGE_SHIFT; > > + > > + ret = __register_one_node(nid); > > + if (ret) > > + goto register_fail; > > + > > + /* > > + * link memory sections under this node. This is already > > + * done when creatig memory section in register_new_memory > > + * but that depends to have the node registered so offline > > + * nodes have to go through register_node. > > + * TODO clean up this mess. > > Is this a work-in-progress or final TODO? :) I do not plan to address it in this series, but I will revisit it later. There are more like this in other patches. -- Michal Hocko SUSE Labs -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org