[PATCH] remote: fix prefix for libxl Xen driver

Daniel P. Berrangé posted 1 patch 2 years, 10 months ago
Test syntax-check failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20210618140926.37809-1-berrange@redhat.com
src/remote/remote_daemon_dispatch.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] remote: fix prefix for libxl Xen driver
Posted by Daniel P. Berrangé 2 years, 10 months ago
The libxl driver supports xen:///system URLs and the daemon socket
uses 'virtxend' as the socket prefix.

Reported-by: Jim Fehlig <jfehlig@suse.com>
Signed-off-by: Daniel P. Berrangé <berrange@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 838f4a925f..dd797a81f7 100644
--- a/src/remote/remote_daemon_dispatch.c
+++ b/src/remote/remote_daemon_dispatch.c
@@ -2021,7 +2021,7 @@ remoteDispatchProbeURI(bool readonly,
          * calls in daemonInitialize */
         const char *drivers[] = {
 # ifdef WITH_LIBXL
-            "libxl",
+            "xen",
 # endif
 # ifdef WITH_QEMU
             "qemu",
-- 
2.31.1

Re: [PATCH] remote: fix prefix for libxl Xen driver
Posted by Jim Fehlig 2 years, 10 months ago
On 6/18/21 8:09 AM, Daniel P. Berrangé wrote:
> The libxl driver supports xen:///system URLs and the daemon socket
> uses 'virtxend' as the socket prefix.
> 
> Reported-by: Jim Fehlig <jfehlig@suse.com>
> Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
> ---
>   src/remote/remote_daemon_dispatch.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Jim Fehlig <jfehlig@suse.com>

Regards,
Jim

> 
> diff --git a/src/remote/remote_daemon_dispatch.c b/src/remote/remote_daemon_dispatch.c
> index 838f4a925f..dd797a81f7 100644
> --- a/src/remote/remote_daemon_dispatch.c
> +++ b/src/remote/remote_daemon_dispatch.c
> @@ -2021,7 +2021,7 @@ remoteDispatchProbeURI(bool readonly,
>            * calls in daemonInitialize */
>           const char *drivers[] = {
>   # ifdef WITH_LIBXL
> -            "libxl",
> +            "xen",
>   # endif
>   # ifdef WITH_QEMU
>               "qemu",
>