<html> <head> <meta http-equiv="Content-Type" content="text/html; charset=gb2312"> <meta name="Generator" content="Microsoft Word 14 (filtered medium)"> <style></style> </head> <body lang="ZH-CN" link="blue" vlink="purple" style="text-justify-trim:punctuation"> <div class="WordSection1"> Hi,<o:p></o:p> <o:p> </o:p> I am getting the following error when I try to migrate VM on my network using virsh migrate:<o:p></o:p> error : qemuMonitorJSONCheckError:359 : internal error unable to execute QEMU command 'getfd': No file descriptor supplied via SCM_RIGHTS ; <o:p></o:p> but, other vm's running on the server successfully migrated to remote server;<o:p></o:p> <o:p> </o:p> Libvirt Version: libvirt-0.10.2-46.el6.x86_64; <o:p></o:p> Qemu-kvm Version: qemu-kvm-0.12.1.2-2.448.el6.3.x86_64; <o:p></o:p> OS Version: Red Hat Enterprise Linux Server release 6.6; <o:p></o:p> <o:p> </o:p> Thanks~<o:p></o:p> <o:p> </o:p> <o:p> </o:p> <o:p> </o:p> <o:p> </o:p> <o:p> </o:p> <o:p> </o:p> Work Hard, Have Fun, Make History!<o:p></o:p> <o:p> </o:p> Mail: raojunming113@pingan.com<o:p></o:p> <o:p> </o:p> <o:p> </o:p> -----邮件原件----- 发件人: libvir-list-bounces@redhat.com [mailto:libvir-list-bounces@redhat.com] 代表 libvir-list-request@redhat.com 发送时间: 2016年5月14日 7:47 收件人: libvir-list@redhat.com 主题: libvir-list Digest, Vol 126, Issue 182 <o:p> </o:p> Send libvir-list mailing list submissions to<o:p></o:p> <a href="mailto:libvir-list@redhat.com"> libvir-list@redhat.com</a><o:p></o:p> <o:p> </o:p> To subscribe or unsubscribe via the World Wide Web, visit<o:p></o:p> <a href="https://www.redhat.com/mailman/listinfo/libvir-list"> https://www.redhat.com/mailman/listinfo/libvir-list</a><o:p></o:p> or, via email, send a message with subject or body 'help' to<o:p></o:p> <a href="mailto:libvir-list-request@redhat.com"> libvir-list-request@redhat.com</a><o:p></o:p> <o:p> </o:p> You can reach the person managing the list at<o:p></o:p> <a href="mailto:libvir-list-owner@redhat.com"> libvir-list-owner@redhat.com</a><o:p></o:p> <o:p> </o:p> When replying, please edit your Subject line so it is more specific than "Re: Contents of libvir-list digest..."<o:p></o:p> <o:p> </o:p> <o:p> </o:p> Today's Topics:<o:p></o:p> <o:p> </o:p> 1. [PATCH 3/6] iscsi: Export virISCSITargetAutologin (John Ferlan)<o:p></o:p> 2. [PATCH 4/6] util: Add "--op nonpersistent" to iSCSI<o:p></o:p> sendtargets (John Ferlan)<o:p></o:p> 3. [PATCH 5/6] iscsi: Inhibit autologin for only libvirt managed<o:p></o:p> targets (John Ferlan)<o:p></o:p> 4. [PATCH 6/6] iscsi: Remove initiatoriqn from<o:p></o:p> virISCSIScanTargets (John Ferlan)<o:p></o:p> 5. [PATCH] capabilities: Advertise cpuselection if -cpu host is<o:p></o:p> usable (Jiri Denemark)<o:p></o:p> 6. Re: [PATCH 0/2] libxl: support pvusb (Jim Fehlig)<o:p></o:p> 7. Re: [PATCH] libxl: add .domainInterfaceAddresses (Jim Fehlig)<o:p></o:p> <o:p> </o:p> <o:p> </o:p> ----------------------------------------------------------------------<o:p></o:p> <o:p> </o:p> Message: 1<o:p></o:p> Date: Fri, 13 May 2016 17:29:23 -0400<o:p></o:p> From: John Ferlan <<a href="mailto:jferlan@redhat.com">jferlan@redhat.com</a>><o:p></o:p> To: <a href="mailto:libvir-list@redhat.com"> libvir-list@redhat.com</a><o:p></o:p> Subject: [libvirt] [PATCH 3/6] iscsi: Export virISCSITargetAutologin<o:p></o:p> Message-ID: <<a href="mailto:1463174966-3185-4-git-send-email-jferlan@redhat.com">1463174966-3185-4-git-send-email-jferlan@redhat.com</a>><o:p></o:p> <o:p> </o:p> Make it accessible for future patches<o:p></o:p> <o:p> </o:p> Signed-off-by: John Ferlan <<a href="mailto:jferlan@redhat.com">jferlan@redhat.com</a>><o:p></o:p> ---<o:p></o:p> src/libvirt_private.syms | 1 +<o:p></o:p> src/util/viriscsi.c | 2 +-<o:p></o:p> src/util/viriscsi.h | 7 +++++++<o:p></o:p> 3 files changed, 9 insertions(+), 1 deletion(-)<o:p></o:p> <o:p> </o:p> diff --git a/src/libvirt_private.syms b/src/libvirt_private.syms index fb24808..719d753 100644<o:p></o:p> --- a/src/libvirt_private.syms<o:p></o:p> +++ b/src/libvirt_private.syms<o:p></o:p> @@ -1675,6 +1675,7 @@ virISCSIGetSession; virISCSINodeUpdate; virISCSIRescanLUNs; virISCSIScanTargets;<o:p></o:p> +virISCSITargetAutologin;<o:p></o:p> <o:p></o:p> <o:p></o:p> # util/virjson.h<o:p></o:p> diff --git a/src/util/viriscsi.c b/src/util/viriscsi.c index be296d7..ea01b3d 100644<o:p></o:p> --- a/src/util/viriscsi.c<o:p></o:p> +++ b/src/util/viriscsi.c<o:p></o:p> @@ -404,7 +404,7 @@ virISCSIGetTargets(char **const groups, }<o:p></o:p> <o:p></o:p> <o:p></o:p> -static int<o:p></o:p> +int<o:p></o:p> virISCSITargetAutologin(const char *portal,<o:p></o:p> const char *initiatoriqn,<o:p></o:p> const char *target, diff --git a/src/util/viriscsi.h b/src/util/viriscsi.h index f4093f7..3e7ea68 100644<o:p></o:p> --- a/src/util/viriscsi.h<o:p></o:p> +++ b/src/util/viriscsi.h<o:p></o:p> @@ -48,6 +48,13 @@ virISCSIRescanLUNs(const char *session)<o:p></o:p> ATTRIBUTE_NONNULL(1) ATTRIBUTE_RETURN_CHECK;<o:p></o:p> <o:p></o:p> int<o:p></o:p> +virISCSITargetAutologin(const char *portal,<o:p></o:p> + const char *initiatoriqn,<o:p></o:p> + const char *target,<o:p></o:p> + bool enable)<o:p></o:p> + ATTRIBUTE_NONNULL(1) ATTRIBUTE_NONNULL(3) ATTRIBUTE_RETURN_CHECK;<o:p></o:p> +<o:p></o:p> +int<o:p></o:p> virISCSIScanTargets(const char *portal,<o:p></o:p> const char *initiatoriqn,<o:p></o:p> size_t *ntargetsret,<o:p></o:p> --<o:p></o:p> 2.5.5<o:p></o:p> <o:p> </o:p> <o:p> </o:p> <o:p> </o:p> ------------------------------<o:p></o:p> <o:p> </o:p> Message: 2<o:p></o:p> Date: Fri, 13 May 2016 17:29:24 -0400<o:p></o:p> From: John Ferlan <<a href="mailto:jferlan@redhat.com">jferlan@redhat.com</a>><o:p></o:p> To: <a href="mailto:libvir-list@redhat.com"> libvir-list@redhat.com</a><o:p></o:p> Subject: [libvirt] [PATCH 4/6] util: Add "--op nonpersistent" to iSCSI<o:p></o:p> sendtargets<o:p></o:p> Message-ID: <<a href="mailto:1463174966-3185-5-git-send-email-jferlan@redhat.com">1463174966-3185-5-git-send-email-jferlan@redhat.com</a>><o:p></o:p> <o:p> </o:p> From: Fritz Elfert <<a href="mailto:fritz@fritz-elfert.de">fritz@fritz-elfert.de</a>><o:p></o:p> <o:p> </o:p> The root cause of commit id '3c12b654' was to ensure that an iSCSI<o:p></o:p> sendtargets command to discover all the available targets on the<o:p></o:p> host didn't reset the "node.startup" value for a future restarts<o:p></o:p> on any target that libvirt is managing.<o:p></o:p> <o:p> </o:p> However, by adding the "--op nonpersistent" inhibits the iSCSI<o:p></o:p> initiator from recording/modifying in the bowels /var/lib/iscsi<o:p></o:p> of any target.<o:p></o:p> <o:p> </o:p> Signed-off-by: John Ferlan <<a href="mailto:jferlan@redhat.com">jferlan@redhat.com</a>><o:p></o:p> ---<o:p></o:p> src/util/viriscsi.c | 1 +<o:p></o:p> tests/viriscsitest.c | 4 +++-<o:p></o:p> 2 files changed, 4 insertions(+), 1 deletion(-)<o:p></o:p> <o:p> </o:p> diff --git a/src/util/viriscsi.c b/src/util/viriscsi.c<o:p></o:p> index ea01b3d..36612c5 100644<o:p></o:p> --- a/src/util/viriscsi.c<o:p></o:p> +++ b/src/util/viriscsi.c<o:p></o:p> @@ -446,6 +446,7 @@ virISCSIScanTargets(const char *portal,<o:p></o:p> "--mode", "discovery",<o:p></o:p> "--type", "sendtargets",<o:p></o:p> "--portal", portal,<o:p></o:p> + "--op", "nonpersistent",<o:p></o:p> NULL);<o:p></o:p> <o:p></o:p> memset(&list, 0, sizeof(list));<o:p></o:p> diff --git a/tests/viriscsitest.c b/tests/viriscsitest.c<o:p></o:p> index c697a4a..b5b0e20 100644<o:p></o:p> --- a/tests/viriscsitest.c<o:p></o:p> +++ b/tests/viriscsitest.c<o:p></o:p> @@ -90,7 +90,9 @@ static void testIscsiadmCb(const char *const*args,<o:p></o:p> args[4] && STREQ(args[4], "sendtargets") &&<o:p></o:p> args[5] && STREQ(args[5], "--portal") &&<o:p></o:p> args[6] && STREQ(args[6], "10.20.30.40:3260,1") &&<o:p></o:p> - args[7] == NULL) {<o:p></o:p> + args[7] && STREQ(args[7], "--op") &&<o:p></o:p> + args[8] && STREQ(args[8], "nonpersistent") &&<o:p></o:p> + args[9] == NULL) {<o:p></o:p> ignore_value(VIR_STRDUP(*output, iscsiadmSendtargetsOutput));<o:p></o:p> } else {<o:p></o:p> *status = -1;<o:p></o:p> -- <o:p></o:p> 2.5.5<o:p></o:p> <o:p> </o:p> <o:p> </o:p> <o:p> </o:p> ------------------------------<o:p></o:p> <o:p> </o:p> Message: 3<o:p></o:p> Date: Fri, 13 May 2016 17:29:25 -0400<o:p></o:p> From: John Ferlan <<a href="mailto:jferlan@redhat.com">jferlan@redhat.com</a>><o:p></o:p> To: <a href="mailto:libvir-list@redhat.com"> libvir-list@redhat.com</a><o:p></o:p> Subject: [libvirt] [PATCH 5/6] iscsi: Inhibit autologin for only<o:p></o:p> libvirt managed targets<o:p></o:p> Message-ID: <<a href="mailto:1463174966-3185-6-git-send-email-jferlan@redhat.com">1463174966-3185-6-git-send-email-jferlan@redhat.com</a>><o:p></o:p> <o:p> </o:p> <a href="https://bugzilla.redhat.com/show_bug.cgi?id=1331552">https://bugzilla.redhat.com/show_bug.cgi?id=1331552</a><o:p></o:p> <o:p> </o:p> Based on code originally posted by Fritz Elfert <<a href="mailto:fritz@fritz-elfert.de">fritz@fritz-elfert.de</a>><o:p></o:p> to remove the Autologin code entirely from libvirt, but reworked to only<o:p></o:p> set Autologin for libvirt managed targets.<o:p></o:p> <o:p> </o:p> Commit id '3c12b654' took a "large hammer" approach to inhibiting logins<o:p></o:p> which causes issues if there are iSCSI targets not being managed by libvirt.<o:p></o:p> <o:p> </o:p> Now that the previous commit ensures that the iscsi initiator doesn't update<o:p></o:p> the /var/lib/iscsi tree with the results for a 'sendtargets' by using the<o:p></o:p> "--op nonpersistent" option, let's remove the code from virISCSIScanTargets<o:p></o:p> that disables autologin for every target, but add that same setting into<o:p></o:p> the start pool code for each managed/started target to ensure that nothing<o:p></o:p> else goes and tries to autologin.<o:p></o:p> <o:p> </o:p> Signed-off-by: John Ferlan <<a href="mailto:jferlan@redhat.com">jferlan@redhat.com</a>><o:p></o:p> ---<o:p></o:p> src/storage/storage_backend_iscsi.c | 11 +++++++++++<o:p></o:p> src/util/viriscsi.c | 15 ++-------------<o:p></o:p> 2 files changed, 13 insertions(+), 13 deletions(-)<o:p></o:p> <o:p> </o:p> diff --git a/src/storage/storage_backend_iscsi.c b/src/storage/storage_backend_iscsi.c<o:p></o:p> index 9e2d01e..5fbf390 100644<o:p></o:p> --- a/src/storage/storage_backend_iscsi.c<o:p></o:p> +++ b/src/storage/storage_backend_iscsi.c<o:p></o:p> @@ -404,6 +404,17 @@ virStorageBackendISCSIStartPool(virConnectPtr conn,<o:p></o:p> NULL, NULL) < 0)<o:p></o:p> goto cleanup;<o:p></o:p> <o:p></o:p> + /* Inhibit our autologin for our managed source device */<o:p></o:p> + if (virISCSITargetAutologin(portal,<o:p></o:p> + pool->def->source.initiator.iqn,<o:p></o:p> + pool->def->source.devices[0].path,<o:p></o:p> + false) < 0) {<o:p></o:p> + virReportError(VIR_ERR_INTERNAL_ERROR,<o:p></o:p> + _("failed to inhibit autologin for target '%s'"),<o:p></o:p> + pool->def->source.devices[0].path);<o:p></o:p> + goto cleanup;<o:p></o:p> + }<o:p></o:p> +<o:p></o:p> if (virStorageBackendISCSISetAuth(portal, conn, &pool->def->source) < 0)<o:p></o:p> goto cleanup;<o:p></o:p> <o:p></o:p> diff --git a/src/util/viriscsi.c b/src/util/viriscsi.c<o:p></o:p> index 36612c5..3133d88 100644<o:p></o:p> --- a/src/util/viriscsi.c<o:p></o:p> +++ b/src/util/viriscsi.c<o:p></o:p> @@ -415,13 +415,14 @@ virISCSITargetAutologin(const char *portal,<o:p></o:p> "--value", enable ? "automatic" : "manual",<o:p></o:p> NULL };<o:p></o:p> <o:p></o:p> + VIR_DEBUG("set autologin for '%s' to '%d'", target, enable);<o:p></o:p> return virISCSIConnection(portal, initiatoriqn, target, extraargv);<o:p></o:p> }<o:p></o:p> <o:p></o:p> <o:p></o:p> int<o:p></o:p> virISCSIScanTargets(const char *portal,<o:p></o:p> - const char *initiatoriqn,<o:p></o:p> + const char *initiatoriqn ATTRIBUTE_UNUSED,<o:p></o:p> size_t *ntargetsret,<o:p></o:p> char ***targetsret)<o:p></o:p> {<o:p></o:p> @@ -459,18 +460,6 @@ virISCSIScanTargets(const char *portal,<o:p></o:p> &list, NULL, NULL) < 0)<o:p></o:p> goto cleanup;<o:p></o:p> <o:p></o:p> - for (i = 0; i < list.ntargets; i++) {<o:p></o:p> - /* We have to ignore failure, because we can't undo<o:p></o:p> - * the results of 'sendtargets', unless we go scrubbing<o:p></o:p> - * around in the dirt in /var/lib/iscsi.<o:p></o:p> - */<o:p></o:p> - if (virISCSITargetAutologin(portal,<o:p></o:p> - initiatoriqn,<o:p></o:p> - list.targets[i], false) < 0)<o:p></o:p> - VIR_WARN("Unable to disable auto-login on iSCSI target %s: %s",<o:p></o:p> - portal, list.targets[i]);<o:p></o:p> - }<o:p></o:p> -<o:p></o:p> if (ntargetsret && targetsret) {<o:p></o:p> *ntargetsret = list.ntargets;<o:p></o:p> *targetsret = list.targets;<o:p></o:p> -- <o:p></o:p> 2.5.5<o:p></o:p> <o:p> </o:p> <o:p> </o:p> <o:p> </o:p> ------------------------------<o:p></o:p> <o:p> </o:p> Message: 4<o:p></o:p> Date: Fri, 13 May 2016 17:29:26 -0400<o:p></o:p> From: John Ferlan <<a href="mailto:jferlan@redhat.com">jferlan@redhat.com</a>><o:p></o:p> To: <a href="mailto:libvir-list@redhat.com"> libvir-list@redhat.com</a><o:p></o:p> Subject: [libvirt] [PATCH 6/6] iscsi: Remove initiatoriqn from<o:p></o:p> virISCSIScanTargets<o:p></o:p> Message-ID: <<a href="mailto:1463174966-3185-7-git-send-email-jferlan@redhat.com">1463174966-3185-7-git-send-email-jferlan@redhat.com</a>><o:p></o:p> <o:p> </o:p> No longer necessary to have it, so remove it.<o:p></o:p> <o:p> </o:p> Signed-off-by: John Ferlan <<a href="mailto:jferlan@redhat.com">jferlan@redhat.com</a>><o:p></o:p> ---<o:p></o:p> src/storage/storage_backend_iscsi.c | 8 ++------<o:p></o:p> src/util/viriscsi.c | 3 +--<o:p></o:p> src/util/viriscsi.h | 1 -<o:p></o:p> tests/viriscsitest.c | 3 +--<o:p></o:p> 4 files changed, 4 insertions(+), 11 deletions(-)<o:p></o:p> <o:p> </o:p> diff --git a/src/storage/storage_backend_iscsi.c b/src/storage/storage_backend_iscsi.c<o:p></o:p> index 5fbf390..b6b23d3 100644<o:p></o:p> --- a/src/storage/storage_backend_iscsi.c<o:p></o:p> +++ b/src/storage/storage_backend_iscsi.c<o:p></o:p> @@ -197,9 +197,7 @@ virStorageBackendISCSIFindPoolSources(virConnectPtr conn ATTRIBUTE_UNUSED,<o:p></o:p> if (!(portal = virStorageBackendISCSIPortal(source)))<o:p></o:p> goto cleanup;<o:p></o:p> <o:p></o:p> - if (virISCSIScanTargets(portal,<o:p></o:p> - source->initiator.iqn,<o:p></o:p> - &ntargets, &targets) < 0)<o:p></o:p> + if (virISCSIScanTargets(portal, &ntargets, &targets) < 0)<o:p></o:p> goto cleanup;<o:p></o:p> <o:p></o:p> if (VIR_ALLOC_N(list.sources, ntargets) < 0)<o:p></o:p> @@ -399,9 +397,7 @@ virStorageBackendISCSIStartPool(virConnectPtr conn,<o:p></o:p> * iscsiadm doesn't let you login to a target, unless you've<o:p></o:p> * first issued a 'sendtargets' command to the portal :-(<o:p></o:p> */<o:p></o:p> - if (virISCSIScanTargets(portal,<o:p></o:p> - pool->def->source.initiator.iqn,<o:p></o:p> - NULL, NULL) < 0)<o:p></o:p> + if (virISCSIScanTargets(portal, NULL, NULL) < 0)<o:p></o:p> goto cleanup;<o:p></o:p> <o:p></o:p> /* Inhibit our autologin for our managed source device */<o:p></o:p> diff --git a/src/util/viriscsi.c b/src/util/viriscsi.c<o:p></o:p> index 3133d88..a2acb67 100644<o:p></o:p> --- a/src/util/viriscsi.c<o:p></o:p> +++ b/src/util/viriscsi.c<o:p></o:p> @@ -321,7 +321,7 @@ virISCSIConnection(const char *portal,<o:p></o:p> * portal. Without the sendtargets all that is received is a<o:p></o:p> * "iscsiadm: No records found"<o:p></o:p> */<o:p></o:p> - if (virISCSIScanTargets(portal, initiatoriqn, NULL, NULL) < 0)<o:p></o:p> + if (virISCSIScanTargets(portal, NULL, NULL) < 0)<o:p></o:p> goto cleanup;<o:p></o:p> <o:p></o:p> break;<o:p></o:p> @@ -422,7 +422,6 @@ virISCSITargetAutologin(const char *portal,<o:p></o:p> <o:p></o:p> int<o:p></o:p> virISCSIScanTargets(const char *portal,<o:p></o:p> - const char *initiatoriqn ATTRIBUTE_UNUSED,<o:p></o:p> size_t *ntargetsret,<o:p></o:p> char ***targetsret)<o:p></o:p> {<o:p></o:p> diff --git a/src/util/viriscsi.h b/src/util/viriscsi.h<o:p></o:p> index 3e7ea68..44fb1f4 100644<o:p></o:p> --- a/src/util/viriscsi.h<o:p></o:p> +++ b/src/util/viriscsi.h<o:p></o:p> @@ -56,7 +56,6 @@ virISCSITargetAutologin(const char *portal,<o:p></o:p> <o:p></o:p> int<o:p></o:p> virISCSIScanTargets(const char *portal,<o:p></o:p> - const char *initiatoriqn,<o:p></o:p> size_t *ntargetsret,<o:p></o:p> char ***targetsret)<o:p></o:p> ATTRIBUTE_NONNULL(1) ATTRIBUTE_RETURN_CHECK;<o:p></o:p> diff --git a/tests/viriscsitest.c b/tests/viriscsitest.c<o:p></o:p> index b5b0e20..40e4d10 100644<o:p></o:p> --- a/tests/viriscsitest.c<o:p></o:p> +++ b/tests/viriscsitest.c<o:p></o:p> @@ -145,8 +145,7 @@ testISCSIScanTargets(const void *data)<o:p></o:p> <o:p></o:p> virCommandSetDryRun(NULL, testIscsiadmCb, NULL);<o:p></o:p> <o:p></o:p> - if (virISCSIScanTargets(info->portal, NULL,<o:p></o:p> - &ntargets, &targets) < 0)<o:p></o:p> + if (virISCSIScanTargets(info->portal, &ntargets, &targets) < 0)<o:p></o:p> goto cleanup;<o:p></o:p> <o:p></o:p> if (info->nexpected != ntargets) {<o:p></o:p> -- <o:p></o:p> 2.5.5<o:p></o:p> <o:p> </o:p> <o:p> </o:p> <o:p> </o:p> ------------------------------<o:p></o:p> <o:p> </o:p> Message: 5<o:p></o:p> Date: Fri, 13 May 2016 23:36:17 +0200<o:p></o:p> From: Jiri Denemark <<a href="mailto:jdenemar@redhat.com">jdenemar@redhat.com</a>><o:p></o:p> To: <a href="mailto:libvir-list@redhat.com"> libvir-list@redhat.com</a><o:p></o:p> Subject: [libvirt] [PATCH] capabilities: Advertise cpuselection if<o:p></o:p> -cpu host is usable<o:p></o:p> Message-ID:<o:p></o:p> <<a href="mailto:a3394e6b8d0f28591147dd981909e9a1ae39f64d.1463175377.git.jdenemar@redhat.com">a3394e6b8d0f28591147dd981909e9a1ae39f64d.1463175377.git.jdenemar@redhat.com</a>><o:p></o:p> <o:p></o:p> <o:p> </o:p> When -cpu host is supported by a QEMU binary, a user can use<o:p></o:p> <cpu mode='host-passthrough'/> in domain XML even when libvirtd failed<o:p></o:p> to find a matching model for the host CPU. Let's make it obvious by<o:p></o:p> advertising <cpuselection/> guest capability whenever -cpu host is<o:p></o:p> supported.<o:p></o:p> <o:p> </o:p> Signed-off-by: Jiri Denemark <<a href="mailto:jdenemar@redhat.com">jdenemar@redhat.com</a>><o:p></o:p> ---<o:p></o:p> src/qemu/qemu_capabilities.c | 7 ++++---<o:p></o:p> tests/qemucaps2xmldata/all_1.6.0-1.xml | 1 +<o:p></o:p> tests/qemucaps2xmldata/nodisksnapshot_1.6.0-1.xml | 1 +<o:p></o:p> 3 files changed, 6 insertions(+), 3 deletions(-)<o:p></o:p> <o:p> </o:p> diff --git a/src/qemu/qemu_capabilities.c b/src/qemu/qemu_capabilities.c<o:p></o:p> index 1bddf43..fcfd1cd 100644<o:p></o:p> --- a/src/qemu/qemu_capabilities.c<o:p></o:p> +++ b/src/qemu/qemu_capabilities.c<o:p></o:p> @@ -928,9 +928,10 @@ virQEMUCapsInitGuestFromBinary(virCapsPtr caps,<o:p></o:p> machines = NULL;<o:p></o:p> nmachines = 0;<o:p></o:p> <o:p></o:p> - if (caps->host.cpu &&<o:p></o:p> - caps->host.cpu->model &&<o:p></o:p> - virQEMUCapsGetCPUDefinitions(qemubinCaps, NULL) > 0 &&<o:p></o:p> + if ((virQEMUCapsGet(qemubinCaps, QEMU_CAPS_CPU_HOST) ||<o:p></o:p> + (caps->host.cpu &&<o:p></o:p> + caps->host.cpu->model &&<o:p></o:p> + virQEMUCapsGetCPUDefinitions(qemubinCaps, NULL) > 0)) &&<o:p></o:p> !virCapabilitiesAddGuestFeature(guest, "cpuselection", true, false))<o:p></o:p> goto cleanup;<o:p></o:p> <o:p></o:p> diff --git a/tests/qemucaps2xmldata/all_1.6.0-1.xml b/tests/qemucaps2xmldata/all_1.6.0-1.xml<o:p></o:p> index 2489f49..84d6000 100644<o:p></o:p> --- a/tests/qemucaps2xmldata/all_1.6.0-1.xml<o:p></o:p> +++ b/tests/qemucaps2xmldata/all_1.6.0-1.xml<o:p></o:p> @@ -18,6 +18,7 @@<o:p></o:p> </domain><o:p></o:p> </arch><o:p></o:p> <features><o:p></o:p> + <cpuselection/><o:p></o:p> <deviceboot/><o:p></o:p> <disksnapshot default='on' toggle='no'/><o:p></o:p> <acpi default='on' toggle='yes'/><o:p></o:p> diff --git a/tests/qemucaps2xmldata/nodisksnapshot_1.6.0-1.xml b/tests/qemucaps2xmldata/nodisksnapshot_1.6.0-1.xml<o:p></o:p> index 281fab0..28762c2 100644<o:p></o:p> --- a/tests/qemucaps2xmldata/nodisksnapshot_1.6.0-1.xml<o:p></o:p> +++ b/tests/qemucaps2xmldata/nodisksnapshot_1.6.0-1.xml<o:p></o:p> @@ -18,6 +18,7 @@<o:p></o:p> </domain><o:p></o:p> </arch><o:p></o:p> <features><o:p></o:p> + <cpuselection/><o:p></o:p> <deviceboot/><o:p></o:p> <disksnapshot default='off' toggle='no'/><o:p></o:p> <acpi default='on' toggle='yes'/><o:p></o:p> -- <o:p></o:p> 2.8.2<o:p></o:p> <o:p> </o:p> <o:p> </o:p> <o:p> </o:p> ------------------------------<o:p></o:p> <o:p> </o:p> Message: 6<o:p></o:p> Date: Fri, 13 May 2016 16:57:30 -0600<o:p></o:p> From: Jim Fehlig <<a href="mailto:jfehlig@suse.com">jfehlig@suse.com</a>><o:p></o:p> To: Joao Martins <<a href="mailto:joao.m.martins@oracle.com">joao.m.martins@oracle.com</a>><o:p></o:p> Cc: <a href="mailto:libvirt-list@redhat.com"> libvirt-list@redhat.com</a><o:p></o:p> Subject: Re: [libvirt] [PATCH 0/2] libxl: support pvusb<o:p></o:p> Message-ID: <<a href="mailto:57365BDA.7090005@suse.com">57365BDA.7090005@suse.com</a>><o:p></o:p> Content-Type: text/plain; charset=windows-1252<o:p></o:p> <o:p> </o:p> On 05/13/2016 07:14 AM, Joao Martins wrote:<o:p></o:p> > On 05/12/2016 11:43 PM, Jim Fehlig wrote:<o:p></o:p> >> Chunyan Liu wrote:<o:p></o:p> >>> This patch series is to support pvusb in libxl driver.<o:p></o:p> >>><o:p> </o:p> >>> Chunyan Liu (2):<o:p></o:p> >>> libxl: support creating guest with USB hostdev<o:p></o:p> >>> libxl: support hotplug USB host device<o:p></o:p> >> This series should also include support for converting domXML USB config to/from<o:p></o:p> >> xl.cfg(5) in src/xenconfig/xen_xl.c, along with a test.<o:p></o:p> >><o:p> </o:p> >> FYI, the series fails to build with Xen < 4.7, where LIBXL_HAVE_PVUSB is not<o:p></o:p> >> defined :-).<o:p></o:p> > Probably a nice follow-up after this series is in, would be to add the USB<o:p></o:p> > subsys type to domain capabilities (that you address in the OVMF series)<o:p></o:p> <o:p> </o:p> Yes, good point. I'll add it to that series if this one makes it in first.<o:p></o:p> <o:p> </o:p> Speaking of the OVMF series, I've fixed a few things after more testing and will<o:p></o:p> rebase and resend a V2. Thanks in advance if either of you have time to look at it.<o:p></o:p> <o:p> </o:p> Regards,<o:p></o:p> Jim<o:p></o:p> <o:p> </o:p> <o:p> </o:p> <o:p> </o:p> ------------------------------<o:p></o:p> <o:p> </o:p> Message: 7<o:p></o:p> Date: Fri, 13 May 2016 17:47:09 -0600<o:p></o:p> From: Jim Fehlig <<a href="mailto:jfehlig@suse.com">jfehlig@suse.com</a>><o:p></o:p> To: Chunyan Liu <<a href="mailto:cyliu@suse.com">cyliu@suse.com</a>>, <a href="mailto:libvirt-list@redhat.com">libvirt-list@redhat.com</a><o:p></o:p> Subject: Re: [libvirt] [PATCH] libxl: add .domainInterfaceAddresses<o:p></o:p> Message-ID: <<a href="mailto:5736677D.8030209@suse.com">5736677D.8030209@suse.com</a>><o:p></o:p> Content-Type: text/plain; charset=windows-1252<o:p></o:p> <o:p> </o:p> On 05/13/2016 12:21 AM, Chunyan Liu wrote:<o:p></o:p> > Add .domainInterfaceAddresses so that user can have a way to<o:p></o:p> > get domain interface address by 'virsh domifaddr'. Currently<o:p></o:p> > it only supports '--source lease'.<o:p></o:p> ><o:p> </o:p> > Signed-off: Chunyan Liu <<a href="mailto:cyliu@suse.com">cyliu@suse.com</a>><o:p></o:p> > ---<o:p></o:p> > src/libxl/libxl_driver.c | 140 +++++++++++++++++++++++++++++++++++++++++++++++<o:p></o:p> > 1 file changed, 140 insertions(+)<o:p></o:p> ><o:p> </o:p> > diff --git a/src/libxl/libxl_driver.c b/src/libxl/libxl_driver.c<o:p></o:p> > index 062d6f8..f2bd6fa 100644<o:p></o:p> > --- a/src/libxl/libxl_driver.c<o:p></o:p> > +++ b/src/libxl/libxl_driver.c<o:p></o:p> > @@ -5425,6 +5425,145 @@ static int libxlNodeGetSecurityModel(virConnectPtr conn,<o:p></o:p> > return 0;<o:p></o:p> > }<o:p></o:p> > <o:p></o:p> > +static int<o:p></o:p> > +libxlGetDHCPInterfaces(virDomainPtr dom,<o:p></o:p> > + virDomainObjPtr vm,<o:p></o:p> > + virDomainInterfacePtr **ifaces)<o:p></o:p> > +{<o:p></o:p> > + int rv = -1;<o:p></o:p> > + int n_leases = 0;<o:p></o:p> > + size_t i, j;<o:p></o:p> > + size_t ifaces_count = 0;<o:p></o:p> > + virNetworkPtr network = NULL;<o:p></o:p> > + char macaddr[VIR_MAC_STRING_BUFLEN];<o:p></o:p> > + virDomainInterfacePtr iface = NULL;<o:p></o:p> > + virNetworkDHCPLeasePtr *leases = NULL;<o:p></o:p> > + virDomainInterfacePtr *ifaces_ret = NULL;<o:p></o:p> > +<o:p></o:p> > + if (!dom->conn->networkDriver ||<o:p></o:p> > + !dom->conn->networkDriver->networkGetDHCPLeases) {<o:p></o:p> > + virReportError(VIR_ERR_INTERNAL_ERROR, "%s",<o:p></o:p> > + _("Network driver does not support DHCP lease query"));<o:p></o:p> > + return -1;<o:p></o:p> > + }<o:p></o:p> > +<o:p></o:p> > + for (i = 0; i < vm->def->nnets; i++) {<o:p></o:p> > + if (vm->def->nets[i]->type != VIR_DOMAIN_NET_TYPE_NETWORK)<o:p></o:p> > + continue;<o:p></o:p> > +<o:p></o:p> > + virMacAddrFormat(&(vm->def->nets[i]->mac), macaddr);<o:p></o:p> > + virObjectUnref(network);<o:p></o:p> > + network = virNetworkLookupByName(dom->conn,<o:p></o:p> > + vm->def->nets[i]->data.network.name);<o:p></o:p> > +<o:p></o:p> > + if ((n_leases = virNetworkGetDHCPLeases(network, macaddr,<o:p></o:p> > + &leases, 0)) < 0)<o:p></o:p> > + goto error;<o:p></o:p> > +<o:p></o:p> > + if (n_leases) {<o:p></o:p> > + if (VIR_EXPAND_N(ifaces_ret, ifaces_count, 1) < 0)<o:p></o:p> > + goto error;<o:p></o:p> > +<o:p></o:p> > + if (VIR_ALLOC(ifaces_ret[ifaces_count - 1]) < 0)<o:p></o:p> > + goto error;<o:p></o:p> > +<o:p></o:p> > + iface = ifaces_ret[ifaces_count - 1];<o:p></o:p> > + /* Assuming each lease corresponds to a separate IP */<o:p></o:p> > + iface->naddrs = n_leases;<o:p></o:p> > +<o:p></o:p> > + if (VIR_ALLOC_N(iface->addrs, iface->naddrs) < 0)<o:p></o:p> > + goto error;<o:p></o:p> > +<o:p></o:p> > + if (VIR_STRDUP(iface->name, vm->def->nets[i]->ifname) < 0)<o:p></o:p> > + goto cleanup;<o:p></o:p> > +<o:p></o:p> > + if (VIR_STRDUP(iface->hwaddr, macaddr) < 0)<o:p></o:p> > + goto cleanup;<o:p></o:p> > + }<o:p></o:p> > +<o:p></o:p> > + for (j = 0; j < n_leases; j++) {<o:p></o:p> > + virNetworkDHCPLeasePtr lease = leases[j];<o:p></o:p> > + virDomainIPAddressPtr ip_addr = &iface->addrs[j];<o:p></o:p> > +<o:p></o:p> > + if (VIR_STRDUP(ip_addr->addr, lease->ipaddr) < 0)<o:p></o:p> > + goto cleanup;<o:p></o:p> > +<o:p></o:p> > + ip_addr->type = lease->type;<o:p></o:p> > + ip_addr->prefix = lease->prefix;<o:p></o:p> > + }<o:p></o:p> > +<o:p></o:p> > + for (j = 0; j < n_leases; j++)<o:p></o:p> > + virNetworkDHCPLeaseFree(leases[j]);<o:p></o:p> > +<o:p></o:p> > + VIR_FREE(leases);<o:p></o:p> > + }<o:p></o:p> > +<o:p></o:p> > + *ifaces = ifaces_ret;<o:p></o:p> > + ifaces_ret = NULL;<o:p></o:p> > + rv = ifaces_count;<o:p></o:p> > +<o:p></o:p> > + cleanup:<o:p></o:p> > + virObjectUnref(network);<o:p></o:p> > + if (leases) {<o:p></o:p> > + for (i = 0; i < n_leases; i++)<o:p></o:p> > + virNetworkDHCPLeaseFree(leases[i]);<o:p></o:p> > + }<o:p></o:p> > + VIR_FREE(leases);<o:p></o:p> > +<o:p></o:p> > + return rv;<o:p></o:p> > +<o:p></o:p> > + error:<o:p></o:p> > + if (ifaces_ret) {<o:p></o:p> > + for (i = 0; i < ifaces_count; i++)<o:p></o:p> > + virDomainInterfaceFree(ifaces_ret[i]);<o:p></o:p> > + }<o:p></o:p> > + VIR_FREE(ifaces_ret);<o:p></o:p> > +<o:p></o:p> > + goto cleanup;<o:p></o:p> > +}<o:p></o:p> <o:p> </o:p> It's unfortunate this is a copy-paste from the qemu driver. The code is not<o:p></o:p> trivial and any bug fixes in one copy could be missed in the other. A lot of the<o:p></o:p> function is domain related, so probably can't be abstracted further to the<o:p></o:p> network code. Have you considered approaches that allow the drivers to share<o:p></o:p> this code?<o:p></o:p> <o:p> </o:p> Regards,<o:p></o:p> Jim<o:p></o:p> <o:p> </o:p> <o:p> </o:p> <o:p> </o:p> ------------------------------<o:p></o:p> <o:p> </o:p> --<o:p></o:p> libvir-list mailing list<o:p></o:p> <a href="mailto:libvir-list@redhat.com">libvir-list@redhat.com</a><o:p></o:p> <a href="https://www.redhat.com/mailman/listinfo/libvir-list">https://www.redhat.com/mailman/listinfo/libvir-list</a><o:p></o:p> <o:p> </o:p> End of libvir-list Digest, Vol 126, Issue 182<o:p></o:p> *********************************************<o:p></o:p> </div> ******************************************************************************************************************************** The information in this email is confidential and may be legally privileged. If you have received this email in error or are not the intended recipient, please immediately notify the sender and delete this message from your computer. Any use, distribution, or copying of this email other than by the intended recipient is strictly prohibited. All messages sent to and from us may be monitored to ensure compliance with internal policies and to protect our business. Emails are not secure and cannot be guaranteed to be error free as they can be intercepted, amended, lost or destroyed, or contain viruses. Anyone who communicates with us by email is taken to accept these risks. 收发邮件者请注意： 本邮件含保密信息，若误收本邮件，请务必通知发送人并直接删去，不得使用、传播或复制本邮件。 进出邮件均受到本公司合规监控。邮件可能发生被截留、被修改、丢失、被破坏或包含计算机病毒等不安全情况。 ********************************************************************************************************************************</body> </html>