Re: [Qemu-devel] [PATCH v2 1/1] drive-mirror: add incremental mode

[Eric Blake]

On 1/2/19 4:04 AM, mahaocong wrote:
> From: mahaocong <address@hidden>
> 
> Signed-off-by: mahaocong <address@hidden>

The subject line says "what" changed, but you are missing a commit body
that says "why" the change is useful.  Much of the content in your cover
letter should be copied into this commit body, as the cover letter does
not get stored into git.

> ---
>  block/mirror.c            | 55 
> ++++++++++++++++++++++++++++++++++++-----------
>  blockdev.c                | 37 +++++++++++++++++++++++++++++--
>  include/block/block_int.h |  3 ++-
>  qapi/block-core.json      |  9 +++++++-
>  4 files changed, 88 insertions(+), 16 deletions(-)
> 

> +++ b/qapi/block-core.json
> @@ -1727,6 +1727,13 @@
>  #        (all the disk, only the sectors allocated in the topmost image, or
>  #        only new I/O).
>  #
> +# @bitmap: the name of user-created-bitmap which is used to initialize the
> +#          unnamed-bitmap used on incremental drive-mirror. It's hbitmap

"It's" should only be used when you could also have said "It is" or "It
has" - but in this sentence, you are using a possessive which is spelled
"Its".  However, the entire concept of an hbitmap is an internal
implementation detail, so I don't know if it makes sense to be
mentioning an hbitmap in this location.

> +#          will be copy to mirror bitmap. If user select incremental mode,
> +#          bitmap-name should not be null, and can not set granularity for
> +#          the mirror bitmap should have the same granularity with
> +#          user-created-bitmap.

Missing a '(since 4.0)' tag.

-- 
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3226
Virtualization:  qemu.org | libvirt.org

signature.asc
Description: OpenPGP digital signature