[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] watchdog: convert to QemuOpts
From: |
Richard W.M. Jones |
Subject: |
Re: [Qemu-devel] [PATCH] watchdog: convert to QemuOpts |
Date: |
Wed, 27 May 2015 13:41:44 +0100 |
User-agent: |
Mutt/1.5.20 (2009-12-10) |
On Wed, May 27, 2015 at 02:35:26PM +0200, Paolo Bonzini wrote:
> case QEMU_OPTION_watchdog:
> - if (watchdog) {
> - fprintf(stderr,
> - "qemu: only one watchdog option may be given\n");
> - return 1;
> + olist = qemu_find_opts("watchdog");
> + opts = qemu_opts_parse(olist, optarg, 1);
> + if (!opts) {
> + exit(1);
> }
Is there an error message missing (before the call to exit)?
Patch seems OK otherwise.
libvirt still uses
-watchdog <model> -watchdog-action <action>
Should it be changed to use -device? I'm guessing for backwards
compatibility with old and current qemu it probably shouldn't be
changed for a few years.
Rich.
--
Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones
Read my programming and virtualization blog: http://rwmj.wordpress.com
Fedora Windows cross-compiler. Compile Windows programs, test, and
build Windows installers. Over 100 libraries supported.
http://fedoraproject.org/wiki/MinGW