On Fri, 26 Sep 2014 13:31:21 +0200 Igor Mammedov <imamm...@redhat.com> wrote:
> On Fri, 26 Sep 2014 13:15:16 +0200 > Cornelia Huck <cornelia.h...@de.ibm.com> wrote: > > > On Fri, 26 Sep 2014 09:28:29 +0000 > > Igor Mammedov <imamm...@redhat.com> wrote: > > > > > Signed-off-by: Igor Mammedov <imamm...@redhat.com> > > > --- > > > v2: > > > fix not working unplug > > > > While the current code for s390-virtio implements unplug, the transport > > per se does not really support unplug (and never has), only hotplugging > > of devices. I removed the ->unplug callback and qemu now behaves like > > I'd expect: I can add devices with device_add, but device_del leaves > > them in place. That would be the way to go for me. > Would a path to remove unplug on top of this one do or you'd prefer > me to respin this patch without unplug? Whatever works best for you. I don't think it's worth adding an extra patch except if it saves you a respin of the whole series.