[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 3/6] QAPI: Introduce memchar_write QMP command
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH 3/6] QAPI: Introduce memchar_write QMP command |
Date: |
Fri, 31 Aug 2012 09:26:46 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.1 (gnu/linux) |
Eric Blake <address@hidden> writes:
> On 08/22/2012 11:14 PM, Lei Li wrote:
>> Signed-off-by: Lei Li <address@hidden>
>
> Subject line uses '_', but the QMP command uses '-' [1]
>
>> ---
>> hmp-commands.hx | 16 ++++++++++++++++
>> hmp.c | 15 +++++++++++++++
>> hmp.h | 1 +
>> qapi-schema.json | 28 ++++++++++++++++++++++++++++
>> qemu-char.c | 36 ++++++++++++++++++++++++++++++++++++
>> qmp-commands.hx | 33 +++++++++++++++++++++++++++++++++
>> 6 files changed, 129 insertions(+), 0 deletions(-)
>>
>> diff --git a/hmp-commands.hx b/hmp-commands.hx
>> index f6104b0..829aea1 100644
>> --- a/hmp-commands.hx
>> +++ b/hmp-commands.hx
>> @@ -797,6 +797,22 @@ Inject an NMI on the given CPU (x86 only).
>> ETEXI
>>
>> {
>> + .name = "memchar-write",
>
> HMP commands should use '_', not '-'.
Both '_' and '-' occur in HMP command names. There are more occurences
of '_', though.
Otherwise identical HMP and QMP commands differing only in '_' vs. '-'
is ugly. Maybe we should simply map '_' in HMP command names to '-' and
be done with it.
[...]
- [Qemu-devel] [PATCH 1/6] qemu-char: Convert MemCharDriver to circular buffer, (continued)
- [Qemu-devel] [PATCH 1/6] qemu-char: Convert MemCharDriver to circular buffer, Lei Li, 2012/08/23
- [Qemu-devel] [PATCH 2/6] monitor: Adjust qmp_human_monitor_command to new MemCharDriver, Lei Li, 2012/08/23
- [Qemu-devel] [PATCH 4/6] QAPI: Introduce memchar_read QMP command, Lei Li, 2012/08/23
- [Qemu-devel] [PATCH 3/6] QAPI: Introduce memchar_write QMP command, Lei Li, 2012/08/23
- [Qemu-devel] [PATCH 5/6] Fix enumeration typo error, Lei Li, 2012/08/23
- [Qemu-devel] [PATCH 6/6] Expose MemCharDriver via command line, Lei Li, 2012/08/23