On Fri, Jul 15, 2016 at 03:49:12PM +0800, Xiao Guangrong wrote:
>
>
> On 07/14/2016 08:17 PM, Stefan Hajnoczi wrote:
> > On Mon, Jul 11, 2016 at 09:45:17PM +0800, Xiao Guangrong wrote:
> > > diff --git a/hw/acpi/memory_hotplug.c b/hw/acpi/memory_hotplug.c
> > > index 73fa62d..d1c2e92 100644
> > > --- a/hw/acpi/memory_hotplug.c
> > > +++ b/hw/acpi/memory_hotplug.c
> > > @@ -239,11 +239,6 @@ void acpi_memory_plug_cb(HotplugHandler *hotplug_dev, MemHotplugState *mem_st,
> > > DeviceState *dev, Error **errp)
> > > {
> > > MemStatus *mdev;
> > > - DeviceClass *dc = DEVICE_GET_CLASS(dev);
> > > -
> > > - if (!dc->hotpluggable) {
> > > - return;
> > > - }
> > >
> > > mdev = acpi_memory_slot_status(mem_st, dev, errp);
> > > if (!mdev) {
> >
> > Did you mean to include this hunk in the patch? Looks like something
> > left over from debugging/prototyping.
> >
>
> As all memory devices, both pc-dimm and nvdimm, have supported hotplug now, i
> think this chunk can be removed.
>
> In fact, this check was introduced by nvdimm. :)
Fair enough.
Stefan
Attachment:
signature.asc
Description: PGP signature
