Re: [Qemu-devel] [PATCH v3 0/8] drive-mirror improvements

From: Stefan Hajnoczi <stefanha@redhat.com>
To: Jeff Cody <jcody@redhat.com>
Cc: qemu-block@nongnu.org, qemu-devel@nongnu.org,
	Fam Zheng <famz@redhat.com>, Kevin Wolf <kwolf@redhat.com>,
	Max Reitz <mreitz@redhat.com>, Eric Blake <eblake@redhat.com>,
	"Denis V. Lunev" <den@openvz.org>
Subject: Re: [Qemu-devel] [PATCH v3 0/8] drive-mirror improvements
Date: Tue, 19 Jul 2016 09:43:10 +0100	[thread overview]
Message-ID: <20160719084310.GD29459@stefanha-x1.localdomain> (raw)
In-Reply-To: <578CF7AB.8090308@openvz.org>

[-- Attachment #1: Type: text/plain, Size: 2088 bytes --]

On Mon, Jul 18, 2016 at 06:37:15PM +0300, Denis V. Lunev wrote:
> On 07/14/2016 04:33 PM, Denis V. Lunev wrote:
> > This patchset contains patches dealing with known-to-be-zero areas in drive
> > mirror from [PATCH 0/9] major rework of drive-mirror patchset.
> > 
> > Changes from v2:
> > - added mirror_throttle helper (patch 3) to address Eric' comment about
> >    last_time_ns
> > - comment tweaks (thank you, Eric)
> > - marked mirror_dirty_init as couroutine_fn
> > 
> > Changes from v1:
> > - only patches dealing with zeroes remains
> > - ported to current HEAD
> > - fixed issue with dirty-bitmap, int length is changed with int64
> > - fixed sectors_in_flight usage
> > - patch 6 is reworked taken into account bugs found in active mirror
> > - fixed patch 7
> > - direct checking of .bdrv_co_write_zeroes is replaced with
> >    bdrv_can_write_zeroes_with_unmap
> >    - added fixes for bdrv_can_write_zeroes_with_unmap
> > 
> > Signed-off-by: Denis V. Lunev <den@openvz.org>
> > CC: Stefan Hajnoczi <stefanha@redhat.com>
> > CC: Fam Zheng <famz@redhat.com>
> > CC: Kevin Wolf <kwolf@redhat.com>
> > CC: Max Reitz <mreitz@redhat.com>
> > CC: Jeff Cody <jcody@redhat.com>
> > CC: Eric Blake <eblake@redhat.com>
> > 
> > Denis V. Lunev (8):
> >    dirty-bitmap: operate with int64_t amount
> >    mirror: make sectors_in_flight int64_t
> >    mirror: create mirror_throttle helper
> >    mirror: create mirror_dirty_init helper for mirror_run
> >    block: remove extra condition in bdrv_can_write_zeroes_with_unmap
> >    mirror: optimize dirty bitmap filling in mirror_run a bit
> >    mirror: efficiently zero out target
> >    mirror: improve performance of mirroring of empty disk
> > 
> >   block.c                      |   2 +-
> >   block/dirty-bitmap.c         |   6 +-
> >   block/mirror.c               | 145 +++++++++++++++++++++++++++++--------------
> >   include/block/block_int.h    |   2 +-
> >   include/block/dirty-bitmap.h |   4 +-
> >   5 files changed, 106 insertions(+), 53 deletions(-)
> > 
> ping

Jeff?

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 473 bytes --]