[libvirt] [PATCH 1/3] block rebase: add new API virDomainBlockRebase

[Eric Blake]

On 02/01/2012 12:40 PM, Laine Stump wrote:
> On 02/01/2012 12:05 AM, Eric Blake wrote:
>> Qemu is adding the ability to do a partial rebase.  That is, given:
>>
>> base<- intermediate<- current
>>
>> virDomainBlockPull will produce:
>>
>> current
>>
>> but qemu now has the ability to leave base in the chain, to produce:
>>
>> base<- current
>>

>> +int virDomainBlockRebase(virDomainPtr dom, const char *disk,
>> +                         const char *base, unsigned long bandwidth,
>> +                         unsigned int flags)
>> +{
>> +    virConnectPtr conn;
>> +
>> +    VIR_DOMAIN_DEBUG(dom, "disk=%p, base=%s bandwidth=%lu, flags=%x",
> 
> disk is just a character string, isn't it? If so, why not use %s instead
> of %p?

Copy-and-paste from virDomainBlockPull, so I'll fix up both places
before pushing.

> 
> Otherwise, this is all pretty straightforward. ACK.

Thanks for the review.

-- 
Eric Blake   eblake at 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: 620 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20120201/eff6cd48/attachment-0001.sig>