[libvirt] [PATCH v2 1/3] Add support for limiting guest coredump

[Martin Kletzander]

On 09/20/2012 11:32 AM, Daniel P. Berrange wrote:
> On Thu, Sep 20, 2012 at 10:58:13AM +0200, Martin Kletzander wrote:
>> Sometimes when guest machine crashes, coredump can get huge due to the
>> guest memory. This can be limited using madvise(2) system call and is
>> being used in QEMU hypervisor. This patch adds an option for configuring
>> that in the domain XML and related documentation.
>> ---
>>  docs/formatdomain.html.in     | 12 +++++++++---
>>  docs/schemas/domaincommon.rng |  8 ++++++++
>>  src/conf/domain_conf.c        | 25 ++++++++++++++++++++++++-
>>  src/conf/domain_conf.h        | 10 ++++++++++
>>  src/libvirt_private.syms      |  2 ++
>>  5 files changed, 53 insertions(+), 4 deletions(-)
>>
>> diff --git a/docs/formatdomain.html.in b/docs/formatdomain.html.in
>> index d021837..210ebe0 100644
>> --- a/docs/formatdomain.html.in
>> +++ b/docs/formatdomain.html.in
>> @@ -515,9 +515,15 @@
>>          However, the value will be rounded up to the nearest kibibyte
>>          by libvirt, and may be further rounded to the granularity
>>          supported by the hypervisor.  Some hypervisors also enforce a
>> -        minimum, such as
>> -        4000KiB. <span class='since'><code>unit</code> since
>> -        0.9.11</span></dd>
>> +        minimum, such as 4000KiB.
>> +
>> +        In the case of crash, optional attribute <code>dump-core</code>
>> +        can be used to control whether the guest memory should be
>> +        included in the generated coredump or not (values "on", "off").
>> +
>> +        <span class='since'><code>unit</code> since 0.9.11</span>,
>> +        <span class='since'><code>dump-core</code> since 0.10.2
>> +        (QEMU only)</span></dd>
>>        <dt><code>currentMemory</code></dt>
>>        <dd>The actual allocation of memory for the guest. This value can
>>          be less than the maximum allocation, to allow for ballooning
>> diff --git a/docs/schemas/domaincommon.rng b/docs/schemas/domaincommon.rng
>> index ed25f58..bf6afbb 100644
>> --- a/docs/schemas/domaincommon.rng
>> +++ b/docs/schemas/domaincommon.rng
>> @@ -470,6 +470,14 @@
>>      <interleave>
>>        <element name="memory">
>>          <ref name='scaledInteger'/>
>> +        <optional>
>> +          <attribute name="dump-core">
> 
> As a general rule we have avoided use of '-' in XML element/attribute
> names, so this ought to be 'dumpCore' IMHO
> 

I didn't realize that, OK. It's fixed, should I send v3 for review
because of that?

Martin