Re: [PATCHv3 1/3] schema: Fix guest timer specification schema according to the docs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 02/07/2014 07:21 AM, Peter Krempa wrote:
> According to the documentation describing various tunables for domain
> timers not all the fields are supported by all the driver types. Express
> these in the RNG:
> 
> - rtc, platform: Only these support the "track" attribute.
> - tsc: only one to support "frequency" and "mode" attributes
> - hpet, pit: tickpolicy/catchup attribute/element
> - kvmclock: no extra attributes are supported
> ---
>  docs/schemas/domaincommon.rng | 153 +++++++++++++++++++++++++-----------------
>  1 file changed, 93 insertions(+), 60 deletions(-)
> 
> +        <optional>
> +          <element name="catchup">
> +            <optional>
> +              <attribute name="threshold">
> +                <ref name="unsignedInt"/>
> +              </attribute>
> +              <attribute name="slew">
> +                <ref name="unsignedInt"/>
> +              </attribute>
> +              <attribute name="limit">
> +                <ref name="unsignedInt"/>
> +              </attribute>

Are all three attributes mandatory as a group?  Or is each attribute
optional in isolation?  The C code says the latter, which means this
part has to be written:

<element name='catchup'>
  <optional>
    <attribute name='threshold'>...
  </optional>
  <optional>
    <attribute name='slew'>...

and so on.

ACK with that fix; it might also be nice to add a testsuite addition
that proves we handle 'threshold' without also having to specify 'slew'
and 'limit'.

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]