From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:50284) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XbUfh-000893-Vd for qemu-devel@nongnu.org; Tue, 07 Oct 2014 09:19:26 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1XbUfb-0004iN-W9 for qemu-devel@nongnu.org; Tue, 07 Oct 2014 09:19:21 -0400 Received: from mx1.redhat.com ([209.132.183.28]:16338) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XbUfb-0004i8-IZ for qemu-devel@nongnu.org; Tue, 07 Oct 2014 09:19:15 -0400 Date: Tue, 7 Oct 2014 15:19:06 +0200 From: Igor Mammedov Message-ID: <20141007151906.0b6b9202@nial.usersys.redhat.com> In-Reply-To: <1411987957-31190-3-git-send-email-guz.fnst@cn.fujitsu.com> References: <1411987957-31190-1-git-send-email-guz.fnst@cn.fujitsu.com> <1411987957-31190-3-git-send-email-guz.fnst@cn.fujitsu.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH V4 2/8] acpi:ich9: convert cpu hotplug handle to hotplug_handler API List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Gu Zheng Cc: chen.fan.fnst@cn.fujitsu.com, isimatu.yasuaki@jp.fujitsu.com, tangchen@cn.fujitsu.com, qemu-devel@nongnu.org, afaerber@suse.de On Mon, 29 Sep 2014 18:52:31 +0800 Gu Zheng wrote: > Convert notifier based hotplug handle to hotplug_handler API. drop 'handle' above and in subj. the same applies to the next patch > > Signed-off-by: Gu Zheng > --- > hw/acpi/ich9.c | 13 ++----------- > include/hw/acpi/ich9.h | 1 - > 2 files changed, 2 insertions(+), 12 deletions(-) > > diff --git a/hw/acpi/ich9.c b/hw/acpi/ich9.c > index 7b14bbb..7585364 100644 > --- a/hw/acpi/ich9.c > +++ b/hw/acpi/ich9.c > @@ -209,15 +209,6 @@ static void pm_powerdown_req(Notifier *n, void *opaque) > acpi_pm1_evt_power_down(&pm->acpi_regs); > } > > -static void ich9_cpu_added_req(Notifier *n, void *opaque) > -{ > - ICH9LPCPMRegs *pm = container_of(n, ICH9LPCPMRegs, cpu_added_notifier); > - > - assert(pm != NULL); > - AcpiCpuHotplug_add(&pm->acpi_regs.gpe, &pm->gpe_cpu, CPU(opaque)); > - acpi_update_sci(&pm->acpi_regs, pm->irq); > -} > - > void ich9_pm_init(PCIDevice *lpc_pci, ICH9LPCPMRegs *pm, > qemu_irq sci_irq) > { > @@ -246,8 +237,6 @@ void ich9_pm_init(PCIDevice *lpc_pci, ICH9LPCPMRegs *pm, > > AcpiCpuHotplug_init(pci_address_space_io(lpc_pci), OBJECT(lpc_pci), > &pm->gpe_cpu, ICH9_CPU_HOTPLUG_IO_BASE); > - pm->cpu_added_notifier.notify = ich9_cpu_added_req; > - qemu_register_cpu_added_notifier(&pm->cpu_added_notifier); > > if (pm->acpi_memory_hotplug.is_enabled) { > acpi_memory_hotplug_init(pci_address_space_io(lpc_pci), OBJECT(lpc_pci), > @@ -304,6 +293,8 @@ void ich9_pm_device_plug_cb(ICH9LPCPMRegs *pm, DeviceState *dev, Error **errp) > object_dynamic_cast(OBJECT(dev), TYPE_PC_DIMM)) { > acpi_memory_plug_cb(&pm->acpi_regs, pm->irq, &pm->acpi_memory_hotplug, > dev, errp); > + } else if (object_dynamic_cast(OBJECT(dev), TYPE_CPU)) { > + acpi_cpu_plug_cb(&pm->acpi_regs, pm->irq, &pm->gpe_cpu, dev, errp); > } else { > error_setg(errp, "acpi: device plug request for not supported device" > " type: %s", object_get_typename(OBJECT(dev))); > diff --git a/include/hw/acpi/ich9.h b/include/hw/acpi/ich9.h > index 7e42448..fe975e6 100644 > --- a/include/hw/acpi/ich9.h > +++ b/include/hw/acpi/ich9.h > @@ -47,7 +47,6 @@ typedef struct ICH9LPCPMRegs { > Notifier powerdown_notifier; > > AcpiCpuHotplug gpe_cpu; > - Notifier cpu_added_notifier; > > MemHotplugState acpi_memory_hotplug; > } ICH9LPCPMRegs;