[libvirt] [PATCH v4 1/3] Add per-guest S3/S4 state configuration

[Eric Blake]

On 08/31/2012 07:59 AM, Martin Kletzander wrote:
> There is a new <pm/> element implemented that can control what ACPI
> sleeping states will be advertised by BIOS and allowed to be switched
> to by libvirt. The default keeps defaults on hypervisor, otherwise
> forces chosen setting.
> The documentation of the pm element is added as well.
> ---

> +<pre>
> +  ...
> +  <pm>
> +    <suspend-to-disk enabled='no'/>
> +    <suspend-to-ram enabled='yes'/>

'no' and 'yes' here...

> +  </pm>
> +  ...</pre>
> +
> +    <dl>
> +      <dt><code>pm</code></dt>
> +      <dd>These elements enable ('on') or disable ('off') BIOS support

'on' and 'off' here...

>    <!--
> +      Control ACPI sleep states (dis)allowed for the domain
> +      For each of the states the following rules apply:
> +      on: the state will be forcefully enabled
> +      off: the state will be forcefully disabled
> +      not specified: hypervisor will be left to decide its defaults

> +  <define name="suspendChoices">
> +    <interleave>
> +      <optional>
> +        <attribute name="enabled">
> +          <choice>
> +            <value>yes</value>
> +            <value>no</value>

...back to 'yes' and 'no' here.  Fix the .html.in version to use the
right naming.

> +          </choice>
> +        </attribute>
> +      </optional>
> +    </interleave>

The <interleave> layer is not necessary here (you only have one
sub-entry; furthermore, the sub-entry is an <attribute> which is already
auto-interleaved; only <element> entries need interleaving).

ACK with those tweaks.

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