From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:47574) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WyBe3-0008CI-4g for qemu-devel@nongnu.org; Fri, 20 Jun 2014 23:07:17 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WyBds-0004Aw-Dt for qemu-devel@nongnu.org; Fri, 20 Jun 2014 23:07:11 -0400 Received: from mail-pd0-f169.google.com ([209.85.192.169]:63438) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WyBds-0004AT-8F for qemu-devel@nongnu.org; Fri, 20 Jun 2014 23:07:00 -0400 Received: by mail-pd0-f169.google.com with SMTP id g10so3583758pdj.14 for ; Fri, 20 Jun 2014 20:06:58 -0700 (PDT) Message-ID: <53A4F6CD.3040600@ozlabs.ru> Date: Sat, 21 Jun 2014 13:06:53 +1000 From: Alexey Kardashevskiy MIME-Version: 1.0 References: <1402905233-26510-1-git-send-email-aik@ozlabs.ru> <1402905233-26510-4-git-send-email-aik@ozlabs.ru> <20140620225528.GA2008@linux.vnet.ibm.com> In-Reply-To: <20140620225528.GA2008@linux.vnet.ibm.com> Content-Type: text/plain; charset=KOI8-R Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 3/7] spapr: Refactor spapr_populate_memory() List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Nishanth Aravamudan Cc: qemu-ppc@nongnu.org, qemu-devel@nongnu.org, Alexander Graf On 06/21/2014 08:55 AM, Nishanth Aravamudan wrote: > On 16.06.2014 [17:53:49 +1000], Alexey Kardashevskiy wrote: >> Current QEMU does not support memoryless NUMA nodes. >> This prepares SPAPR for that. >> >> This moves 2 calls of spapr_populate_memory_node() into >> the existing loop which handles nodes other than than >> the first one. >> >> Signed-off-by: Alexey Kardashevskiy >> --- >> hw/ppc/spapr.c | 31 +++++++++++-------------------- >> 1 file changed, 11 insertions(+), 20 deletions(-) >> >> diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c >> index cb3a10a..666b676 100644 >> --- a/hw/ppc/spapr.c >> +++ b/hw/ppc/spapr.c >> @@ -689,28 +689,13 @@ static void spapr_populate_memory_node(void *fdt, int nodeid, hwaddr start, >> >> static int spapr_populate_memory(sPAPREnvironment *spapr, void *fdt) >> { >> - hwaddr node0_size, mem_start, node_size; >> + hwaddr mem_start, node_size; >> int i; >> >> - /* memory node(s) */ >> - if (nb_numa_nodes > 1 && node_mem[0] < ram_size) { >> - node0_size = node_mem[0]; >> - } else { >> - node0_size = ram_size; >> - } >> - >> - /* RMA */ >> - spapr_populate_memory_node(fdt, 0, 0, spapr->rma_size); >> - >> - /* RAM: Node 0 */ >> - if (node0_size > spapr->rma_size) { >> - spapr_populate_memory_node(fdt, 0, spapr->rma_size, >> - node0_size - spapr->rma_size); >> - } >> - >> - /* RAM: Node 1 and beyond */ >> - mem_start = node0_size; >> - for (i = 1; i < nb_numa_nodes; i++) { >> + for (i = 0, mem_start = 0; i < nb_numa_nodes; ++i) { >> + if (!node_mem[i]) { >> + continue; >> + } > > Doesn't this skip memoryless nodes? What actually puts the memoryless > node in the device-tree? It does skip. > And if you were to put them in, wouldn't spapr_populate_memory_node() > fail because we'd be creating two nodes with memory@XXX where XXX is the > same (starting address) for both? I cannot do this now - there is no way to tell from the command line where I want NUMA node memory start from so I'll end up with multiple nodes with the same name and QEMU won't start. When NUMA fixes reach upstream, I'll try to work out something on top of that. -- Alexey