[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH 1/2] QMP: Introduce commands doc
From: |
Jan Kiszka |
Subject: |
[Qemu-devel] Re: [PATCH 1/2] QMP: Introduce commands doc |
Date: |
Fri, 14 May 2010 19:08:07 +0200 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686 (x86_64); de; rv:1.8.1.12) Gecko/20080226 SUSE/2.0.0.12-1.1 Thunderbird/2.0.0.12 Mnenhy/0.7.5.666 |
Avi Kivity wrote:
> On 05/14/2010 08:01 PM, Avi Kivity wrote:
>> On 05/14/2010 07:52 PM, Jan Kiszka wrote:
>>>> In order not to compromise QMP adoption and make users' life easier,
>>>> this commit adds a simple text documentation which fully describes
>>>> all QMP supported commands.
>>>>
>>>> This is not ideal for a number of reasons (harder to maintain,
>>>> text-only, etc) but does improve the current situation.
>>> Even if it's temporary - maintaining it in a separate file looks rather
>>> unhandy.
>>>
>>> Can't we generate the per-command documentation snippets also from
>>> qemu-monitor.hx and merge them with a header/footer into some text file?
>>> That .hx file is the one anyone adding/converting commands has to touch
>>> anyway.
>> If we do, then the generated documentation should be included in the
>> patch changelog for review.
>>
>
> I mean, a patch introducing or modifying a monitor command.
The snippets should be readable by themselves. I'm only proposing to
keep them in the central file, at the same location where the others
are. There is no difference compared to existing monitor commands, we
just add the third documentation snippet, this time for QMP.
Distributing a generated QMP/qmp-commands.txt is another thing, maybe a
useful one.
Jan
--
Siemens AG, Corporate Technology, CT T DE IT 1
Corporate Competence Center Embedded Linux
- Re: [Qemu-devel] [PATCH 1/2] QMP: Introduce commands doc, (continued)
- Re: [Qemu-devel] [PATCH 1/2] QMP: Introduce commands doc, Avi Kivity, 2010/05/15
- Re: [Qemu-devel] [PATCH 1/2] QMP: Introduce commands doc, Markus Armbruster, 2010/05/17
- Re: [Qemu-devel] [PATCH 1/2] QMP: Introduce commands doc, Anthony Liguori, 2010/05/17
- Re: [Qemu-devel] [PATCH 1/2] QMP: Introduce commands doc, Avi Kivity, 2010/05/17
- Re: [Qemu-devel] [PATCH 1/2] QMP: Introduce commands doc, Markus Armbruster, 2010/05/18
- Re: [Qemu-devel] [PATCH 1/2] QMP: Introduce commands doc, Luiz Capitulino, 2010/05/18
- Re: [Qemu-devel] [PATCH 1/2] QMP: Introduce commands doc, Markus Armbruster, 2010/05/14
[Qemu-devel] Re: [PATCH 1/2] QMP: Introduce commands doc, Jan Kiszka, 2010/05/14
- [Qemu-devel] Re: [PATCH 1/2] QMP: Introduce commands doc, Avi Kivity, 2010/05/14
- [Qemu-devel] Re: [PATCH 1/2] QMP: Introduce commands doc, Avi Kivity, 2010/05/14
- [Qemu-devel] Re: [PATCH 1/2] QMP: Introduce commands doc,
Jan Kiszka <=
- [Qemu-devel] Re: [PATCH 1/2] QMP: Introduce commands doc, Avi Kivity, 2010/05/14
- [Qemu-devel] Re: [PATCH 1/2] QMP: Introduce commands doc, Luiz Capitulino, 2010/05/14
- [Qemu-devel] Re: [PATCH 1/2] QMP: Introduce commands doc, Jan Kiszka, 2010/05/15
- [Qemu-devel] Re: [PATCH 1/2] QMP: Introduce commands doc, Luiz Capitulino, 2010/05/17
- Re: [Qemu-devel] Re: [PATCH 1/2] QMP: Introduce commands doc, Markus Armbruster, 2010/05/18
- Re: [Qemu-devel] Re: [PATCH 1/2] QMP: Introduce commands doc, Luiz Capitulino, 2010/05/18