[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 01/18] qapi/block-core: Introduce BackupCommo
From: |
Max Reitz |
Subject: |
Re: [Qemu-devel] [PATCH v2 01/18] qapi/block-core: Introduce BackupCommon |
Date: |
Thu, 4 Jul 2019 16:25:45 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.7.0 |
On 03.07.19 23:55, John Snow wrote:
> drive-backup and blockdev-backup have an awful lot of things in common
> that are the same. Let's fix that.
>
> I don't deduplicate 'target', because the semantics actually did change
> between each structure. Leave that one alone so it can be documented
> separately.
>
> Signed-off-by: John Snow <address@hidden>
> ---
> qapi/block-core.json | 103 ++++++++++++++-----------------------------
> 1 file changed, 33 insertions(+), 70 deletions(-)
>
> diff --git a/qapi/block-core.json b/qapi/block-core.json
> index 0d43d4f37c..7b23efcf13 100644
> --- a/qapi/block-core.json
> +++ b/qapi/block-core.json
> @@ -1315,32 +1315,23 @@
> 'data': { 'node': 'str', 'overlay': 'str' } }
>
> ##
> -# @DriveBackup:
> +# @BackupCommon:
> #
> # @job-id: identifier for the newly-created block job. If
> # omitted, the device name will be used. (Since 2.7)
> #
> # @device: the device name or node-name of a root node which should be
> copied.
> #
> -# @target: the target of the new image. If the file exists, or if it
> -# is a device, the existing file/device will be used as the new
> -# destination. If it does not exist, a new file will be created.
> -#
> -# @format: the format of the new destination, default is to
> -# probe if @mode is 'existing', else the format of the source
> -#
> # @sync: what parts of the disk image should be copied to the destination
> # (all the disk, only the sectors allocated in the topmost image,
> from a
> # dirty bitmap, or only new I/O).
> #
> -# @mode: whether and how QEMU should create a new image, default is
> -# 'absolute-paths'.
> -#
> -# @speed: the maximum speed, in bytes per second
> +# @speed: the maximum speed, in bytes per second. The default is 0,
> +# for unlimited.
> #
> # @bitmap: the name of dirty bitmap if sync is "incremental".
> # Must be present if sync is "incremental", must NOT be present
> -# otherwise. (Since 2.4)
> +# otherwise. (Since 2.4 (Drive), 3.1 (Blockdev))
s/Drive/drive-backup/, s/Blockdev/blockdev-backup/?
Other than that:
Reviewed-by: Max Reitz <address@hidden>
signature.asc
Description: OpenPGP digital signature
[Qemu-devel] [PATCH v2 02/18] drive-backup: create do_backup_common, John Snow, 2019/07/03
[Qemu-devel] [PATCH v2 03/18] blockdev-backup: utilize do_backup_common, John Snow, 2019/07/03