qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
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 --]

  reply	other threads:[~2016-07-19  8:43 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-07-14 13:33 [Qemu-devel] [PATCH v3 0/8] drive-mirror improvements Denis V. Lunev
2016-07-14 13:33 ` [Qemu-devel] [PATCH v3 1/8] dirty-bitmap: operate with int64_t amount Denis V. Lunev
2016-07-14 13:33 ` [Qemu-devel] [PATCH v3 2/8] mirror: make sectors_in_flight int64_t Denis V. Lunev
2016-07-14 13:33 ` [Qemu-devel] [PATCH v3 3/8] mirror: create mirror_throttle helper Denis V. Lunev
2016-07-14 15:21   ` Eric Blake
2016-07-14 13:33 ` [Qemu-devel] [PATCH v3 4/8] mirror: create mirror_dirty_init helper for mirror_run Denis V. Lunev
2016-07-14 16:19   ` Eric Blake
2016-07-14 16:48     ` Denis V. Lunev
2016-07-14 13:33 ` [Qemu-devel] [PATCH v3 5/8] block: remove extra condition in bdrv_can_write_zeroes_with_unmap Denis V. Lunev
2016-07-14 13:33 ` [Qemu-devel] [PATCH v3 6/8] mirror: optimize dirty bitmap filling in mirror_run a bit Denis V. Lunev
2016-07-14 13:33 ` [Qemu-devel] [PATCH v3 7/8] mirror: efficiently zero out target Denis V. Lunev
2016-07-14 13:33 ` [Qemu-devel] [PATCH v3 8/8] mirror: improve performance of mirroring of empty disk Denis V. Lunev
2016-07-18 15:37 ` [Qemu-devel] [PATCH v3 0/8] drive-mirror improvements Denis V. Lunev
2016-07-19  8:43   ` Stefan Hajnoczi [this message]
2016-07-19 21:07 ` Jeff Cody

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20160719084310.GD29459@stefanha-x1.localdomain \
    --to=stefanha@redhat.com \
    --cc=den@openvz.org \
    --cc=eblake@redhat.com \
    --cc=famz@redhat.com \
    --cc=jcody@redhat.com \
    --cc=kwolf@redhat.com \
    --cc=mreitz@redhat.com \
    --cc=qemu-block@nongnu.org \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).