Il 04/09/2012 17:13, Cornelia Huck ha scritto: > +VirtioCcwBus *virtio_ccw_bus_init(void) > +{ > + VirtioCcwBus *cbus; > + BusState *bus; > + DeviceState *dev; > + > + /* Create bridge device */ > + dev = qdev_create(NULL, "virtio-ccw-bridge"); > + qdev_init_nofail(dev); > + > + /* Create bus on bridge device */ > + bus = qbus_create(TYPE_VIRTIO_CCW_BUS, dev, "virtio-ccw"); > + cbus = DO_UPCAST(VirtioCcwBus, bus, bus); > + > + /* Enable hotplugging */ > + bus->allow_hotplug = 1; > + > + qemu_register_reset(virtio_ccw_reset_subchannels, cbus);
Please use qdev device-reset and bus-reset callbacks instead of this. In particular, when writing the status you should call qdev_reset_all(DEVICE(sch)), and whatever state should be reset will have to be cleared by the device-reset callback of SubchDev, including calling virtio_reset. Everything else will be cleared instead by the bus-reset callback of virtio-ccw-bus, similar to what you are doing in virtio_ccw_reset_subchannels. Paolo > + return cbus; > +}