[libvirt] [PATCH 1/3] block rebase: add new API virDomainBlockRebase
Eric Blake
eblake at redhat.com
Wed Feb 1 20:07:32 UTC 2012
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>
More information about the libvir-list
mailing list