[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [RFC PATCH v2 9/9] spapr: CPU hot unplug support
From: |
Bharata B Rao |
Subject: |
Re: [Qemu-ppc] [RFC PATCH v2 9/9] spapr: CPU hot unplug support |
Date: |
Wed, 16 Mar 2016 11:07:44 +0530 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On Wed, Mar 16, 2016 at 04:27:04PM +1100, David Gibson wrote:
> On Fri, Mar 11, 2016 at 10:24:38AM +0530, Bharata B Rao wrote:
> > Remove the CPU core device by removing the underlying CPU thread devices.
> > Hot removal of CPU for sPAPR guests is achieved by sending the hot unplug
> > notification to the guest. Release the vCPU object after CPU hot unplug so
> > that vCPU fd can be parked and reused.
> >
> > Signed-off-by: Bharata B Rao <address@hidden>
> > ---
> > hw/ppc/spapr.c | 21 ++++++++++
> > hw/ppc/spapr_cpu_core.c | 86
> > +++++++++++++++++++++++++++++++++++++++++
> > include/hw/ppc/spapr.h | 1 +
> > include/hw/ppc/spapr_cpu_core.h | 12 ++++++
> > 4 files changed, 120 insertions(+)
> >
> > diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c
> > index 822c87d..b1e9ba2 100644
> > --- a/hw/ppc/spapr.c
> > +++ b/hw/ppc/spapr.c
> > @@ -2345,7 +2345,12 @@ static void spapr_machine_device_plug(HotplugHandler
> > *hotplug_dev,
> >
> > spapr_memory_plug(hotplug_dev, dev, node, errp);
> > } else if (object_dynamic_cast(OBJECT(dev), TYPE_SPAPR_CPU_CORE)) {
> > + /*
> > + * TODO: Move this check to pre_plug handler at which point
> > + * spapr_core_release() won't be necessary.
> > + */
> > if (!smc->dr_cpu_enabled && dev->hotplugged) {
> > + spapr_core_release(dev);
> > error_setg(errp, "CPU hotplug not supported for this machine");
> > return;
> > }
>
> This hunk doesn't look like its related to unplug. Did it belong in
> another patch?
Yes, it actually belongs to hot-plug but the whole infrastructure to release
the core and associated threads get introduced in this patch, hence put
this hunk here.
However, if pre_plug is the way to go forward, we woudn't need this
altogether as roll-back is much easier from there than from here.
Regards,
Bharata.
- Re: [Qemu-ppc] [Qemu-devel] [RFC PATCH v2 6/9] spapr: CPU core device, (continued)
- [Qemu-ppc] [RFC PATCH v2 5/9] cpu: Abstract CPU core type, Bharata B Rao, 2016/03/10
- [Qemu-ppc] [RFC PATCH v2 7/9] spapr: CPU hotplug support, Bharata B Rao, 2016/03/10
- [Qemu-ppc] [RFC PATCH v2 8/9] xics, xics_kvm: Handle CPU unplug correctly, Bharata B Rao, 2016/03/10
- [Qemu-ppc] [RFC PATCH v2 9/9] spapr: CPU hot unplug support, Bharata B Rao, 2016/03/10
- Re: [Qemu-ppc] [Qemu-devel] [RFC PATCH v2 0/9] Core based CPU hotplug for PowerPC sPAPR, Igor Mammedov, 2016/03/14
- Re: [Qemu-ppc] [Qemu-devel] [RFC PATCH v2 0/9] Core based CPU hotplug for PowerPC sPAPR, Bharata B Rao, 2016/03/15
- Re: [Qemu-ppc] [Qemu-devel] [RFC PATCH v2 0/9] Core based CPU hotplug for PowerPC sPAPR, Igor Mammedov, 2016/03/16
- Re: [Qemu-ppc] [Qemu-devel] [RFC PATCH v2 0/9] Core based CPU hotplug for PowerPC sPAPR, David Gibson, 2016/03/17
- Re: [Qemu-ppc] [Qemu-devel] [RFC PATCH v2 0/9] Core based CPU hotplug for PowerPC sPAPR, Bharata B Rao, 2016/03/17
- Re: [Qemu-ppc] [Qemu-devel] [RFC PATCH v2 0/9] Core based CPU hotplug for PowerPC sPAPR, David Gibson, 2016/03/20
- Re: [Qemu-ppc] [Qemu-devel] [RFC PATCH v2 0/9] Core based CPU hotplug for PowerPC sPAPR, Igor Mammedov, 2016/03/21
- Re: [Qemu-ppc] [Qemu-devel] [RFC PATCH v2 0/9] Core based CPU hotplug for PowerPC sPAPR, David Gibson, 2016/03/21
- Re: [Qemu-ppc] [Qemu-devel] [RFC PATCH v2 0/9] Core based CPU hotplug for PowerPC sPAPR, Igor Mammedov, 2016/03/22