[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH v12 10/19] block: Add QMP support for streaming
From: |
Eric Blake |
Subject: |
Re: [Qemu-block] [PATCH v12 10/19] block: Add QMP support for streaming to an intermediate layer |
Date: |
Thu, 27 Oct 2016 09:46:38 -0500 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.4.0 |
On 10/27/2016 03:58 AM, Kevin Wolf wrote:
>>>> -# @backing-file: #optional The backing file string to write into the
>>>> active
>>>> -# layer. This filename is not validated.
>>>> +# @backing-file: #optional The backing file string to write into the top
>>>> +# image. This filename is not validated.
>>>> #
>>>> # If a pathname string is such that it cannot be
>>>> # resolved by QEMU, that means that subsequent
>>>> QMP or
>>>
>>> As we discussed in v10, this is not discoverable through
>>> introspection. You added patch 18 which introduces a base-node option
>>> and can serve as a witness for the changed semantics, which is
>>> good. Should this be documented here?
>>
>> In the commit message I don't see why not, but in the JSON file?
>>
>> "This feature was added together with the base-node parameter" ?
>
> Eric may have a better suggestion for the wording, but maybe something
> like this:
>
> "Presence of this feature can't directly be tested with introspection;
> check for the presence of base-node instead as a witness for it."
The sentence on checking for the feature should go earlier, with this
paragraph. Maybe as follows:
# The node that receives the data is called the top image, and can be
# located in any part of the chain (but always above the base image;
# see below) and can be specified using its device or node name.
# Earlier qemu versions only allowed 'device' to name the top level
# node; presence of the 'base-node' parameter during introspection can
# be used as a witness of the enhanced semantics of 'device'.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- Re: [Qemu-block] [PATCH v12 03/19] block: Add block_job_add_bdrv(), (continued)
[Qemu-block] [PATCH v12 08/19] block: Block all intermediate nodes in commit_active_start(), Alberto Garcia, 2016/10/26
[Qemu-block] [PATCH v12 18/19] block: Add 'base-node' parameter to the 'block-stream' command, Alberto Garcia, 2016/10/26
[Qemu-block] [PATCH v12 14/19] qemu-iotests: Test overlapping stream and commit operations, Alberto Garcia, 2016/10/26