From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:50844) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ag2SO-0005hn-77 for qemu-devel@nongnu.org; Tue, 15 Mar 2016 23:49:13 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ag2SL-0003ru-1X for qemu-devel@nongnu.org; Tue, 15 Mar 2016 23:49:12 -0400 Received: from e23smtp03.au.ibm.com ([202.81.31.145]:39700) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ag2SK-0003rc-DH for qemu-devel@nongnu.org; Tue, 15 Mar 2016 23:49:08 -0400 Received: from localhost by e23smtp03.au.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Wed, 16 Mar 2016 13:49:05 +1000 Date: Wed, 16 Mar 2016 09:18:03 +0530 From: Bharata B Rao Message-ID: <20160316034803.GC13176@in.ibm.com> References: <1457672078-17307-1-git-send-email-bharata@linux.vnet.ibm.com> <20160314104728.5ff09f86@nial.brq.redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20160314104728.5ff09f86@nial.brq.redhat.com> Subject: Re: [Qemu-devel] [RFC PATCH v2 0/9] 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, qemu-devel@nongnu.org, armbru@redhat.com, borntraeger@de.ibm.com, qemu-ppc@nongnu.org, afaerber@suse.de, pbonzini@redhat.com, mdroth@linux.vnet.ibm.com, david@gibson.dropbear.id.au On Mon, Mar 14, 2016 at 10:47:28AM +0100, Igor Mammedov wrote: > On Fri, 11 Mar 2016 10:24:29 +0530 > Bharata B Rao wrote: > > > Hi, > > > > This is the next version of "Core based CPU hotplug for PowerPC sPAPR" that > > was posted at > > https://lists.gnu.org/archive/html/qemu-ppc/2016-03/msg00081.html > > > > device_add semantics > > -------------------- > > For -smp 16,sockets=1,cores=2,threads=8,maxcpus=32 > > (qemu) device_add spapr-cpu-core,id=core2,core=16,cpu_model=host[,threads=8] > do you plan to allow user to hotplug different cpu_models? > If not it would be better to hide cpu_model from user > and set it from machine pre_plug handler. In my earlier implementations I derived cpu model from -cpu and threads from -smp,threads= commandline options and never exposed them to device_add command. Though we don't support heterogenous systems (different cpu models and/or threads) now, it was felt that it should be easy enough to support such systems if required in future, that's how cpu_model and threads became options for device_add. One of the things that David felt was missing from my earlier QMP query command (and which is true in your QMP query implementation also) is that we aren't exporting cpu_model at all, at least for not-yet-plugged cores. So should we include that or let management figure that out since it would already know about the CPU model. Regards, Bharata.