[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v4 03/10] block/mirror: move dirty bitmap to filter
From: |
Eric Blake |
Subject: |
Re: [PATCH v4 03/10] block/mirror: move dirty bitmap to filter |
Date: |
Tue, 31 Oct 2023 13:08:04 -0500 |
User-agent: |
NeoMutt/20231023 |
On Tue, Oct 31, 2023 at 02:54:24PM +0100, Fiona Ebner wrote:
> In preparation to allow switching to active mode without draining.
> Initialization of the bitmap in mirror_dirty_init() still happens with
> the original/backing BlockDriverState, which should be fine, because
> the mirror top has the same length.
>
> Suggested-by: Vladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru>
> Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>
> ---
>
> Changes in v4:
> * Also set actively_synced to false when setting the bitmap in
> bdrv_mirror_top_do_write.
>
> block/mirror.c | 17 +++++++++++++----
> 1 file changed, 13 insertions(+), 4 deletions(-)
>
Reviewed-by: Eric Blake <eblake@redhat.com>
--
Eric Blake, Principal Software Engineer
Red Hat, Inc.
Virtualization: qemu.org | libguestfs.org
- [PATCH v4 00/10] mirror: allow switching from background to active mode, Fiona Ebner, 2023/10/31
- [PATCH v4 08/10] blockjob: query driver-specific info via a new 'query' driver method, Fiona Ebner, 2023/10/31
- [PATCH v4 03/10] block/mirror: move dirty bitmap to filter, Fiona Ebner, 2023/10/31
- Re: [PATCH v4 03/10] block/mirror: move dirty bitmap to filter,
Eric Blake <=
- [PATCH v4 04/10] block/mirror: determine copy_to_target only once, Fiona Ebner, 2023/10/31
- [PATCH v4 01/10] blockjob: introduce block-job-change QMP command, Fiona Ebner, 2023/10/31
- [PATCH v4 02/10] block/mirror: set actively_synced even after the job is ready, Fiona Ebner, 2023/10/31
- [PATCH v4 10/10] iotests: add test for changing mirror's copy_mode, Fiona Ebner, 2023/10/31
- [PATCH v4 09/10] mirror: return mirror-specific information upon query, Fiona Ebner, 2023/10/31
- [PATCH v4 06/10] qapi/block-core: use JobType for BlockJobInfo's type, Fiona Ebner, 2023/10/31
- [PATCH v4 07/10] qapi/block-core: turn BlockJobInfo into a union, Fiona Ebner, 2023/10/31
- [PATCH v4 05/10] mirror: implement mirror_change method, Fiona Ebner, 2023/10/31