Commit 9037a993 authored by Michal Hocko's avatar Michal Hocko Committed by Linus Torvalds

mm, memory_hotplug: split up register_one_node()

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.

Link: http://lkml.kernel.org/r/20170515085827.16474-6-mhocko@kernel.orgSigned-off-by: default avatarMichal Hocko <mhocko@suse.com>
Acked-by: default avatarVlastimil Babka <vbabka@suse.cz>
Cc: Andi Kleen <ak@linux.intel.com>
Cc: Andrea Arcangeli <aarcange@redhat.com>
Cc: Balbir Singh <bsingharora@gmail.com>
Cc: Dan Williams <dan.j.williams@intel.com>
Cc: Daniel Kiper <daniel.kiper@oracle.com>
Cc: David Rientjes <rientjes@google.com>
Cc: Heiko Carstens <heiko.carstens@de.ibm.com>
Cc: Igor Mammedov <imammedo@redhat.com>
Cc: Jerome Glisse <jglisse@redhat.com>
Cc: Joonsoo Kim <js1304@gmail.com>
Cc: Martin Schwidefsky <schwidefsky@de.ibm.com>
Cc: Mel Gorman <mgorman@suse.de>
Cc: Reza Arbab <arbab@linux.vnet.ibm.com>
Cc: Tobias Regnery <tobias.regnery@gmail.com>
Cc: Toshi Kani <toshi.kani@hpe.com>
Cc: Vitaly Kuznetsov <vkuznets@redhat.com>
Cc: Xishi Qiu <qiuxishi@huawei.com>
Cc: Yasuaki Ishimatsu <isimatu.yasuaki@jp.fujitsu.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 1b862aec
...@@ -461,10 +461,9 @@ int unregister_mem_sect_under_nodes(struct memory_block *mem_blk, ...@@ -461,10 +461,9 @@ int unregister_mem_sect_under_nodes(struct memory_block *mem_blk,
return 0; return 0;
} }
static int link_mem_sections(int nid) int link_mem_sections(int nid, unsigned long start_pfn, unsigned long nr_pages)
{ {
unsigned long start_pfn = NODE_DATA(nid)->node_start_pfn; unsigned long end_pfn = start_pfn + nr_pages;
unsigned long end_pfn = start_pfn + NODE_DATA(nid)->node_spanned_pages;
unsigned long pfn; unsigned long pfn;
struct memory_block *mem_blk = NULL; struct memory_block *mem_blk = NULL;
int err = 0; int err = 0;
...@@ -552,9 +551,6 @@ static int node_memory_callback(struct notifier_block *self, ...@@ -552,9 +551,6 @@ static int node_memory_callback(struct notifier_block *self,
return NOTIFY_OK; return NOTIFY_OK;
} }
#endif /* CONFIG_HUGETLBFS */ #endif /* CONFIG_HUGETLBFS */
#else /* !CONFIG_MEMORY_HOTPLUG_SPARSE */
static int link_mem_sections(int nid) { return 0; }
#endif /* CONFIG_MEMORY_HOTPLUG_SPARSE */ #endif /* CONFIG_MEMORY_HOTPLUG_SPARSE */
#if !defined(CONFIG_MEMORY_HOTPLUG_SPARSE) || \ #if !defined(CONFIG_MEMORY_HOTPLUG_SPARSE) || \
...@@ -569,14 +565,12 @@ static void init_node_hugetlb_work(int nid) { } ...@@ -569,14 +565,12 @@ static void init_node_hugetlb_work(int nid) { }
#endif #endif
int register_one_node(int nid) int __register_one_node(int nid)
{ {
int error = 0;
int cpu;
if (node_online(nid)) {
int p_node = parent_node(nid); int p_node = parent_node(nid);
struct node *parent = NULL; struct node *parent = NULL;
int error;
int cpu;
if (p_node != nid) if (p_node != nid)
parent = node_devices[p_node]; parent = node_devices[p_node];
...@@ -593,15 +587,10 @@ int register_one_node(int nid) ...@@ -593,15 +587,10 @@ int register_one_node(int nid)
register_cpu_under_node(cpu, nid); register_cpu_under_node(cpu, nid);
} }
/* link memory sections under this node */
error = link_mem_sections(nid);
/* initialize work queue for memory hot plug */ /* initialize work queue for memory hot plug */
init_node_hugetlb_work(nid); init_node_hugetlb_work(nid);
}
return error; return error;
} }
void unregister_one_node(int nid) void unregister_one_node(int nid)
......
...@@ -30,9 +30,38 @@ struct memory_block; ...@@ -30,9 +30,38 @@ struct memory_block;
extern struct node *node_devices[]; extern struct node *node_devices[];
typedef void (*node_registration_func_t)(struct node *); typedef void (*node_registration_func_t)(struct node *);
#if defined(CONFIG_MEMORY_HOTPLUG_SPARSE) && defined(CONFIG_NUMA)
extern int link_mem_sections(int nid, unsigned long start_pfn, unsigned long nr_pages);
#else
static inline int link_mem_sections(int nid, unsigned long start_pfn, unsigned long nr_pages)
{
return 0;
}
#endif
extern void unregister_node(struct node *node); extern void unregister_node(struct node *node);
#ifdef CONFIG_NUMA #ifdef CONFIG_NUMA
extern int register_one_node(int nid); /* Core of the node registration - only memory hotplug should use this */
extern int __register_one_node(int nid);
/* Registers an online node */
static inline int register_one_node(int nid)
{
int error = 0;
if (node_online(nid)) {
struct pglist_data *pgdat = NODE_DATA(nid);
error = __register_one_node(nid);
if (error)
return error;
/* link memory sections under this node */
error = link_mem_sections(nid, pgdat->node_start_pfn, pgdat->node_spanned_pages);
}
return error;
}
extern void unregister_one_node(int nid); extern void unregister_one_node(int nid);
extern int register_cpu_under_node(unsigned int cpu, unsigned int nid); extern int register_cpu_under_node(unsigned int cpu, unsigned int nid);
extern int unregister_cpu_under_node(unsigned int cpu, unsigned int nid); extern int unregister_cpu_under_node(unsigned int cpu, unsigned int nid);
...@@ -46,6 +75,10 @@ extern void register_hugetlbfs_with_node(node_registration_func_t doregister, ...@@ -46,6 +75,10 @@ extern void register_hugetlbfs_with_node(node_registration_func_t doregister,
node_registration_func_t unregister); node_registration_func_t unregister);
#endif #endif
#else #else
static inline int __register_one_node(int nid)
{
return 0;
}
static inline int register_one_node(int nid) static inline int register_one_node(int nid)
{ {
return 0; return 0;
......
...@@ -1388,7 +1388,22 @@ int __ref add_memory_resource(int nid, struct resource *res, bool online) ...@@ -1388,7 +1388,22 @@ int __ref add_memory_resource(int nid, struct resource *res, bool online)
node_set_online(nid); node_set_online(nid);
if (new_node) { 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.
*/
ret = link_mem_sections(nid, start_pfn, nr_pages);
register_fail:
/* /*
* If sysfs file of new node can't create, cpu on the node * If sysfs file of new node can't create, cpu on the node
* can't be hot-added. There is no rollback way now. * can't be hot-added. There is no rollback way now.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment