[libvirt] [PATCH v2 4/7] dumpxml: let blockinfo reuse virStorageSource
Eric Blake
eblake at redhat.com
Tue Nov 18 16:35:54 UTC 2014
On 11/18/2014 08:00 AM, Adam Litke wrote:
> On 18/11/14 06:31 -0700, Eric Blake wrote:
>> In a future patch, the implementation of VIR_DOMAIN_XML_BLOCK_INFO
>> will use information stored in virStorageSource. In order to
>> maximize code reuse, it is easiest if BlockInfo code also uses the
>> same location for information.
>>
>> +++ b/src/util/virstoragefile.h
>> @@ -252,8 +252,9 @@ struct _virStorageSource {
>>
>> virStoragePermsPtr perms;
>> virStorageTimestampsPtr timestamps;
>> - unsigned long long allocation; /* in bytes, 0 if unknown */
>> unsigned long long capacity; /* in bytes, 0 if unknown */
>> + unsigned long long allocation; /* in bytes, 0 if unknown */
>
> (Pedantic and superficial) allocation dropped and re-added. Maybe you
> prefer it to appear after capacity in light of physical now being
> present?
Just reordering fields so that they come in the same order as their
public struct virDomainBlockInfo counterpart in libvirt-domain.h.
Order doesn't strictly matter, if you want me to avoid the churn.
--
Eric Blake eblake 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: 539 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20141118/0c0aa426/attachment-0001.sig>
More information about the libvir-list
mailing list