From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:42143) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1anSPA-0005x9-DH for qemu-devel@nongnu.org; Tue, 05 Apr 2016 10:56:33 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1anSP5-0004I9-Gv for qemu-devel@nongnu.org; Tue, 05 Apr 2016 10:56:32 -0400 Received: from e28smtp01.in.ibm.com ([125.16.236.1]:45544) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1anSP4-0004GR-Rv for qemu-devel@nongnu.org; Tue, 05 Apr 2016 10:56:27 -0400 Received: from localhost by e28smtp01.in.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Tue, 5 Apr 2016 20:26:23 +0530 Date: Tue, 5 Apr 2016 20:25:57 +0530 From: Bharata B Rao Message-ID: <20160405145557.GA32764@in.ibm.com> References: <1459413561-30745-1-git-send-email-bharata@linux.vnet.ibm.com> <20160404164427.3eed276b@igors-macbook-pro.local> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20160404164427.3eed276b@igors-macbook-pro.local> Subject: Re: [Qemu-devel] [RFC PATCH v2.1 00/12] Core based CPU hotplug for PowerPC sPAPR Reply-To: bharata@linux.vnet.ibm.com List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Igor Mammedov Cc: mjrosato@linux.vnet.ibm.com, agraf@suse.de, thuth@redhat.com, pkrempa@redhat.com, ehabkost@redhat.com, aik@ozlabs.ru, armbru@redhat.com, qemu-devel@nongnu.org, borntraeger@de.ibm.com, qemu-ppc@nongnu.org, pbonzini@redhat.com, mdroth@linux.vnet.ibm.com, afaerber@suse.de, david@gibson.dropbear.id.au On Mon, Apr 04, 2016 at 04:44:27PM +0200, Igor Mammedov wrote: > On Thu, 31 Mar 2016 14:09:09 +0530 > Bharata B Rao wrote: > > > Hi, > > > > This is v2.1 of "Core based CPU hotplug for PowerPC sPAPR". v2 was > > posted here: > > https://lists.gnu.org/archive/html/qemu-ppc/2016-03/msg00201.html > > > > I am making this v2.1 instead of v3 as this introduces a change in > > device_add semantics and if we don't want to continue this, I will go > > back to the previous v2 semantics from v3 onwards. v2 had > > spapr-cpu-core device that served as CPU core device for all types of > > PowerPCCPU threads. In this version however, spapr-cpu-core becomes > > an abstract device and we will have different core devices for each > > of the PowerPCCPU type. So in this version, the hotplug semantics > > looks like this: > > > > (qemu) device_add POWER8-spapr-cpu-core,id=core2,core=16[,threads=4] > > > > cpu_model specification is gone as it becomes redundant with different > > core types for each CPU type. CPU core types are defined only for > > host, POWER7 and POWER8 models only (and not for their > > variants/aliases) yet. > > > > I have mostly taken care of all the review comments that I got for v2. > Could you rebase on top of current master, pls? Sure, I usually update, but missed it this time. > > and fix compile error: > hw/cpu/core.c: In function ‘core_prop_set_core’: > hw/cpu/core.c:31:9: error: implicit declaration of function ‘error_propagate’ [-Werror=implicit-function-declaration] > error_propagate(errp, local_err); I don't see this when compiling all the targets either on x86 or PPC systems. May be this is seen with latest master, in any case will check this out when spinning next version. However do you have any specific comment to be addressed from this version before I post the next one ? Regards, Bharata.