[libvirt] [RFC PATCH v2 2/3] qemu: RDMA migration support using 'rdma' URI

Eric Blake eblake at redhat.com
Mon Feb 3 15:44:14 UTC 2014


On 02/03/2014 08:19 AM, Jiri Denemark wrote:
> On Mon, Jan 13, 2014 at 14:28:11 +0800, mrhines at linux.vnet.ibm.com wrote:
>> From: "Michael R. Hines" <mrhines at us.ibm.com>
>>
>> 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

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 604 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20140203/d5e7389b/attachment-0001.sig>


More information about the libvir-list mailing list