[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2] watchdog: convert to QemuOpts
|
From: |
Paolo Bonzini |
|
Subject: |
Re: [Qemu-devel] [PATCH v2] watchdog: convert to QemuOpts |
|
Date: |
Wed, 03 Jun 2015 17:38:20 +0200 |
|
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 |
On 03/06/2015 17:27, Markus Armbruster wrote:
> I agree that "watchdog-action" is an ugly name for backend
> configuration. "watchdog" would be fine, but we foolishly burned that
> on silly command line sugar for the frontend.
>
> "watchdog-config"? Precedent: "semihosting-config".
Sucks, but at least is consistent. It was hard to convince me on this
as I was just typing "git send-pull" when I got your review, but you did
it. :)
Paolo
> Then make -watchdog T pure sugar for -device T, and -watchdog-action A
> pure sugar for -watchdog-config action=A.
>
> Not demands, just ideas.