[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH V11 11/17] qmp: add ImageInfo in BlockDeviceInfo
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH V11 11/17] qmp: add ImageInfo in BlockDeviceInfo used by query-block |
Date: |
Thu, 11 Apr 2013 11:31:13 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.1 (gnu/linux) |
Wenchao Xia <address@hidden> writes:
> δΊ 2013-4-11 2:57, Luiz Capitulino ει:
>> On Wed, 10 Apr 2013 18:17:04 +0200
>> Markus Armbruster <address@hidden> wrote:
>>
>>> Stefan Hajnoczi <address@hidden> writes:
>>>
>>>> On Tue, Apr 02, 2013 at 07:47:24PM +0800, Wenchao Xia wrote:
>>>>> diff --git a/qmp-commands.hx b/qmp-commands.hx
>>>>> index 6b20684..b856be7 100644
>>>>> --- a/qmp-commands.hx
>>>>> +++ b/qmp-commands.hx
>>>>> @@ -1703,6 +1703,47 @@ Each json-object contain the following:
>>>>> - "iops": limit total I/O operations per second (json-int)
>>>>> - "iops_rd": limit read operations per second (json-int)
>>>>> - "iops_wr": limit write operations per second (json-int)
>>>>> + - "image": the detail of the image, it is a json-object
>>>>> containing
>>>>> + the following:
>>>>> + - "filename": image file name (json-string)
>>>>> + - "format": image format (json-string)
>>>>> + - "virtual-size": image capacity in bytes (json-int)
>>>>> + - "dirty-flag": true if image is not cleanly closed, not
>>>>> present
>>>>> + means clean (json-bool, optional)
>>>>> + - "actual-size": actual size on disk in bytes of the image, not
>>>>> + present when image does not support thin
>>>>> + provision (json-int, optional)
>>>>> + - "cluster-size": size of a cluster in bytes, not present if image
>>>>> + format does not support it (json-int, optional)
>>>>> + - "encrypted": true if the image is encrypted, not present
>>>>> means
>>>>> + false or the image format does not support
>>>>> + encryption (json-bool, optional)
>>>>> + - "backing_file": backing file name, not present means no backing
>>>>> + file is used or the image format does not
>>>>> + support backing file chain
>>>>> + (json-string, optional)
>>>>> + - "full-backing-filename": full path of the backing file,
>>>>> not
>>>>> + present if it equals backing_file or no
>>>>> + backing file is used
>>>>> + (json-string, optional)
>>>>> + - "backing-filename-format": the format of the backing file,
>>>>> not
>>>>> + present means unknown or no backing
>>>>> + file (json-string, optional)
>>>>> + - "snapshots": the internal snapshot info, it is an optional list
>>>>> + of json-object containing the following:
>>>>> + - "id": unique snapshot id (json-string)
>>>>> + - "name": snapshot name (json-string)
>>>>> + - "vm-state-size": size of the VM state in bytes (json-int)
>>>>> + - "date-sec": UTC date of the snapshot in seconds (json-int)
>>>>> + - "date-nsec": fractional part in nanoseconds to be used with
>>>>> + date-sec(json-int)
>>>>> + - "vm-clock-sec": VM clock relative to boot in seconds
>>>>> + (json-int)
>>>>> + - "vm-clock-nsec": fractional part in nanoseconds to be used
>>>>> + with vm-clock-sec (json-int)
>>>>> + - "backing-image": the detail of the backing image, it is an
>>>>> + optional json-object only present when a
>>>>> + backing image present for this image
>>>>
>>>> Please don't duplicate the ImageInfo documentation from
>>>> qapi-schema.json.
>>>
>>> No, this is actually how it still needs to be done.
>>>
>>> qmp-commands.hx predates qapi-schema.json. We still generate
>>> QMP/qmp-commands.txt from qmp-commands.hx, not from qapi-schema.json.
>>
>> Unfortunately, yes. However, as there are just a few commands missing
>> to be converted to the qapi, we could start generating the docs from
>> qapi-schema.json and avoid the duplication.
>>
>> There's a problem though. qapi-schema.json is not tied to the QMP protocol
>> as qmp-commands.hx is, so I'm not sure it's a good place for the examples,
> I agree that example should not get mixed with qapi-schema.json, which
> faces developer instead of user. Personally I think a standalone example
> file is better(only call in and out, no text for the structure defines).
How do we ensure the examples stay up-to-date?
Crazy idea: generate them as part of "make check".
[...]
- Re: [Qemu-devel] [PATCH V11 12/17] block: move bdrv_snapshot_dump() and dump_human_image_info() to block/qapi.c, (continued)
[Qemu-devel] [PATCH V11 11/17] qmp: add ImageInfo in BlockDeviceInfo used by query-block, Wenchao Xia, 2013/04/02
- Re: [Qemu-devel] [PATCH V11 11/17] qmp: add ImageInfo in BlockDeviceInfo used by query-block, Kevin Wolf, 2013/04/08
- Re: [Qemu-devel] [PATCH V11 11/17] qmp: add ImageInfo in BlockDeviceInfo used by query-block, Stefan Hajnoczi, 2013/04/08
- Re: [Qemu-devel] [PATCH V11 11/17] qmp: add ImageInfo in BlockDeviceInfo used by query-block, Markus Armbruster, 2013/04/10
- Re: [Qemu-devel] [PATCH V11 11/17] qmp: add ImageInfo in BlockDeviceInfo used by query-block, Luiz Capitulino, 2013/04/10
- Re: [Qemu-devel] [PATCH V11 11/17] qmp: add ImageInfo in BlockDeviceInfo used by query-block, Wenchao Xia, 2013/04/11
- Re: [Qemu-devel] [PATCH V11 11/17] qmp: add ImageInfo in BlockDeviceInfo used by query-block,
Markus Armbruster <=
- Re: [Qemu-devel] [PATCH V11 11/17] qmp: add ImageInfo in BlockDeviceInfo used by query-block, Wenchao Xia, 2013/04/11
Re: [Qemu-devel] [PATCH V11 11/17] qmp: add ImageInfo in BlockDeviceInfo used by query-block, Markus Armbruster, 2013/04/10
[Qemu-devel] [PATCH V11 13/17] block: dump to buffer for bdrv_snapshot_dump() and bdrv_image_info_dump(), Wenchao Xia, 2013/04/02