On 02/03/2014 08:19 AM, Jiri Denemark wrote: > On Mon, Jan 13, 2014 at 14:28:11 +0800, mrhines@xxxxxxxxxxxxxxxxxx wrote: >> From: "Michael R. Hines" <mrhines@xxxxxxxxxx> >> >> The switch from x-rdma => rdma has not yet happened, >> but at least we can review the patch until it goes >> through on qemu-devel. > > The paragraph above would better fit after "---" below so that it > disappears once this patch gets applied as the statement won't be valid > anymore at that time. > >> USAGE: $ virsh migrate --live --migrateuri x-rdma:hostname domain qemu+ssh://hostname/system > > s/x-rdma/rdma/ and I believe we should use rdma://hostname as the URI > >> Full documenation of the feature: http://wiki.qemu.org/Features/RDMALiveMigration s/documenation/documentation/ >> @@ -2561,6 +2570,10 @@ virQEMUCapsInitQMPMonitor(virQEMUCapsPtr qemuCaps, >> if (qemuCaps->version >= 1006000) >> virQEMUCapsSet(qemuCaps, QEMU_CAPS_DEVICE_VIDEO_PRIMARY); >> >> + if (qemuCaps->version >= 2000000) >> + virQEMUCapsSet(qemuCaps, QEMU_CAPS_MIGRATE_QEMU_RDMA); >> + >> + > > And here we need a better check for rdma migration. What if someone > compiles QEMU without RDMA support? Better than hard-coding it to a version string is to probe the results of query-migrate-capabilities and only setting the capability if the resulting list includes rdma-pin-all, as that will serve as a reliable witness of qemu being new enough to support rdma without an x- prefix. -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org
Attachment:
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list