On Wed, 24 Sep 2014 11:47:49 +0000
Igor Mammedov <imamm...@redhat.com> wrote:

> Series unifies different hotplug mechanisms to a recent
> hotplug-handler API and does shallow conversion of
> devices that still use legacy qdev hotplug to it dropping
> not used after that legacy hotplug path [29/30].
> It also relaces SCSI's own way to do hotplug/unplug with
> hotplug-handler callbacks leaving it the only method
> perform hotplug tasks.
> And the last patch [30/30] allows to unplug of BUS-less
> devices using hotplug-handler API.
> 
> Converted devices are covered with new hotplug
> unit-tests, except of:
>  s390x-*: I have no idea how or means to test it, but code
>           is close to virtio, so it's converted using
>           virtio template
>  pvscsi: is broken, so no means to test it
> 
> Git tree for testing:
>   https://github.com/imammedo/qemu/commits/hp_ctrl_conversion_v1

Gave it a quick test with the s390-ccw-virtio machine.

device_add virtio-balloon-ccw,id=xxx,devno=fe.2.1111

works (device added in qemu and crws injected to guest). However,

device_del xxx

removes the device in qemu but no crws are injected to the guest. That
seems to indicate that virtio_ccw_busdev_unplug() isn't called?

<runs under gdb>

Indeed, the callback isn't invoked for hotunplug.


Reply via email to