[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 09/18] QMP: Introduce command dispatch table
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH 09/18] QMP: Introduce command dispatch table |
Date: |
Wed, 29 Sep 2010 13:31:13 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/23.1 (gnu/linux) |
Luiz Capitulino <address@hidden> writes:
> Also update QMP functions to use it. The table is generated
> from the qemu-monitor-qmp.hx file.
>
>>From now on, QMP and HMP have different command dispatch
> tables.
>
> Signed-off-by: Luiz Capitulino <address@hidden>
> ---
> Makefile.target | 7 +++++--
> monitor.c | 11 +++++++++--
> 2 files changed, 14 insertions(+), 4 deletions(-)
>
> diff --git a/Makefile.target b/Makefile.target
> index a4e80b1..7fab81b 100644
> --- a/Makefile.target
> +++ b/Makefile.target
> @@ -299,7 +299,7 @@ obj-alpha-y = alpha_palcode.o
>
> main.o: QEMU_CFLAGS+=$(GPROF_CFLAGS)
>
> -monitor.o: qemu-monitor.h
> +monitor.o: qemu-monitor.h qemu-monitor-qmp.h
>
> $(obj-y) $(obj-$(TARGET_BASE_ARCH)-y): $(GENERATED_HEADERS)
>
> @@ -323,10 +323,13 @@ gdbstub-xml.c: $(TARGET_XML_FILES)
> $(SRC_PATH)/feature_to_c.sh
> qemu-monitor.h: $(SRC_PATH)/qemu-monitor.hx
> $(call quiet-command,sh $(SRC_PATH)/hxtool -h < $< > $@," GEN
> $(TARGET_DIR)$@")
>
> +qemu-monitor-qmp.h: $(SRC_PATH)/qemu-monitor-qmp.hx
> + $(call quiet-command,sh $(SRC_PATH)/hxtool -h < $< > $@," GEN
> $(TARGET_DIR)$@")
> +
> clean:
> rm -f *.o *.a *~ $(PROGS) nwfpe/*.o fpu/*.o
> rm -f *.d */*.d tcg/*.o ide/*.o
> - rm -f qemu-monitor.h gdbstub-xml.c
> + rm -f qemu-monitor.h qmp-monitor-qmp.h gdbstub-xml.c
Typo: qmp-monitor-qmp.h instead of qemu-monitor-qmp.h.
I'm not thrilled about qemu-monitor-qmp; because if I expand the
abbreviation, I get "QEMU monitor QEMU Monitor Protocol".
What about just qmp?
[...]
> install: all
> ifneq ($(PROGS),)
> diff --git a/monitor.c b/monitor.c
> index 7b699db..fe1a29a 100644
> --- a/monitor.c
> +++ b/monitor.c
> @@ -189,6 +189,8 @@ static QLIST_HEAD(mon_list, Monitor) mon_list;
> static const mon_cmd_t mon_cmds[];
> static const mon_cmd_t info_cmds[];
>
> +static const mon_cmd_t qmp_cmds[];
> +
> Monitor *cur_mon;
> Monitor *default_mon;
>
> @@ -742,7 +744,7 @@ static void do_info_commands(Monitor *mon, QObject
> **ret_data)
>
> cmd_list = qlist_new();
>
> - for (cmd = mon_cmds; cmd->name != NULL; cmd++) {
> + for (cmd = qmp_cmds; cmd->name != NULL; cmd++) {
> if (monitor_handler_ported(cmd) && !monitor_cmd_user_only(cmd) &&
> !compare_cmd(cmd->name, "info")) {
> qlist_append_obj(cmd_list, get_cmd_dict(cmd->name));
We need to switch here, because "info commands" is about QMP commands,
not about HMP commands. Curious, isn't it?
> @@ -2632,6 +2634,11 @@ static const mon_cmd_t info_cmds[] = {
> },
> };
>
> +static const mon_cmd_t qmp_cmds[] = {
> +#include "qemu-monitor-qmp.h"
> + { /* NULL */ },
> +};
> +
> /*******************************************************************/
>
> static const char *pch;
> @@ -3364,7 +3371,7 @@ static const mon_cmd_t *qmp_find_query_cmd(const char
> *info_item)
>
> static const mon_cmd_t *qmp_find_cmd(const char *cmdname)
> {
> - return find_dispatch_table(mon_cmds, cmdname);
> + return find_dispatch_table(qmp_cmds, cmdname);
> }
>
> static const mon_cmd_t *monitor_parse_command(Monitor *mon,
[Qemu-devel] [PATCH 07/18] Monitor: Introduce the qemu-monitor-qmp.hx file, Luiz Capitulino, 2010/09/16
[Qemu-devel] [PATCH 10/18] QMP: Introduce query commands dispatch table, Luiz Capitulino, 2010/09/16
[Qemu-devel] [PATCH 11/18] QMP: Simplify do_info_commands(), Luiz Capitulino, 2010/09/16
[Qemu-devel] [PATCH 14/18] Monitor: Drop monitor_cmd_user_only(), Luiz Capitulino, 2010/09/16
[Qemu-devel] [PATCH 13/18] Monitor: Drop QMP info from the qemu-monitor.hx file, Luiz Capitulino, 2010/09/16
[Qemu-devel] [PATCH 15/18] Monitor: Rename monitor_handler_ported(), Luiz Capitulino, 2010/09/16
[Qemu-devel] [PATCH 16/18] Monitor: Rename monitor_handler_is_async(), Luiz Capitulino, 2010/09/16
[Qemu-devel] [PATCH 17/18] Monitor: Directly call QObject handlers, Luiz Capitulino, 2010/09/16