From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx0a-001b2d01.pphosted.com (mx0b-001b2d01.pphosted.com [148.163.158.5]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 3ws4sy65b4zDqgg for ; Tue, 20 Jun 2017 08:10:30 +1000 (AEST) Received: from pps.filterd (m0098421.ppops.net [127.0.0.1]) by mx0a-001b2d01.pphosted.com (8.16.0.20/8.16.0.20) with SMTP id v5JM9IaR040951 for ; Mon, 19 Jun 2017 18:10:28 -0400 Received: from e36.co.us.ibm.com (e36.co.us.ibm.com [32.97.110.154]) by mx0a-001b2d01.pphosted.com with ESMTP id 2b6mkmdr0a-1 (version=TLSv1.2 cipher=AES256-SHA bits=256 verify=NOT) for ; Mon, 19 Jun 2017 18:10:27 -0400 Received: from localhost by e36.co.us.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Mon, 19 Jun 2017 16:10:27 -0600 To: linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org Cc: Benjamin Herrenschmidt , Paul Mackerras , Michael Ellerman , Michael Bringmann , David Gibson , Reza Arbab , John Allen , Thomas Gleixner , Bharata B Rao , Shailendra Singh , "Aneesh Kumar K.V" , Sebastian Andrzej Siewior , Rashmica Gupta , Ingo Molnar In-Reply-To: <081bb19d-3008-842d-6872-f51fccb94139@linux.vnet.ibm.com> From: Michael Bringmann Subject: [PATCH V6 1/2] powerpc/hotplug: Ensure enough nodes avail for operations Date: Mon, 19 Jun 2017 17:10:20 -0500 MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Message-Id: List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , powerpc/hotplug: On systems like PowerPC which allow 'hot-add' of CPU or memory resources, it may occur that the new resources are to be inserted into nodes that were not used for these resources at bootup. In the kernel, any node that is used must be defined and initialized at boot. In order to meet both needs, this patch adds a new kernel command line option (numnodes=) for use by the PowerPC architecture- specific code that defines the maximum number of nodes that the kernel will ever need in its current hardware environment. The boot code that initializes nodes for PowerPC will read this value and use it to ensure that all of the desired nodes are setup in the 'node_possible_map', and elsewhere. Signed-off-by: Michael Bringmann --- --- arch/powerpc/mm/numa.c | 31 +++++++++++++++++++++++++++++++ 1 file changed, 31 insertions(+) diff --git a/arch/powerpc/mm/numa.c b/arch/powerpc/mm/numa.c index 2b808a1..e6ee829 100644 --- a/arch/powerpc/mm/numa.c +++ b/arch/powerpc/mm/numa.c @@ -60,10 +60,27 @@ static int n_mem_addr_cells, n_mem_size_cells; static int form1_affinity; +#define TOPOLOGY_DEF_NUM_NODES 0 #define MAX_DISTANCE_REF_POINTS 4 static int distance_ref_points_depth; static const __be32 *distance_ref_points; static int distance_lookup_table[MAX_NUMNODES][MAX_DISTANCE_REF_POINTS]; +static int topology_num_nodes = TOPOLOGY_DEF_NUM_NODES; + +/* + * Topology-related early parameters + */ +static int __init early_num_nodes(char *p) +{ + if (!p) + return 1; + + topology_num_nodes = memparse(p, &p); + dbg("topology num nodes = 0x%d\n", topology_num_nodes); + + return 0; +} +early_param("numnodes", early_num_nodes); /* * Allocate node_to_cpumask_map based on number of available nodes @@ -892,6 +909,18 @@ static void __init setup_node_data(int nid, u64 start_pfn, u64 end_pfn) NODE_DATA(nid)->node_spanned_pages = spanned_pages; } +static void __init setup_min_nodes(void) +{ + int i, l = topology_num_nodes; + + for (i = 0; i < l; i++) { + if (!node_possible(i)) { + setup_node_data(i, 0, 0); + node_set(i, node_possible_map); + } + } +} + void __init initmem_init(void) { int nid, cpu; @@ -911,6 +940,8 @@ void __init initmem_init(void) */ nodes_and(node_possible_map, node_possible_map, node_online_map); + setup_min_nodes(); + for_each_online_node(nid) { unsigned long start_pfn, end_pfn;