[libvirt] [PATCH v3 1/4] add new virDomainMemoryDump API

Eric Blake eblake at redhat.com
Thu Feb 27 15:47:23 UTC 2014


On 02/27/2014 07:43 AM, Daniel P. Berrange wrote:

>> +/* Domain memory dump's format */

/* */ comments don't make it through to the html doc pages.  Here, you
want something like:

/**
 * virDomainCoreDumpFormat:
 * Values for specifying different formats of domain core dumps.
 */

>> +typedef enum {
>> +    VIR_MEMORY_DUMP_COMPRESS_ZLIB   = 1, /* dump guest memory in
>> +                                            kdump-compressed format, with
>> +                                            zlib-compressed */
>> +    VIR_MEMORY_DUMP_COMPRESS_LZO    = 2, /* dump guest memory in
>> +                                            kdump-compressed format, with
>> +                                            lzo-compressed */
>> +    VIR_MEMORY_DUMP_COMPRESS_SNAPPY = 3, /* dump guest memory in
>> +                                            kdump-compressed format, with
>> +                                            snappy-compressed */
>> +} virMemoryDumpFormat;
> 

Missing a *_LAST enum value, under proper #ifdef.

> s/virMemoryDumpFormat/virDomainCoreDumpFormat/
> 
> No need to assign values to each entry here since this
> is a plain enum, not bitflags.
> 
> Also  s/MEMORY_COMPRESS/DOMAIN_CORE_FORMAT/ for every member
> 
> And add  VIR_MEMORY_DUMP_FORMAT_RAW  for the non-compressed
> format, which probably ought to be the default (ie first).
> 

-- 
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: 604 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20140227/e0c4f873/attachment-0001.sig>


More information about the libvir-list mailing list