[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH for-2.13 v2 3/5] pc: simplify MachineClass::get_
From: |
Eduardo Habkost |
Subject: |
Re: [Qemu-devel] [PATCH for-2.13 v2 3/5] pc: simplify MachineClass::get_hotplug_handler handling |
Date: |
Wed, 18 Apr 2018 11:56:03 -0300 |
User-agent: |
Mutt/1.9.2 (2017-12-15) |
On Wed, Apr 18, 2018 at 04:28:03PM +0200, Igor Mammedov wrote:
> By default MachineClass::get_hotplug_handler is NULL and concrete board
> should set it to it's own handler.
> Considering there isn't any default handler, drop saving empty
> MachineClass::get_hotplug_handler in child class and make PC code
> consistent with spapr/s390x boards.
>
> We can bring this back when actual usecase surfaces and do it
> consistently across boards that use get_hotplug_handler().
>
> Suggested-by: David Gibson <address@hidden>
> Signed-off-by: Igor Mammedov <address@hidden>
Reviewed-by: Eduardo Habkost <address@hidden>
[...]
> @@ -2344,7 +2341,6 @@ static void pc_machine_class_init(ObjectClass *oc, void
> *data)
> HotplugHandlerClass *hc = HOTPLUG_HANDLER_CLASS(oc);
> NMIClass *nc = NMI_CLASS(oc);
>
> - pcmc->get_hotplug_handler = mc->get_hotplug_handler;
What about assert(!mc->get_hotplug_handler) just before
overwriting it?
--
Eduardo