Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752579AbdFUJyf (ORCPT ); Wed, 21 Jun 2017 05:54:35 -0400 Received: from ozlabs.org ([103.22.144.67]:38235 "EHLO ozlabs.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751154AbdFUJye (ORCPT ); Wed, 21 Jun 2017 05:54:34 -0400 From: Michael Ellerman To: Michael Bringmann , linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org Cc: Nathan Fontenot , Michael Bringmann , ltc-virtual-io@lists.linux.ibm.com Subject: Re: [PATCH V6 0/2] powerpc/dlpar: Correct display of hot-add/hot-remove CPUs and memory In-Reply-To: References: User-Agent: Notmuch/0.21 (https://notmuchmail.org) Date: Wed, 21 Jun 2017 19:54:32 +1000 Message-ID: <8760fpu0mf.fsf@concordia.ellerman.id.au> MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2008 Lines: 43 Michael Bringmann writes: > On Power systems with shared configurations of CPUs and memory, there > are some issues with association of additional CPUs and memory to nodes > when hot-adding resources. These patches address some of those problems. > > 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. > > powerpc/numa: Correct the currently broken capability to set the > topology for shared CPUs in LPARs. At boot time for shared CPU > lpars, the topology for each shared CPU is set to node zero, however, > this is now updated correctly using the Virtual Processor Home Node > (VPHN) capabilities information provided by the pHyp. The VPHN handling > in Linux is disabled, if PRRN handling is present. > > Signed-off-by: Michael Bringmann > > Michael Bringmann (2): > powerpc/hotplug: Add option to define max nodes allowing dynamic > growth of resources. > powerpc/numa: Update CPU topology when VPHN enabled > --- > Changes in V6: > -- Reorder some code to better eliminate unused functions in > conditional builds. What changed between yesterday's V6 and this V6? If you're going to resend, please bump the version number, we have tools that parse the subject and version, and resending multiple times with the same number breaks those. cheers