Add new function to manage adding the watchdog device options to the
command line removing that task from the mainline qemuBuildCommandLine.
Also since qemuBuildWatchdogDevStr was only local here, make it static as
well as modifying the const virDomainDef.
Signed-off-by: John Ferlan <jferlan(a)redhat.com>
---
src/qemu/qemu_command.c | 93 ++++++++++++++++++++++++++++---------------------
src/qemu/qemu_command.h | 4 ---
2 files changed, 53 insertions(+), 44 deletions(-)
diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c
index a89b62d..af09e1e 100644
--- a/src/qemu/qemu_command.c
+++ b/src/qemu/qemu_command.c
@@ -3274,8 +3274,8 @@ qemuBuildHostNetStr(virDomainNetDefPtr net,
}
-char *
-qemuBuildWatchdogDevStr(virDomainDefPtr def,
+static char *
+qemuBuildWatchdogDevStr(const virDomainDef *def,
virDomainWatchdogDefPtr dev,
virQEMUCapsPtr qemuCaps)
{
@@ -3303,6 +3303,55 @@ qemuBuildWatchdogDevStr(virDomainDefPtr def,
}
+static int
+qemuBuildWatchdogCommandLine(virCommandPtr cmd,
+ const virDomainDef *def,
+ virQEMUCapsPtr qemuCaps)
+{
+ virDomainWatchdogDefPtr watchdog = def->watchdog;
+ char *optstr;
+ const char *action;
+
+ if (!def->watchdog)
+ return 0;
+
+ if (virQEMUCapsGet(qemuCaps, QEMU_CAPS_DEVICE)) {
+ virCommandAddArg(cmd, "-device");
+
+ optstr = qemuBuildWatchdogDevStr(def, watchdog, qemuCaps);
+ if (!optstr)
+ return -1;
+ } else {
+ virCommandAddArg(cmd, "-watchdog");
+
+ const char *model = virDomainWatchdogModelTypeToString(watchdog->model);
+ if (!model) {
+ virReportError(VIR_ERR_INTERNAL_ERROR,
+ "%s", _("missing watchdog model"));
+ return -1;
+ }
+
+ if (VIR_STRDUP(optstr, model) < 0)
+ return -1;
+ }
+ virCommandAddArg(cmd, optstr);
+ VIR_FREE(optstr);
+
+ if (watchdog->action == VIR_DOMAIN_WATCHDOG_ACTION_DUMP)
+ watchdog->action = VIR_DOMAIN_WATCHDOG_ACTION_PAUSE;
+
+ action = virDomainWatchdogActionTypeToString(watchdog->action);
+ if (!action) {
+ virReportError(VIR_ERR_INTERNAL_ERROR,
+ "%s", _("invalid watchdog action"));
+ return -1;
+ }
+ virCommandAddArgList(cmd, "-watchdog-action", action, NULL);
+
+ return 0;
+}
+
+
char *
qemuBuildMemballoonDevStr(virDomainDefPtr def,
virDomainMemballoonDefPtr dev,
@@ -8710,44 +8759,8 @@ qemuBuildCommandLine(virConnectPtr conn,
if (qemuBuildSoundCommandLine(cmd, def, qemuCaps) < 0)
goto error;
- /* Add watchdog hardware */
- if (def->watchdog) {
- virDomainWatchdogDefPtr watchdog = def->watchdog;
- char *optstr;
-
- if (virQEMUCapsGet(qemuCaps, QEMU_CAPS_DEVICE)) {
- virCommandAddArg(cmd, "-device");
-
- optstr = qemuBuildWatchdogDevStr(def, watchdog, qemuCaps);
- if (!optstr)
- goto error;
- } else {
- virCommandAddArg(cmd, "-watchdog");
-
- const char *model = virDomainWatchdogModelTypeToString(watchdog->model);
- if (!model) {
- virReportError(VIR_ERR_INTERNAL_ERROR,
- "%s", _("missing watchdog model"));
- goto error;
- }
-
- if (VIR_STRDUP(optstr, model) < 0)
- goto error;
- }
- virCommandAddArg(cmd, optstr);
- VIR_FREE(optstr);
-
- int act = watchdog->action;
- if (act == VIR_DOMAIN_WATCHDOG_ACTION_DUMP)
- act = VIR_DOMAIN_WATCHDOG_ACTION_PAUSE;
- const char *action = virDomainWatchdogActionTypeToString(act);
- if (!action) {
- virReportError(VIR_ERR_INTERNAL_ERROR,
- "%s", _("invalid watchdog action"));
- goto error;
- }
- virCommandAddArgList(cmd, "-watchdog-action", action, NULL);
- }
+ if (qemuBuildWatchdogCommandLine(cmd, def, qemuCaps) < 0)
+ goto error;
/* Add redirected devices */
for (i = 0; i < def->nredirdevs; i++) {
diff --git a/src/qemu/qemu_command.h b/src/qemu/qemu_command.h
index a26861b..6793777 100644
--- a/src/qemu/qemu_command.h
+++ b/src/qemu/qemu_command.h
@@ -125,10 +125,6 @@ char *qemuBuildControllerDevStr(const virDomainDef *domainDef,
virQEMUCapsPtr qemuCaps,
int *nusbcontroller);
-char *qemuBuildWatchdogDevStr(virDomainDefPtr domainDef,
- virDomainWatchdogDefPtr dev,
- virQEMUCapsPtr qemuCaps);
-
char *qemuBuildMemballoonDevStr(virDomainDefPtr domainDef,
virDomainMemballoonDefPtr dev,
virQEMUCapsPtr qemuCaps);
--
2.5.0