From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:43073) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Wzlvh-00040X-6g for qemu-devel@nongnu.org; Wed, 25 Jun 2014 08:04:05 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WzlvY-0001T4-7h for qemu-devel@nongnu.org; Wed, 25 Jun 2014 08:03:56 -0400 Message-ID: <53AABAA2.1090907@suse.de> Date: Wed, 25 Jun 2014 14:03:46 +0200 From: Alexander Graf MIME-Version: 1.0 References: <1403628191-5574-1-git-send-email-aik@ozlabs.ru> In-Reply-To: <1403628191-5574-1-git-send-email-aik@ozlabs.ru> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v2 0/6] spapr: rework memory nodes List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Alexey Kardashevskiy , qemu-devel@nongnu.org Cc: qemu-ppc@nongnu.org On 24.06.14 18:43, Alexey Kardashevskiy wrote: > c4177479 "spapr: make sure RMA is in first mode of first memory node" > introduced regression which prevents from running guests with memoryless > NUMA node#0 which may happen on real POWER8 boxes and which would make > sense to debug in QEMU. > > This patchset aim is to fix that and also fix various code problems in > memory nodes generation. > > These 2 patches could be merged (the resulting patch looks rather ugly): > spapr: Use DT memory node rendering helper for other nodes > spapr: Move DT memory node rendering to a helper > > Changes: > v2: > * minor cosmetic change in spapr_node0_size() > * spapr_populate_memory() fixed to work in a no-numa config > * patch changing max numa nodes is removed > > Please comment. Thanks! This does not sound like 2.1 material to me. Alex