From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pa0-x241.google.com (mail-pa0-x241.google.com [IPv6:2607:f8b0:400e:c03::241]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 3sd46B3nLlzDsd9 for ; Mon, 19 Sep 2016 21:53:38 +1000 (AEST) Received: by mail-pa0-x241.google.com with SMTP id vz6so6920418pab.1 for ; Mon, 19 Sep 2016 04:53:38 -0700 (PDT) Subject: Re: [PATCH v2 2/3] powerpc/mm: allow memory hotplug into a memoryless node To: Reza Arbab , Michael Ellerman , Benjamin Herrenschmidt , Paul Mackerras , Rob Herring , Frank Rowand , Jonathan Corbet , Andrew Morton References: <1473883618-14998-1-git-send-email-arbab@linux.vnet.ibm.com> <1473883618-14998-3-git-send-email-arbab@linux.vnet.ibm.com> Cc: Bharata B Rao , Nathan Fontenot , Stewart Smith , Alistair Popple , linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, devicetree@vger.kernel.org, linux-mm@kvack.org From: Balbir Singh Message-ID: <15d62e9c-38e6-dfd3-0ee2-6885cbfbe315@gmail.com> Date: Mon, 19 Sep 2016 21:53:49 +1000 MIME-Version: 1.0 In-Reply-To: <1473883618-14998-3-git-send-email-arbab@linux.vnet.ibm.com> Content-Type: text/plain; charset=windows-1252 List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On 15/09/16 06:06, Reza Arbab wrote: > Remove the check which prevents us from hotplugging into an empty node. > > This limitation has been questioned before [1], and judging by the > response, there doesn't seem to be a reason we can't remove it. No issues > have been found in light testing. > > [1] http://lkml.kernel.org/r/CAGZKiBrmkSa1yyhbf5hwGxubcjsE5SmkSMY4tpANERMe2UG4bg@mail.gmail.com > http://lkml.kernel.org/r/20160511215051.GF22115@arbab-laptop.austin.ibm.com > > Signed-off-by: Reza Arbab > Acked-by: Balbir Singh > Cc: Nathan Fontenot > Cc: Bharata B Rao > --- > arch/powerpc/mm/numa.c | 13 +------------ > 1 file changed, 1 insertion(+), 12 deletions(-) > I presume you've tested with CONFIG_NODES_SHIFT of 8 (255 nodes?) Balbir Singh.