From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965720Ab3CZQji (ORCPT ); Tue, 26 Mar 2013 12:39:38 -0400 Received: from smtp.citrix.com ([66.165.176.89]:10771 "EHLO SMTP.CITRIX.COM" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965531Ab3CZQjh (ORCPT ); Tue, 26 Mar 2013 12:39:37 -0400 X-IronPort-AV: E=Sophos;i="4.84,913,1355097600"; d="scan'208";a="15668495" Date: Tue, 26 Mar 2013 16:39:30 +0000 From: Stefano Stabellini X-X-Sender: sstabellini@kaball.uk.xensource.com To: Arnd Bergmann CC: Stefano Stabellini , "xen-devel@lists.xensource.com" , "linux-kernel@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "konrad.wilk@oracle.com" , Ian Campbell , "rob.herring@calxeda.com" , "will.deacon@arm.com" , "marc.zyngier@arm.com" Subject: Re: [PATCH v2 5/6] xenvm: add a simple PSCI node and a second cpu In-Reply-To: <201303261500.53490.arnd@arndb.de> Message-ID: References: <1364308875-26484-5-git-send-email-stefano.stabellini@eu.citrix.com> <201303261500.53490.arnd@arndb.de> User-Agent: Alpine 2.02 (DEB 1266 2009-07-14) MIME-Version: 1.0 Content-Type: text/plain; charset="US-ASCII" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, 26 Mar 2013, Arnd Bergmann wrote: > On Tuesday 26 March 2013, Stefano Stabellini wrote: > > Signed-off-by: Stefano Stabellini > > CC: rob.herring@calxeda.com > > CC: will.deacon@arm.com > > CC: marc.zyngier@arm.com > > CC: arnd@arndb.de > > I wonder how this is supposed to work on real systems. Shouldn't the dt > blob be filled out with the correct number of CPUs at the time you start > the guest? > This change looks like you just make all guests use a hardcoded > set of two CPUs instead of just one, but you probably want to allow any > number between 1 and the number of physically present cores. That's right, the DT passed on to the guests is generated by the hypervisor. This is just an example, adding a cpu and a psci node is just meant to make it clear that Xen supports multi-vcpu guests using PSCI to bootstrap secondary cpus.