Re: [Xen-devel] [PATCH V5 3/3] libxl, Introduce a QMP client

Anthony PERARD writes ("[Xen-devel] [PATCH V5 3/3] libxl, Introduce a QMP 
client"):
> QMP stands for QEMU Monitor Protocol and it is used to query information
> from QEMU or to control QEMU.
> 
> This implementation will ask QEMU the list of chardevice and store the
> path to serial0 in xenstored. So we will be able to use xl console with
> QEMU upstream.

Can I make a suggestion ?  I think the formulaic json parser stuff
could usefully live in a separate file.

> +static inline yajl_gen_status yajl_gen_asciiz(yajl_gen hand, const
char *str)

Isn't this a hostage to fortune ?  yajl may grow an identically-named
function in which case this will no longer build.

> +#ifdef DEBUG_ANSWER
> +    if (qmp->g)
> +        yajl_gen_free(qmp->g);
> +#endif

Can this #ifdef not be shuffled off somewhere ?  Ie, make a debug
function (or macro) which we call unconditionally.

The rest looks plausible.

Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel

<Prev in Thread]

Current Thread

[Next in Thread>

Previous by Date:

[Xen-devel] Re: [PATCH 2/2] xen: Add alias to autoload backend drivers, Konrad Rzeszutek Wilk

Next by Date:

Re: [Xen-devel] [PATCH RESENT] xl: Use macros for param parsing in network-attach, to prevent use explicit sizeof("param"), Ian Jackson

Previous by Thread:

[Xen-devel] [PATCH V5 3/3] libxl, Introduce a QMP client, Anthony PERARD

Next by Thread:

Re: [Xen-devel] [PATCH V5 3/3] libxl, Introduce a QMP client, Anthony PERARD

Indexes:

[Date] [Thread] [Top] [All Lists]

WARNING - OLD ARCHIVES

xen-devel

Re: [Xen-devel] [PATCH V5 3/3] libxl, Introduce a QMP client