[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC PATCH v0 5/6] qmp, spapr: Show hot-plugged/pluggab
From: |
Bharata B Rao |
Subject: |
Re: [Qemu-devel] [RFC PATCH v0 5/6] qmp, spapr: Show hot-plugged/pluggable CPU slots in the Machine |
Date: |
Fri, 26 Feb 2016 15:10:32 +0530 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
David, I am responding to just one comment here and will respond to others
in detail later.
On Fri, Feb 26, 2016 at 03:03:17PM +1100, David Gibson wrote:
>
> > + if (!strstart(prop->name, SPAPR_MACHINE_CPU_CORE_PROP, NULL)) {
> > + continue;
> > + }
> > +
> > + elem = g_new0(CPUSlotInfoList, 1);
> > + s = g_new0(CPUSlotInfo, 1);
> > +
> > + obj = object_property_get_link(root_container, prop->name, NULL);
> > + if (obj) {
> > + /* Slot populated */
> > + dev = DEVICE(obj);
> > + core = SPAPR_CPU_CORE(obj);
> > +
> > + if (dev->id) {
> > + s->has_id = true;
> > + s->id = g_strdup(dev->id);
> > + }
> > + s->realized = object_property_get_bool(obj, "realized", NULL);
> > + s->nr_cpus = core->nr_threads;
> > + s->has_nr_cpus = true;
> > + s->qom_path = object_get_canonical_path(obj);
> > + s->has_qom_path = true;
> > + if (s->realized) {
> > + spapr_cpuinfo_list(obj, &cpu_prev);
> > + }
> > + s->has_cpus = true;
> > + } else {
> > + /* Slot empty */
> > + s->has_id = false;
> > + s->has_nr_cpus = false;
> > + s->has_qom_path = false;
> > + s->has_cpus = false;
> > + s->realized = false;
>
> Um.. doesn't this leave out the cpu model and nr_threads for a
> non-populated slot? Which is crucial information for mgmt to populate
> the slot.
Actually if we don't allow heterogenous configurations, management currently
knows how to get model and threads that have already been specified with
-cpu and -smp global options respectively.
If we want to support heterogenous configurations, should we then have
supported model types and thread values listed here ? At the moment I
am not sure what those supported types and thread values could be.
Regards,
Bharata.
- Re: [Qemu-devel] [RFC PATCH v0 2/6] spapr: CPU core device, (continued)
- [Qemu-devel] [RFC PATCH v0 5/6] qmp, spapr: Show hot-plugged/pluggable CPU slots in the Machine, Bharata B Rao, 2016/02/25
- Re: [Qemu-devel] [RFC PATCH v0 5/6] qmp, spapr: Show hot-plugged/pluggable CPU slots in the Machine, David Gibson, 2016/02/25
- Re: [Qemu-devel] [RFC PATCH v0 5/6] qmp, spapr: Show hot-plugged/pluggable CPU slots in the Machine,
Bharata B Rao <=
- Re: [Qemu-devel] [RFC PATCH v0 5/6] qmp, spapr: Show hot-plugged/pluggable CPU slots in the Machine, Eric Blake, 2016/02/26
- Re: [Qemu-devel] [RFC PATCH v0 5/6] qmp, spapr: Show hot-plugged/pluggable CPU slots in the Machine, Thomas Huth, 2016/02/26
- Re: [Qemu-devel] [RFC PATCH v0 5/6] qmp, spapr: Show hot-plugged/pluggable CPU slots in the Machine, Igor Mammedov, 2016/02/29
- [Qemu-devel] [RFC PATCH v0 4/6] spapr: CPU hotplug support, Bharata B Rao, 2016/02/25
- [Qemu-devel] [RFC PATCH v0 6/6] hmp: Implement 'info cpu-slots', Bharata B Rao, 2016/02/25