From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:49064) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VHCT4-0000E7-NO for qemu-devel@nongnu.org; Wed, 04 Sep 2013 08:46:00 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1VHCSy-0001up-Of for qemu-devel@nongnu.org; Wed, 04 Sep 2013 08:45:54 -0400 Received: from cantor2.suse.de ([195.135.220.15]:54984 helo=mx2.suse.de) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VHCSy-0001ua-9h for qemu-devel@nongnu.org; Wed, 04 Sep 2013 08:45:48 -0400 Message-ID: <52272B78.3010804@suse.de> Date: Wed, 04 Sep 2013 14:45:44 +0200 From: =?ISO-8859-15?Q?Andreas_F=E4rber?= MIME-Version: 1.0 References: <1375366359-11553-1-git-send-email-jjherne@us.ibm.com> In-Reply-To: <1375366359-11553-1-git-send-email-jjherne@us.ibm.com> Content-Type: text/plain; charset=ISO-8859-15 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH 0/8] [PATCH RFC v3] s390 cpu hotplug List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Jason J. Herne" Cc: ehabkost@redhat.com, qemu-devel@nongnu.org, agraf@suse.de, borntraeger@de.ibm.com, jfrei@linux.vnet.ibm.com, imammedo@redhat.com, Luiz Capitulino Hello, Am 01.08.2013 16:12, schrieb Jason J. Herne: > From: "Jason J. Herne" >=20 > Latest code for cpu Hotplug on S390 architecture. This one is vastly = simpler > than v2 as we have decided to avoid the command line specification=20 > of -device s390-cpu. >=20 > The last version can be found here: > http://lists.gnu.org/archive/html/qemu-devel/2013-06/msg01183.html >=20 > There is also a patch in this series to add cpu-add to the Qemu monitor > interface. >=20 > Hotplugged cpus are created in the configured state and can be used by = the > guest after the guest onlines the cpu by:=20 > "echo 1 > /sys/bus/cpu/devices/cpuN/online" >=20 > Hot unplugging is currently not implemented by this code.=20 We have been having several off-list discussions since then that I'll try to briefly summarize here, please correct or extend as needed: 1) CPU topology for QOM Physically a System z machine may have an MCM with, e.g., 6 CPUs with 6 cores each. But unlike x86, there is PR/SM, LPAR and possibly z/VM in between Linux and hardware, so we do actually want to be able to hot-plug in quantities of 1 and not by 6 on s390x for the foreseeable future. We seem willing to set a QOM ABI in stone based on that assumptio= n. =3D> s390-cpu (or future subtypes) to be used with device_add. =3D> Flat /machine/cpu[n] list in composition tree a possibility. 1a) CPU topology for guests STSI instruction topology support not implemented yet. =3D> Guest unaware of any emulated topology today. hyptop tool requires hypfs implementation for KVM. =3D> Guest unaware of sibling VMs today, unlike z/VM and LPAR. 2) CPU hot-unplug Hotplug will always use a unique linear CPU address, even if hot-unplug leads to a sparse address space. =3D> cpu_num !=3D cpu_index With all that in mind, I'll now need to review the s390 patches again. For the HMP patch I am waiting on feedback from Igor once he returns from his vacation and, if there are no objections, would like to see that patch go through Luiz' queue since unrelated to s390x. Regards, Andreas >=20 > Jason J. Herne (8): > s390-qemu: cpu hotplug - Define New SCLP Codes > s390-qemu: cpu hotplug - SCLP CPU Info > s390-qemu: cpu hotplug - SCLP Event integration > s390-qemu: cpu hotplug - Storage key global access > s390-qemu: cpu hotplug - ipi_states enhancements > s390-qemu: cpu hotplug - s390 cpu init improvements for hotplug > s390-qemu: cpu hotplug - Implement hot_add_cpu hook > qemu-monitor: HMP cpu-add wrapper >=20 > hmp-commands.hx | 13 ++++ > hmp.c | 10 ++++ > hmp.h | 1 + > hw/s390x/Makefile.objs | 2 +- > hw/s390x/event-facility.c | 7 +++ > hw/s390x/s390-virtio-ccw.c | 8 ++- > hw/s390x/s390-virtio.c | 47 +++++++++------ > hw/s390x/s390-virtio.h | 2 +- > hw/s390x/sclp.c | 53 +++++++++++++++- > hw/s390x/sclpcpu.c | 120 +++++++++++++++++++++++++++++= ++++++++ > include/hw/s390x/event-facility.h | 3 + > include/hw/s390x/sclp.h | 41 +++++++++++++ > target-s390x/cpu.c | 36 ++++++++++- > target-s390x/cpu.h | 7 +++ > target-s390x/helper.c | 12 ++++ > 15 files changed, 336 insertions(+), 26 deletions(-) > create mode 100644 hw/s390x/sclpcpu.c >=20 --=20 SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 N=FCrnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imend=F6rffer; HRB 16746 AG N=FCrnbe= rg