[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 03/15] monitor: register gen:false commands m
From: |
Marc-André Lureau |
Subject: |
Re: [Qemu-devel] [PATCH v3 03/15] monitor: register gen:false commands manually |
Date: |
Tue, 09 Aug 2016 17:16:23 +0000 |
On Tue, Aug 9, 2016 at 11:54 AM Markus Armbruster <address@hidden> wrote:
> address@hidden writes:
>
> > From: Marc-André Lureau <address@hidden>
> >
> > Since a few commands are using 'gen': false, they are not registered
> > automatically by the generator. Register manually instead.
> >
> > This is in preparation for removal of qapi 'middle' mode generation.
> >
> > Note that this function isn't run yet, until
>
> "these functions aren't"
>
> Can touch up on commit.
>
I meant the qmp_init_marshal(), I'll correct the comment.
>
> > module_call_init(MODULE_INIT_QAPI) is added in a later patch.
> >
> > Signed-off-by: Marc-André Lureau <address@hidden>
> > Reviewed-by: Eric Blake <address@hidden>
> > ---
> > monitor.c | 13 +++++++++++++
> > 1 file changed, 13 insertions(+)
> >
> > diff --git a/monitor.c b/monitor.c
> > index cb1a0c7..c9191e5 100644
> > --- a/monitor.c
> > +++ b/monitor.c
> > @@ -79,6 +79,7 @@
> > #include "sysemu/block-backend.h"
> > #include "sysemu/qtest.h"
> > #include "qemu/cutils.h"
> > +#include "qapi/qmp/dispatch.h"
> >
> > /* for hmp_info_irq/pic */
> > #if defined(TARGET_SPARC)
> > @@ -1000,6 +1001,18 @@ static void qmp_query_qmp_schema(QDict *qdict,
> QObject **ret_data,
> > *ret_data = qobject_from_json(qmp_schema_json);
> > }
> >
> > +static void qmp_init_marshal(void)
> > +{
> > + qmp_register_command("query-qmp-schema", qmp_query_qmp_schema,
> > + QCO_NO_OPTIONS);
> > + qmp_register_command("device_add", qmp_device_add,
> > + QCO_NO_OPTIONS);
> > + qmp_register_command("netdev_add", qmp_netdev_add,
> > + QCO_NO_OPTIONS);
> > +}
> > +
> > +qapi_init(qmp_init_marshal);
> > +
> > /* set the current CPU defined by the user */
> > int monitor_set_cpu(int cpu_index)
> > {
>
> --
Marc-André Lureau
- [Qemu-devel] [PATCH v3 00/15] qapi: remove the 'middle' mode, marcandre . lureau, 2016/08/08
- [Qemu-devel] [PATCH v3 01/15] qapi-schema: use generated marshaller for 'qmp_capabilities', marcandre . lureau, 2016/08/08
- [Qemu-devel] [PATCH v3 02/15] qapi-schema: add 'device_add', marcandre . lureau, 2016/08/08
- [Qemu-devel] [PATCH v3 03/15] monitor: register gen:false commands manually, marcandre . lureau, 2016/08/08
- [Qemu-devel] [PATCH v3 04/15] monitor: remove usage of generated marshal functions, marcandre . lureau, 2016/08/08
- [Qemu-devel] [PATCH v3 05/15] qapi: add 'export-marshal' command key, marcandre . lureau, 2016/08/08
- [Qemu-devel] [PATCH v3 06/15] monitor: register the qapi generated commands, marcandre . lureau, 2016/08/08
- [Qemu-devel] [PATCH v3 07/15] monitor: remove mhandler.cmd_new, marcandre . lureau, 2016/08/08
- [Qemu-devel] [PATCH v3 08/15] monitor: implement 'qmp_query_commands' without qmp_cmds, marcandre . lureau, 2016/08/08