Re: [PATCH v4 2/5] blockjob: turn on qemu capability bit for active commit

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

 



On 06/24/2014 01:15 AM, Peter Krempa wrote:
> On 06/24/14 01:30, Eric Blake wrote:
>> Use the probing functionality added in the last patch to turn on
>> a capability bit when active commit is present, and gate active
>> commit on that capability.
>>
>> For my own reference: the difference between BLOCKJOB_SYNC and
>> BLOCKJOB_ASYNC is whether qemu generated an event at the
>> conclusion of blockpull; basically, RHEL 6.2 was the only release
>> of qemu that has the sync semantics and lacks the event.  RHEL
>> 6.3 added blockcopy, but also picked up on the upstream style
>> of qemu generating events.  As no one is likely to backport
>> active commit to RHEL 6.2, it's safe for blockcommit to always
>> require async blockjob support.
>>
>> Modifying qemucapabilitiestest is painful; the .replies files would
>> be so much easier if they had comments correlating which command
>> generated the given reply.  Maybe I'll fix that up later...
>>

> 
> ACK,

Pushed.

-- 
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]