[libvirt] [PATCH] qemu_hotplug: Remove virQEMUDriverPtr arguments

Cole Robinson crobinso at redhat.com
Wed Apr 3 17:24:07 UTC 2019


On 3/23/19 8:41 PM, Suyang Chen wrote:
> Since commit 2e6ecba1bcac, the pointer to the qemu driver is saved in
> domain object's private data and hence does not have to be passed as
> yet another parameter if domain object is already one of them.
> 
> This just changed qemuDomainChangeDiskLive and
> qemuDomainChangeEjectableMedia functions
> 
> Signed-off-by: Suyang Chen <dawson0xff at gmail.com>

Thanks for the patch. Unfortunately a similar change prompted a
discussion over here:

https://www.redhat.com/archives/libvir-list/2019-March/msg01639.html

And not everyone is in agreement that these types of changes, in
isolation, are useful. So I don't think this patch will be applied. I
will remove the associated bitesizedtask entry from the wiki so this
doesn't cause additional confusion

Thanks,
Cole




More information about the libvir-list mailing list