From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:53917) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Xqbc2-00076d-8f for qemu-devel@nongnu.org; Tue, 18 Nov 2014 00:46:07 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Xqbbx-0006iL-4m for qemu-devel@nongnu.org; Tue, 18 Nov 2014 00:46:02 -0500 Received: from [59.151.112.132] (port=47052 helo=heian.cn.fujitsu.com) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Xqbbw-0006h6-MR for qemu-devel@nongnu.org; Tue, 18 Nov 2014 00:45:57 -0500 Message-ID: <546AD6D5.8070408@cn.fujitsu.com> Date: Tue, 18 Nov 2014 13:19:17 +0800 From: Gu Zheng MIME-Version: 1.0 References: <1415841032-4565-1-git-send-email-guz.fnst@cn.fujitsu.com> In-Reply-To: <1415841032-4565-1-git-send-email-guz.fnst@cn.fujitsu.com> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH 0/6] cpu: add device_add foo-x86_64-cpu support List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: imammedo@redhat.com, qemu-devel@nongnu.org Cc: zhugh.fnst@cn.fujitsu.com, tangchen@cn.fujitsu.com, chen.fan.fnst@cn.fujitsu.com, isimatu.yasuaki@jp.fujitsu.com, Gu Zheng , anshul.makkar@profitbricks.com, afaerber@suse.de ping... On 11/13/2014 09:10 AM, Gu Zheng wrote: > This series is based on the previous patchset from Chen Fan: > https://lists.nongnu.org/archive/html/qemu-devel/2014-05/msg02360.html >=20 > We try to make cpu hotplug with device_add, and make > "-device foo-x86_64-cpu" available=EF=BC=8Calso we can set apic-id > property with command line, if without setting apic-id property, > we offer the first unoccupied apic id as the default new apic id. > When hotplug cpu with device_add, additional check of APIC ID will be > done after cpu object initialization which was different from > 'cpu_add' command that check 'ids' at the beginning. >=20 > --- > Changelog since RFC: > -split out APIC vmstate/QMP-monitor changes into separate patches. > -add the handle of the startup cpus(-device foo). > -remove duplicated checking about env->cpuid_apic_id. > -do actual APIC ID allocation at realize time if it is not set before. > -remove the unneeded x86_cpu_cpudef_instance_init(). > -split off device_del support out here. > --- >=20 > Chen Fan (2): > cpu: introduce CpuTopoInfo structure for argument simplification > cpu: add device_add foo-x86_64-cpu support >=20 > Gu Zheng (4): > qom/cpu: move register_vmstate to common CPUClass.realizefn > qom/cpu: move apic vmstate register into x86_cpu_apic_realize > monitor: use cc->get_arch_id as the cpu index > acpi:cpu hotplug: set pcmachine as icc bus' hotplug handler >=20 > cpus.c | 4 ++- > exec.c | 32 +++++++++++++--------- > hw/acpi/cpu_hotplug.c | 5 +++- > hw/cpu/icc_bus.c | 15 ----------- > hw/i386/pc.c | 6 ---- > hw/i386/pc_piix.c | 6 +++- > hw/i386/pc_q35.c | 5 +++ > hw/intc/apic_common.c | 3 +- > include/hw/cpu/icc_bus.h | 14 ++++++++++ > include/hw/i386/apic_internal.h | 3 ++ > include/qom/cpu.h | 2 + > monitor.c | 4 ++- > qom/cpu.c | 2 + > target-i386/cpu.c | 54 +++++++++++++++++++++++++++++++++= ++--- > target-i386/topology.h | 51 +++++++++++++++++++++++++--------= --- > 15 files changed, 145 insertions(+), 61 deletions(-) >=20