[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 04/21] qapi: add QAPI visitor core
From: |
Luiz Capitulino |
Subject: |
Re: [Qemu-devel] [PATCH v3 04/21] qapi: add QAPI visitor core |
Date: |
Wed, 15 Jun 2011 11:33:36 -0300 |
On Mon, 13 Jun 2011 21:31:09 -0500
Michael Roth <address@hidden> wrote:
> Base definitions/includes for Visiter interface used by generated
> visiter/marshalling code.
>
> Includes a GenericList type. Our lists require an embedded element.
> Since these types are generated, if you want to use them in a different
> type of data structure, there's no easy way to add another embedded
> element. The solution is to have non-embedded lists and that what this is.
>
> Signed-off-by: Michael Roth <address@hidden>
> ---
> Makefile.objs | 6 +++
> qapi/qapi-types-core.h | 21 ++++++++++
> qapi/qapi-visit-core.c | 101
> ++++++++++++++++++++++++++++++++++++++++++++++++
> qapi/qapi-visit-core.h | 68 ++++++++++++++++++++++++++++++++
> 4 files changed, 196 insertions(+), 0 deletions(-)
> create mode 100644 qapi/qapi-types-core.h
> create mode 100644 qapi/qapi-visit-core.c
> create mode 100644 qapi/qapi-visit-core.h
>
> diff --git a/Makefile.objs b/Makefile.objs
> index a7807e8..68d7b5a 100644
> --- a/Makefile.objs
> +++ b/Makefile.objs
> @@ -364,6 +364,12 @@ endif
>
> libcacard-y = cac.o event.o vcard.o vreader.o vcard_emul_nss.o
> vcard_emul_type.o card_7816.o
>
> +######################################################################
> +# qapi
> +
> +qapi-nested-y = qapi-visit-core.o
> +qapi-obj-y = $(addprefix qapi/, $(qapi-nested-y))
To build this I have to add qapi-obj-y to common-obj-y and add the
qapi directory to the DIR variable in configure (so that it builds
in a different directory).
I saw that you send the configure change in a different patch, but it
has to be done here.
> +
> vl.o: QEMU_CFLAGS+=$(GPROF_CFLAGS)
>
> vl.o: QEMU_CFLAGS+=$(SDL_CFLAGS)
> diff --git a/qapi/qapi-types-core.h b/qapi/qapi-types-core.h
> new file mode 100644
> index 0000000..de733ab
> --- /dev/null
> +++ b/qapi/qapi-types-core.h
> @@ -0,0 +1,21 @@
> +/*
> + * Core Definitions for QAPI-generated Types
> + *
> + * Copyright IBM, Corp. 2011
> + *
> + * Authors:
> + * Anthony Liguori <address@hidden>
> + *
> + * This work is licensed under the terms of the GNU LGPL, version 2.1 or
> later.
> + * See the COPYING.LIB file in the top-level directory.
> + *
> + */
> +
> +#ifndef QAPI_TYPES_CORE_H
> +#define QAPI_TYPES_CORE_H
> +
> +#include <stdbool.h>
> +#include <stdint.h>
> +#include "error.h"
> +
> +#endif
> diff --git a/qapi/qapi-visit-core.c b/qapi/qapi-visit-core.c
> new file mode 100644
> index 0000000..948818e
> --- /dev/null
> +++ b/qapi/qapi-visit-core.c
> @@ -0,0 +1,101 @@
> +#include "qapi/qapi-visit-core.h"
No license info.
> +
> +void visit_start_handle(Visitor *v, void **obj, const char *kind, const char
> *name, Error **errp)
> +{
> + if (!error_is_set(errp) && v->start_handle) {
> + v->start_handle(v, obj, kind, name, errp);
> + }
> +}
> +
> +void visit_end_handle(Visitor *v, Error **errp)
> +{
> + if (!error_is_set(errp) && v->end_handle) {
> + v->end_handle(v, errp);
> + }
> +}
> +
> +void visit_start_struct(Visitor *v, void **obj, const char *kind, const char
> *name, size_t size, Error **errp)
> +{
> + if (!error_is_set(errp)) {
> + v->start_struct(v, obj, kind, name, size, errp);
> + }
> +}
> +
> +void visit_end_struct(Visitor *v, Error **errp)
> +{
> + if (!error_is_set(errp)) {
> + v->end_struct(v, errp);
> + }
> +}
> +
> +void visit_start_list(Visitor *v, const char *name, Error **errp)
> +{
> + if (!error_is_set(errp)) {
> + v->start_list(v, name, errp);
> + }
> +}
> +
> +GenericList *visit_next_list(Visitor *v, GenericList **list, Error **errp)
> +{
> + if (!error_is_set(errp)) {
> + return v->next_list(v, list, errp);
> + }
> +
> + return 0;
> +}
> +
> +void visit_end_list(Visitor *v, Error **errp)
> +{
> + if (!error_is_set(errp)) {
> + v->end_list(v, errp);
> + }
> +}
> +
> +void visit_start_optional(Visitor *v, bool *present, const char *name, Error
> **errp)
> +{
> + if (!error_is_set(errp) && v->start_optional) {
> + v->start_optional(v, present, name, errp);
> + }
> +}
> +
> +void visit_end_optional(Visitor *v, Error **errp)
> +{
> + if (!error_is_set(errp) && v->end_optional) {
> + v->end_optional(v, errp);
> + }
> +}
> +
> +void visit_type_enum(Visitor *v, int *obj, const char *kind, const char
> *name, Error **errp)
> +{
> + if (!error_is_set(errp)) {
> + v->type_enum(v, obj, kind, name, errp);
> + }
> +}
> +
> +void visit_type_int(Visitor *v, int64_t *obj, const char *name, Error **errp)
> +{
> + if (!error_is_set(errp)) {
> + v->type_int(v, obj, name, errp);
> + }
> +}
> +
> +void visit_type_bool(Visitor *v, bool *obj, const char *name, Error **errp)
> +{
> + if (!error_is_set(errp)) {
> + v->type_bool(v, obj, name, errp);
> + }
> +}
> +
> +void visit_type_str(Visitor *v, char **obj, const char *name, Error **errp)
> +{
> + if (!error_is_set(errp)) {
> + v->type_str(v, obj, name, errp);
> + }
> +}
> +
> +void visit_type_number(Visitor *v, double *obj, const char *name, Error
> **errp)
> +{
> + if (!error_is_set(errp)) {
> + v->type_number(v, obj, name, errp);
> + }
> +}
> diff --git a/qapi/qapi-visit-core.h b/qapi/qapi-visit-core.h
> new file mode 100644
> index 0000000..8350a6d
> --- /dev/null
> +++ b/qapi/qapi-visit-core.h
> @@ -0,0 +1,68 @@
> +/*
> + * Core Definitions for QAPI Visitor Classes
> + *
> + * Copyright IBM, Corp. 2011
> + *
> + * Authors:
> + * Anthony Liguori <address@hidden>
> + *
> + * This work is licensed under the terms of the GNU LGPL, version 2.1 or
> later.
> + * See the COPYING.LIB file in the top-level directory.
> + *
> + */
> +#ifndef QAPI_VISITOR_CORE_H
> +#define QAPI_VISITOR_CORE_H
> +
> +#include "qapi/qapi-types-core.h"
> +#include "error.h"
> +#include <stdlib.h>
> +
> +typedef struct GenericList
> +{
> + void *value;
> + struct GenericList *next;
> +} GenericList;
> +
> +typedef struct Visitor Visitor;
> +
> +struct Visitor
> +{
> + /* Must be set */
> + void (*start_struct)(Visitor *v, void **obj, const char *kind, const
> char *name, size_t size, Error **errp);
> + void (*end_struct)(Visitor *v, Error **errp);
> +
> + void (*start_list)(Visitor *v, const char *name, Error **errp);
> + GenericList *(*next_list)(Visitor *v, GenericList **list, Error **errp);
> + void (*end_list)(Visitor *v, Error **errp);
> +
> + void (*type_enum)(Visitor *v, int *obj, const char *kind, const char
> *name, Error **errp);
> +
> + void (*type_int)(Visitor *v, int64_t *obj, const char *name, Error
> **errp);
> + void (*type_bool)(Visitor *v, bool *obj, const char *name, Error **errp);
> + void (*type_str)(Visitor *v, char **obj, const char *name, Error **errp);
> + void (*type_number)(Visitor *v, double *obj, const char *name, Error
> **errp);
> +
> + /* May be NULL */
> + void (*start_optional)(Visitor *v, bool *present, const char *name,
> Error **errp);
> + void (*end_optional)(Visitor *v, Error **errp);
> +
> + void (*start_handle)(Visitor *v, void **obj, const char *kind, const
> char *name, Error **errp);
> + void (*end_handle)(Visitor *v, Error **errp);
> +};
> +
> +void visit_start_handle(Visitor *v, void **obj, const char *kind, const char
> *name, Error **errp);
> +void visit_end_handle(Visitor *v, Error **errp);
> +void visit_start_struct(Visitor *v, void **obj, const char *kind, const char
> *name, size_t size, Error **errp);
> +void visit_end_struct(Visitor *v, Error **errp);
> +void visit_start_list(Visitor *v, const char *name, Error **errp);
> +GenericList *visit_next_list(Visitor *v, GenericList **list, Error **errp);
> +void visit_end_list(Visitor *v, Error **errp);
> +void visit_start_optional(Visitor *v, bool *present, const char *name, Error
> **errp);
> +void visit_end_optional(Visitor *v, Error **errp);
> +void visit_type_enum(Visitor *v, int *obj, const char *kind, const char
> *name, Error **errp);
> +void visit_type_int(Visitor *v, int64_t *obj, const char *name, Error
> **errp);
> +void visit_type_bool(Visitor *v, bool *obj, const char *name, Error **errp);
> +void visit_type_str(Visitor *v, char **obj, const char *name, Error **errp);
> +void visit_type_number(Visitor *v, double *obj, const char *name, Error
> **errp);
> +
> +#endif
- [Qemu-devel] [QAPI+QGA 2/3] QAPI code generation infrastructure v3, Michael Roth, 2011/06/13
- [Qemu-devel] [PATCH v3 02/21] qlist: add qlist_first()/qlist_next(), Michael Roth, 2011/06/13
- [Qemu-devel] [PATCH v3 01/21] Add hard build dependency on glib, Michael Roth, 2011/06/13
- [Qemu-devel] [PATCH v3 04/21] qapi: add QAPI visitor core, Michael Roth, 2011/06/13
- Re: [Qemu-devel] [PATCH v3 04/21] qapi: add QAPI visitor core,
Luiz Capitulino <=
- [Qemu-devel] [PATCH v3 03/21] qapi: add module init types for qapi, Michael Roth, 2011/06/13
- [Qemu-devel] [PATCH v3 05/21] qapi: add QMP input visitor, Michael Roth, 2011/06/13
- [Qemu-devel] [PATCH v3 06/21] qapi: add QMP output visitor, Michael Roth, 2011/06/13
- [Qemu-devel] [PATCH v3 07/21] qapi: add QAPI dealloc visitor, Michael Roth, 2011/06/13
- [Qemu-devel] [PATCH v3 08/21] qapi: add QMP command registration/lookup functions, Michael Roth, 2011/06/13