[libvirt] [PATCH] Fix detection of Xen sysctl version 9

Jim Fehlig jfehlig at suse.com
Tue Oct 23 17:31:56 UTC 2012


In commit 371ddc98, I mistakenly added the check for sysctl
version 9 after setting the hypercall version to 1, which will
fail with

error : xenHypervisorDoV1Op:967 : Unable to issue hypervisor
ioctl 3166208: Function not implemented

This check should be included along with the others that use
hypercall version 2.
---
 src/xen/xen_hypervisor.c |   14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/xen/xen_hypervisor.c b/src/xen/xen_hypervisor.c
index 3244bbd..406079b 100644
--- a/src/xen/xen_hypervisor.c
+++ b/src/xen/xen_hypervisor.c
@@ -2157,13 +2157,6 @@ xenHypervisorInit(struct xenHypervisorVersions *override_versions)
         }
     }
 
-    hv_versions.hypervisor = 1;
-    hv_versions.sys_interface = -1;
-    if (virXen_getdomaininfo(fd, 0, &info) == 1) {
-        VIR_DEBUG("Using hypervisor call v1");
-        goto done;
-    }
-
     /* Xen 4.2
      * sysctl version 9 -> xen-unstable c/s 24102:dc8e55c90604
      * domctl version 8 -> unchanged from Xen 4.1
@@ -2177,6 +2170,13 @@ xenHypervisorInit(struct xenHypervisorVersions *override_versions)
         }
     }
 
+    hv_versions.hypervisor = 1;
+    hv_versions.sys_interface = -1;
+    if (virXen_getdomaininfo(fd, 0, &info) == 1) {
+        VIR_DEBUG("Using hypervisor call v1");
+        goto done;
+    }
+
     /*
      * we failed to make the getdomaininfolist hypercall
      */
-- 
1.7.10.4




More information about the libvir-list mailing list