From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:39759) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bK4by-0004zY-09 for qemu-devel@nongnu.org; Mon, 04 Jul 2016 10:12:35 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bK4bt-0001vT-Kf for qemu-devel@nongnu.org; Mon, 04 Jul 2016 10:12:32 -0400 Received: from mx1.redhat.com ([209.132.183.28]:53219) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bK4bt-0001vN-Ck for qemu-devel@nongnu.org; Mon, 04 Jul 2016 10:12:29 -0400 Date: Mon, 4 Jul 2016 17:12:23 +0300 From: "Michael S. Tsirkin" Message-ID: <20160704171204-mutt-send-email-mst@redhat.com> References: <1466784366-281935-1-git-send-email-imammedo@redhat.com> <1466784366-281935-7-git-send-email-imammedo@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1466784366-281935-7-git-send-email-imammedo@redhat.com> Subject: Re: [Qemu-devel] [PATCH v2 06/18] target-i386: add socket/core/thread properties to X86CPU List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Igor Mammedov Cc: qemu-devel@nongnu.org, pbonzini@redhat.com, rth@twiddle.net, ehabkost@redhat.com, armbru@redhat.com, eduardo.otubo@profitbricks.com, eblake@redhat.com, pkrempa@redhat.com, marcel@redhat.com On Fri, Jun 24, 2016 at 06:05:54PM +0200, Igor Mammedov wrote: > these properties will be used by as address where to plug > CPU with help -device/device_add commands. > > Signed-off-by: Igor Mammedov > --- > v2: > - rename socket/core/thread properties to socket-id/core-id/thread-id > - add mismatch checks for apic_id and socket-id/core-id/thread-id > in case both are set > --- > hw/i386/pc.c | 29 +++++++++++++++++++++++++++++ > target-i386/cpu.c | 6 ++++++ > target-i386/cpu.h | 4 ++++ > 3 files changed, 39 insertions(+) > > diff --git a/hw/i386/pc.c b/hw/i386/pc.c > index db792e6..eaa4b59 100644 > --- a/hw/i386/pc.c > +++ b/hw/i386/pc.c > @@ -1763,6 +1763,7 @@ static void pc_cpu_pre_plug(HotplugHandler *hotplug_dev, > DeviceState *dev, Error **errp) > { > int idx; > + X86CPUTopoInfo topo; > X86CPU *cpu = X86_CPU(dev); > PCMachineState *pcms = PC_MACHINE(hotplug_dev); > CPUArchId *cpu_slot = pc_find_cpu_slot(pcms, CPU(dev), &idx); > @@ -1780,6 +1781,34 @@ static void pc_cpu_pre_plug(HotplugHandler *hotplug_dev, > cpu->apic_id); > return; > } > + > + /* if 'address' properties socket-id/core-id/thread-id are not set, set them > + * so that query_hotpluggable_cpus would show correct values > + */ > + /* TODO: move socket_id/core_id/thread_id checks into x86_cpu_realizefn() > + * once -smp refactoring is complete and there will be CPU private > + * CPUState::nr_cores and CPUState::nr_threads fields instead of globals */ > + x86_topo_ids_from_apicid(cpu->apic_id, smp_cores, smp_threads, &topo); > + if (cpu->socket_id != -1 && cpu->socket_id != topo.pkg_id) { > + error_setg(errp, "CPU socket-id: %d mismatch with apic-id: %u", > + cpu->socket_id, cpu->apic_id); why not list the pkg_id too? Same below. > + return; > + } > + cpu->socket_id = topo.pkg_id; > + > + if (cpu->core_id != -1 && cpu->core_id != topo.core_id) { > + error_setg(errp, "CPU core-id: %d mismatch with apic-id: %u", > + cpu->core_id, cpu->apic_id); > + return; > + } > + cpu->core_id = topo.core_id; > + > + if (cpu->thread_id != -1 && cpu->thread_id != topo.smt_id) { > + error_setg(errp, "CPU thread-id: %d mismatch with apic-id: %u", > + cpu->thread_id, cpu->apic_id); > + return; > + } > + cpu->thread_id = topo.smt_id; > } > > static void pc_machine_device_pre_plug_cb(HotplugHandler *hotplug_dev, > diff --git a/target-i386/cpu.c b/target-i386/cpu.c > index 9294b3d..1ec40a0 100644 > --- a/target-i386/cpu.c > +++ b/target-i386/cpu.c > @@ -3164,8 +3164,14 @@ static Property x86_cpu_properties[] = { > #ifdef CONFIG_USER_ONLY > /* apic_id = 0 by default for *-user, see commit 9886e834 */ > DEFINE_PROP_UINT32("apic-id", X86CPU, apic_id, 0), > + DEFINE_PROP_INT32("thread-id", X86CPU, thread_id, 0), > + DEFINE_PROP_INT32("core-id", X86CPU, core_id, 0), > + DEFINE_PROP_INT32("socket-id", X86CPU, socket_id, 0), > #else > DEFINE_PROP_UINT32("apic-id", X86CPU, apic_id, UNASSIGNED_APIC_ID), > + DEFINE_PROP_INT32("thread-id", X86CPU, thread_id, -1), > + DEFINE_PROP_INT32("core-id", X86CPU, core_id, -1), > + DEFINE_PROP_INT32("socket-id", X86CPU, socket_id, -1), > #endif > DEFINE_PROP_BOOL("pmu", X86CPU, enable_pmu, false), > { .name = "hv-spinlocks", .info = &qdev_prop_spinlocks }, > diff --git a/target-i386/cpu.h b/target-i386/cpu.h > index 8b09cfa..e110633 100644 > --- a/target-i386/cpu.h > +++ b/target-i386/cpu.h > @@ -1190,6 +1190,10 @@ struct X86CPU { > Notifier machine_done; > > struct kvm_msrs *kvm_msr_buf; > + > + int32_t socket_id; > + int32_t core_id; > + int32_t thread_id; > }; > > static inline X86CPU *x86_env_get_cpu(CPUX86State *env) > -- > 1.8.3.1