[libvirt PATCH] remote: fix driver name check for libxl driver
Jim Fehlig
jfehlig at suse.com
Mon May 4 17:33:46 UTC 2020
On 5/4/20 10:48 AM, Daniel P. Berrangé wrote:
> The virConnectGetType() returns "xenlight" for libxl, not "LIBXL".
The libxl driver implements connectGetType, where it returns "Xen"
https://gitlab.com/libvirt/libvirt/-/blob/master/src/libxl/libxl_driver.c#L909
Is the driver function table not initialized, in which case virConnectGetType
returns the driver's name? Either way, I'm really lamenting my choice of names
and inconsistent use of them in the libxl driver :-(. But I don't think it is
possible to change the type returned through virConnectGetType, as that could
break existing users.
Regards,
Jim
> This prevents users opening a connection to the libxl driver when using
> the modular daemons.
>
> Signed-off-by: Daniel P. Berrangé <berrange at redhat.com>
> ---
> src/remote/remote_daemon_dispatch.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/remote/remote_daemon_dispatch.c b/src/remote/remote_daemon_dispatch.c
> index 5d1c6971c0..a8ac795d71 100644
> --- a/src/remote/remote_daemon_dispatch.c
> +++ b/src/remote/remote_daemon_dispatch.c
> @@ -2111,7 +2111,7 @@ remoteDispatchConnectOpen(virNetServerPtr server G_GNUC_UNUSED,
>
> VIR_DEBUG("Primary driver type is '%s'", type);
> if (STREQ(type, "QEMU") ||
> - STREQ(type, "LIBXL") ||
> + STREQ(type, "xenlight") ||
> STREQ(type, "LXC") ||
> STREQ(type, "VBOX") ||
> STREQ(type, "bhyve") ||
>
More information about the libvir-list
mailing list