On 06/17/2014 04:25 PM, Eric Blake wrote:
> On 06/17/2014 03:53 PM, Jeff Cody wrote:
>> Now that active layer block-commit is supported, the 'top' argument
>> no longer needs to be mandatory.
>>
>> Change it to optional, with the default being the active layer in the
>> device chain.
>>
>> Reviewed-by: Eric Blake <ebl...@redhat.com>
>> Reviewed-by: Benoit Canet <ben...@irqsave.net>
>> Signed-off-by: Jeff Cody <jc...@redhat.com>
>> ---
> 
> As mentioned elsewhere, this is the patch that changed from v5.  But so
> there's no confusion, I've gone back over this one, and my R-b still stands.
> 
>> +    /* Important Note:
>> +     *  libvirt relies on the DeviceNotFound error class in order to probe 
>> for
>> +     *  live commit feature versions; for this to work, we must make sure to
>> +     *  perform the device lookup before any generic errors that may occur 
>> in a
>> +     *  scenario in which all optional arguments are omitted. */
>>      bs = bdrv_find(device);
>>      if (!bs) {
>>          error_set(errp, QERR_DEVICE_NOT_FOUND, device);
> 
> Thanks for adding the comment :)
> 

Should we split this into a different series, since it is
non-controversial (unrelated to the rest of the series' impact on
child-node op-blockers), and useful for libvirt to turn on active
commit?  The rest of the series lets libvirt turn on relative backing
chain work, but we might as well get the easy part into the tree now.

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to