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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]