[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [Qemu-devel] [PATCH v2 0/1] drive-mirror: add increment
Re: [Qemu-block] [Qemu-devel] [PATCH v2 0/1] drive-mirror: add incremental mode
Wed, 2 Jan 2019 09:51:02 -0600
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.3.1
On 1/2/19 4:04 AM, mahaocong wrote:
> From: mahaocong <address@hidden>
> This patch adds possibility to start mirroring with user-created-bitmap.
> Compare with v1, this patch fix some shortcomings.
> 1.separate feature about copy dirty-bitmap to an individual patch.
> 2.remove checking for cancelled after mirror_dirty_init_incremental for bitmap
> copyimg don't have yield point.
> 3.adjuest input parameters on mirror_start_job and mirror_start, and so It is
> no need to find bitmap on mirror_dirty_init_incremental again.
> 4.assert the bitmap name is NULL on blockdev_mirror_common.
> 5.change the parameter's name in qmp command 'drive-mirror' from 'bitmap_name'
> to 'bitmap'.
> As for the doubt about the new incremental mode, I think it is means that the
> initial bitmap is appointed by user, compare with the full mode and top mode.
> There is no different in other behaves, such as in iteration and complete
> mahaocong (1):
> drive-mirror: add incremental mode
> mirror: add incremental mode. we must add a user-named-bitmap first, then
> this bitmap as the initial bitmap on incremental mode drive-mirror.
> in details, I first create unnamed-bitmap with the same granularity
> user-bitmap. Next, copy It's hbitmap to unnamed-bitmap. Then, start
> mirror with this unname-bitmap.
> dirty-bitmap: add new API to copy dirty-bitmap. This feature is add in a
> separate patch called "add new function to copy dirty-bitmap"
This is confusing. The first line 'mahaocong (1):' looks like you are
sending a patch series with only one patch (by the way, you can use 'git
commit format.coverletter auto' to only send a 0/N cover letter for N >
1, since a single patch is the only case where you don't technically
need a cover letter). But the remaining three entries make it look like
you have three patches intended to be sent (and where the patch titled
"mirror: add incremental mode. we must..." is an awfully long subject line).
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3226
Virtualization: qemu.org | libvirt.org
Description: OpenPGP digital signature