On 04/09/2018 05:45 PM, Daniel P. Berrangé wrote: > The motivating goal of this series was/is to make it possible to answer > whether there is any driver registered to handle a given URI scheme > without having to call into the drivers. > > Ultimately this ended up being a nice simplification and cleanup of > driver URI opening code, removing repetative logic from all drivers. > > Daniel P. Berrangé (8): > xen: encourage use of xen:///system URI as preferred format > lxc: allow use of lxc:///system URI as preferred format > driver: introduce a driver method for probing default URIs > driver: allow drivers to indicate if they permit remote connections > driver: declare supported URI schemes in virConnectDriver struct > driver: ensure NULL URI isn't passed to drivers with whitelisted URIs > driver: enforce a non-NULL URI scheme > driver: ensure URI path is non-NULL to simplify drivers > > docs/drvlxc.html.in | 34 ++++++------- > docs/drvxen.html.in | 14 +++--- > docs/hvsupport.pl | 6 +-- > docs/remote.html.in | 6 +-- > docs/uri.html.in | 16 +++---- > examples/lxcconvert/virt-lxc-convert | 2 +- > src/bhyve/bhyve_driver.c | 48 +++++++++---------- > src/driver-hypervisor.h | 4 ++ > src/driver.h | 8 ++++ > src/esx/esx_driver.c | 31 +----------- > src/hyperv/hyperv_driver.c | 24 +--------- > src/interface/interface_backend_netcf.c | 47 +++++++----------- > src/interface/interface_backend_udev.c | 47 +++++++----------- > src/libvirt.c | 69 +++++++++++++++++++++++--- > src/libxl/libxl_driver.c | 58 +++++++++++----------- > src/lxc/lxc_driver.c | 56 ++++++++++------------ > src/lxc/lxc_process.c | 4 +- > src/network/bridge_driver.c | 47 +++++++----------- > src/node_device/node_device_driver.c | 45 +++++++---------- > src/node_device/node_device_hal.c | 2 + > src/node_device/node_device_udev.c | 2 + > src/nwfilter/nwfilter_driver.c | 35 +++++--------- > src/openvz/openvz_driver.c | 70 +++++++++++++-------------- > src/phyp/phyp_driver.c | 9 +--- > src/qemu/qemu_driver.c | 85 +++++++++++++++------------------ > src/remote/remote_driver.c | 74 +++++++++++++--------------- > src/secret/secret_driver.c | 47 +++++++----------- > src/storage/storage_driver.c | 47 +++++++----------- > src/test/test_driver.c | 18 ++----- > src/uml/uml_driver.c | 71 +++++++++++++-------------- > src/vbox/vbox_common.c | 26 ++++------ > src/vbox/vbox_driver.c | 18 ++----- > src/vmware/vmware_driver.c | 29 ++++------- > src/vz/vz_driver.c | 25 ++-------- > src/xenapi/xenapi_driver.c | 6 +-- > tools/libvirt-guests.sysconf | 2 +- > tools/virsh.pod | 4 +- > tools/virt-login-shell.c | 2 +- > 38 files changed, 489 insertions(+), 649 deletions(-) > ACK series, but please before pushing: a) read my comments, b) make sure 'make check syntax-check' passes after each patch ;-) Michal -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list
