Daniel P. Berrange wrote: > On Tue, Jan 26, 2010 at 11:34:28PM -0700, Jim Fehlig wrote: > >> xen-unstable c/s 20762 bumped XEN_SYSCTL_INTERFACE_VERSION to 7. I >> don't see how the interface change affects libvirt, other than failing >> xenHypervisorInit() since version 7 is not tried. >> >> The attached patch accommodates the upcoming Xen 4.0 release by checking >> for XEN_SYSCTL_INTERFACE_VERSION 7. If found, it sets >> XEN_DOMCTL_INTERFACE_VERSION to 6, which is also new to Xen 4.0. >> >> Regards, >> Jim >> > > >> Index: libvirt-0.7.4/src/xen/xen_hypervisor.c >> =================================================================== >> --- libvirt-0.7.4.orig/src/xen/xen_hypervisor.c >> +++ libvirt-0.7.4/src/xen/xen_hypervisor.c >> @@ -2100,12 +2100,14 @@ xenHypervisorInit(void) >> DEBUG0("Using hypervisor call v2, sys ver6 dom ver5\n"); >> goto done; >> } >> - /* Xen 4.0 */ >> + } >> + >> + /* Xen 4.0 */ >> + sys_interface_version = 7; /* XEN_SYSCTL_INTERFACE_VERSION */ >> + if (virXen_getdomaininfo(fd, 0, &info) == 1) { >> dom_interface_version = 6; /* XEN_DOMCTL_INTERFACE_VERSION */ >> - if (virXen_getvcpusinfo(fd, 0, 0, ipt, NULL, 0) == 0){ >> - DEBUG0("Using hypervisor call v2, sys ver6 dom ver6\n"); >> - goto done; >> - } >> + DEBUG0("Using hypervisor call v2, sys ver7 dom ver6\n"); >> + goto done; >> } >> >> hypervisor_version = 1; >> > > > ACK > Thanks. I've pushed this fix. Jim -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list