On Fri, Jan 19, 2024 at 04:35:12PM +0000, Peter Maydell wrote: > Date: Fri, 19 Jan 2024 16:35:12 +0000 > From: Peter Maydell <[email protected]> > Subject: [PATCH 5/5] hw/core: Remove transitional infrastructure from > BusClass > X-Mailer: git-send-email 2.34.1 > > BusClass currently has transitional infrastructure to support > subclasses which implement the legacy BusClass::reset method rather > than the Resettable interface. We have now removed all the users of > BusClass::reset in the tree, so we can remove the transitional > infrastructure. > > Signed-off-by: Peter Maydell <[email protected]> > --- > include/hw/qdev-core.h | 2 -- > hw/core/bus.c | 67 ------------------------------------------ > 2 files changed, 69 deletions(-)
Reviewed-by: Zhao Liu <[email protected]> It seems the similar cleanup for DeviceClass needs a lot of effort. > > diff --git a/include/hw/qdev-core.h b/include/hw/qdev-core.h > index 151d9682380..986c924fa55 100644 > --- a/include/hw/qdev-core.h > +++ b/include/hw/qdev-core.h > @@ -329,8 +329,6 @@ struct BusClass { > */ > char *(*get_fw_dev_path)(DeviceState *dev); > > - void (*reset)(BusState *bus); > - > /* > * Return whether the device can be added to @bus, > * based on the address that was set (via device properties) > diff --git a/hw/core/bus.c b/hw/core/bus.c > index c7831b5293b..b9d89495cdf 100644 > --- a/hw/core/bus.c > +++ b/hw/core/bus.c > @@ -232,57 +232,6 @@ static char *default_bus_get_fw_dev_path(DeviceState > *dev) > return g_strdup(object_get_typename(OBJECT(dev))); > } > > -/** > - * bus_phases_reset: > - * Transition reset method for buses to allow moving > - * smoothly from legacy reset method to multi-phases > - */ > -static void bus_phases_reset(BusState *bus) > -{ > - ResettableClass *rc = RESETTABLE_GET_CLASS(bus); > - > - if (rc->phases.enter) { > - rc->phases.enter(OBJECT(bus), RESET_TYPE_COLD); > - } > - if (rc->phases.hold) { > - rc->phases.hold(OBJECT(bus)); > - } > - if (rc->phases.exit) { > - rc->phases.exit(OBJECT(bus)); > - } > -} > - > -static void bus_transitional_reset(Object *obj) > -{ > - BusClass *bc = BUS_GET_CLASS(obj); > - > - /* > - * This will call either @bus_phases_reset (for multi-phases transitioned > - * buses) or a bus's specific method for not-yet transitioned buses. > - * In both case, it does not reset children. > - */ > - if (bc->reset) { > - bc->reset(BUS(obj)); > - } > -} > - > -/** > - * bus_get_transitional_reset: > - * check if the bus's class is ready for multi-phase > - */ > -static ResettableTrFunction bus_get_transitional_reset(Object *obj) > -{ > - BusClass *dc = BUS_GET_CLASS(obj); > - if (dc->reset != bus_phases_reset) { > - /* > - * dc->reset has been overridden by a subclass, > - * the bus is not ready for multi phase yet. > - */ > - return bus_transitional_reset; > - } > - return NULL; > -} > - > static void bus_class_init(ObjectClass *class, void *data) > { > BusClass *bc = BUS_CLASS(class); > @@ -293,22 +242,6 @@ static void bus_class_init(ObjectClass *class, void > *data) > > rc->get_state = bus_get_reset_state; > rc->child_foreach = bus_reset_child_foreach; > - > - /* > - * @bus_phases_reset is put as the default reset method below, allowing > - * to do the multi-phase transition from base classes to leaf classes. It > - * allows a legacy-reset Bus class to extend a multi-phases-reset > - * Bus class for the following reason: > - * + If a base class B has been moved to multi-phase, then it does not > - * override this default reset method and may have defined phase > methods. > - * + A child class C (extending class B) which uses > - * bus_class_set_parent_reset() (or similar means) to override the > - * reset method will still work as expected. @bus_phases_reset function > - * will be registered as the parent reset method and effectively call > - * parent reset phases. > - */ > - bc->reset = bus_phases_reset; > - rc->get_transitional_function = bus_get_transitional_reset; > } > > static void qbus_finalize(Object *obj) > -- > 2.34.1 > >
