|
|
|
|
|
|
|
|
|
|
xen-devel
Re: [Xen-devel] [PATCH V4] libxl, Introduce a QMP client
On Mon, Jun 20, 2011 at 15:50, Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx> wrote:
> anthony.perard@xxxxxxxxxx writes ("[Xen-devel] [PATCH V4] libxl, Introduce a
> QMP client"):
>> QMP stands for QEMU Monitor Protocol and it is used to query information
>> from QEMU or to control QEMU.
>
> Thanks for this; it's coming along. I have some comments....
>
[...]
>> +/* QMP Command that can be send */
>> +enum libxl__qmp_command_e {
>> + QMP_COMMAND_QUERY_CHARDEV,
>> + QMP_COMMAND_NUMBER,
>> +};
> ...
>> +_hidden int libxl__qmp_send_command(libxl__qmp_handler *qmp,
>> libxl__qmp_command_e command);
>
> I'm not entirely convinced by this interface. Wouldn't it be better
> to have a specific function for each command ? After all commands may
> have arguments. Something like:
>
> _hidden int libxl__qmp_send_query_chardev(libxl__qmp_handler *qmp);
Indeed, it should be better to have specific functions. Some commands
will have arguments. And have a generic function for all of them will
maybe not be easy and not clear.
--
Anthony PERARD
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
|
|
|
|
|