summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMartin Kletzander <mkletzan@redhat.com>2012-09-18 12:30:52 +0200
committerMartin Kletzander <mkletzan@redhat.com>2012-09-20 16:41:01 +0200
commitc33a922faad1de363aa4212edc1a329bd47e52b0 (patch)
tree8645408e76936342771a72bef7ce7edfdd0edf03
parentsecurity: Don't ignore errors when parsing DAC security labels (diff)
downloadlibvirt-c33a922faad1de363aa4212edc1a329bd47e52b0.tar.gz
libvirt-c33a922faad1de363aa4212edc1a329bd47e52b0.tar.bz2
libvirt-c33a922faad1de363aa4212edc1a329bd47e52b0.zip
Add support for reboot-timeout
Whenever the guest machine fails to boot, new parameter (reboot-timeout) controls whether it should reboot and after how many ms it should do so. Docs included.
-rw-r--r--docs/formatdomain.html.in11
-rw-r--r--docs/schemas/domaincommon.rng24
-rw-r--r--src/conf/domain_conf.c33
-rw-r--r--src/conf/domain_conf.h3
4 files changed, 56 insertions, 15 deletions
diff --git a/docs/formatdomain.html.in b/docs/formatdomain.html.in
index 51f897cd9..a403ba8c5 100644
--- a/docs/formatdomain.html.in
+++ b/docs/formatdomain.html.in
@@ -105,7 +105,7 @@
&lt;boot dev='cdrom'/&gt;
&lt;bootmenu enable='yes'/&gt;
&lt;smbios mode='sysinfo'/&gt;
- &lt;bios useserial='yes'/&gt;
+ &lt;bios useserial='yes' rebootTimeout='0'/&gt;
&lt;/os&gt;
...</pre>
@@ -175,8 +175,13 @@
Serial Graphics Adapter which allows users to see BIOS messages
on a serial port. Therefore, one needs to have
<a href="#elementCharSerial">serial port</a> defined.
- <span class="since">Since 0.9.4</span>
- </dd>
+ <span class="since">Since 0.9.4</span>.
+ <span class="since">Since 0.10.2 (QEMU only)</span> there is
+ another attribute, <code>rebootTimeout</code> that controls
+ whether and after how long the guest should start booting
+ again in case the boot fails (according to BIOS). The value is
+ in milliseconds with maximum of <code>65535</code> and special
+ value <code>-1</code> disables the reboot.
</dl>
<h4><a name="elementsOSBootloader">Host bootloader</a></h4>
diff --git a/docs/schemas/domaincommon.rng b/docs/schemas/domaincommon.rng
index aafb10c57..afa6bbd19 100644
--- a/docs/schemas/domaincommon.rng
+++ b/docs/schemas/domaincommon.rng
@@ -3190,12 +3190,19 @@
<define name="bios">
<element name="bios">
- <attribute name="useserial">
- <choice>
- <value>yes</value>
- <value>no</value>
- </choice>
- </attribute>
+ <optional>
+ <attribute name="useserial">
+ <choice>
+ <value>yes</value>
+ <value>no</value>
+ </choice>
+ </attribute>
+ </optional>
+ <optional>
+ <attribute name="rebootTimeout">
+ <ref name="rebootTimeoutDelay"/>
+ </attribute>
+ </optional>
</element>
</define>
@@ -3469,6 +3476,11 @@
<param name='minInclusive'>-1</param>
</data>
</define>
+ <define name="rebootTimeoutDelay">
+ <data type="short">
+ <param name="minInclusive">-1</param>
+ </data>
+ </define>
<define name="PortNumber">
<data type="short">
<param name="minInclusive">-1</param>
diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index 35814fb02..d6f2ebfe2 100644
--- a/src/conf/domain_conf.c
+++ b/src/conf/domain_conf.c
@@ -8136,7 +8136,7 @@ virDomainDefParseBootXML(xmlXPathContextPtr ctxt,
{
xmlNodePtr *nodes = NULL;
int i, n;
- char *bootstr;
+ char *bootstr, *tmp;
char *useserial = NULL;
int ret = -1;
unsigned long deviceBoot, serialPorts;
@@ -8214,10 +8214,25 @@ virDomainDefParseBootXML(xmlXPathContextPtr ctxt,
}
}
+ tmp = virXPathString("string(./os/bios[1]/@rebootTimeout)", ctxt);
+ if (tmp) {
+ /* that was really just for the check if it is there */
+
+ if (virStrToLong_i(tmp, NULL, 0, &def->os.bios.rt_delay) < 0 ||
+ def->os.bios.rt_delay < -1 || def->os.bios.rt_delay > 65535) {
+ virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
+ _("invalid value for rebootTimeout, "
+ "must be in range [-1,65535]"));
+ goto cleanup;
+ }
+ def->os.bios.rt_set = true;
+ }
+
*bootCount = deviceBoot;
ret = 0;
cleanup:
+ VIR_FREE(tmp);
VIR_FREE(useserial);
VIR_FREE(nodes);
return ret;
@@ -13494,11 +13509,17 @@ virDomainDefFormatInternal(virDomainDefPtr def,
virBufferAsprintf(buf, " <bootmenu enable='%s'/>\n", enabled);
}
- if (def->os.bios.useserial) {
- const char *useserial = (def->os.bios.useserial ==
- VIR_DOMAIN_BIOS_USESERIAL_YES ? "yes"
- : "no");
- virBufferAsprintf(buf, " <bios useserial='%s'/>\n", useserial);
+ if (def->os.bios.useserial || def->os.bios.rt_set) {
+ virBufferAddLit(buf, " <bios");
+ if (def->os.bios.useserial)
+ virBufferAsprintf(buf, " useserial='%s'",
+ (def->os.bios.useserial ==
+ VIR_DOMAIN_BIOS_USESERIAL_YES ? "yes"
+ : "no"));
+ if (def->os.bios.rt_set)
+ virBufferAsprintf(buf, " rebootTimeout='%d'", def->os.bios.rt_delay);
+
+ virBufferAddLit(buf, "/>\n");
}
}
diff --git a/src/conf/domain_conf.h b/src/conf/domain_conf.h
index 510406a63..d719d57da 100644
--- a/src/conf/domain_conf.h
+++ b/src/conf/domain_conf.h
@@ -1420,6 +1420,9 @@ typedef struct _virDomainBIOSDef virDomainBIOSDef;
typedef virDomainBIOSDef *virDomainBIOSDefPtr;
struct _virDomainBIOSDef {
int useserial;
+ /* reboot-timeout parameters */
+ bool rt_set;
+ int rt_delay;
};
/* Operating system configuration data & machine / arch */