Re: [libvirt] [PATCH 1/3] docs, conf: add support for bootmenu timeout

On 08/22/2014 09:09 AM, Martin Kletzander wrote: > Signed-off-by: Martin Kletzander <mkletzan(a)redhat.com&gt; > --- > docs/formatdomain.html.in | 7 +++++- > docs/schemas/domaincommon.rng | 5 ++++ > src/conf/domain_conf.c | 21 ++++++++++++++-- > src/conf/domain_conf.h | 2 ++ > ...qemuxml2argv-boot-menu-disable-with-timeout.xml | 29 ++++++++++++++++++++++ > ...2argv-boot-menu-enable-with-timeout-invalid.xml | 29 ++++++++++++++++++++++ > .../qemuxml2argv-boot-menu-enable-with-timeout.xml | 29 ++++++++++++++++++++++ > tests/qemuxml2argvtest.c | 1 + > ...muxml2xmlout-boot-menu-disable-with-timeout.xml | 29 ++++++++++++++++++++++ > tests/qemuxml2xmltest.c | 2 ++ > 10 files changed, 151 insertions(+), 3 deletions(-) > create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-boot-menu-disable-with-timeout.xml > create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-boot-menu-enable-with-timeout-invalid.xml > create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-boot-menu-enable-with-timeout.xml > create mode 100644 tests/qemuxml2xmloutdata/qemuxml2xmlout-boot-menu-disable-with-timeout.xml > @@ -158,6 +158,11 @@ > startup. The <code>enable</code> attribute can be either "yes" or "no". > If not specified, the hypervisor default is used. <span class="since"> > Since 0.8.3</span> > + Additional attribute <code>timeout</code> takes the number of milliseconds > + the boot menu should wait until it times out. Allowed values are numbers > + in range [0, 65535] inclusive and it is valid if and only if the previous > + <code>enable</code> is set to "yes". You know, it is possible to have RelaxNG enforce this restriction: