On Fri, 10 Jan 2025 10:19:02 +0100 Philippe Mathieu-Daudé <[email protected]> wrote:
> Akihiko's v4 patch was doing too many things at once to > my taste, so I split it to follow dumbly each steps. > https://lore.kernel.org/qemu-devel/[email protected]/ refactoring in some cases would lead to device being created and then deleted, where as before it we were bailing out earlier without creating a device. I guess there is no much harm in that overhead and we get consolidated handling instead of scattered checks. Just make checkpatch happy, with that Reviewed-by: Igor Mammedov <[email protected]> > > Akihiko Odaki (6): > hw/qdev: Pass bus argument to qdev_hotplug_allowed() > hw/qdev: Factor qdev_hotunplug_allowed() out > hw/qdev: Introduce qdev_hotplug_unplug_allowed_common() > hw/qdev: Check DevClass::hotpluggable in hotplug_unplug_allowed_common > hw/qdev: Check qbus_is_hotpluggable in hotplug_unplug_allowed_common > hw/qdev: Check machine_hotplug_handler in > hotplug_unplug_allowed_common > > include/hw/qdev-core.h | 3 ++- > hw/core/qdev-hotplug.c | 42 +++++++++++++++++++++++++++++++++++++++++- > system/qdev-monitor.c | 37 +++++-------------------------------- > 3 files changed, 48 insertions(+), 34 deletions(-) >
