From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:50733)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <stefanha@redhat.com>) id 1WnrEs-0001m5-No
	for qemu-devel@nongnu.org; Fri, 23 May 2014 11:18:36 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <stefanha@redhat.com>) id 1WnrEm-0003vj-Pu
	for qemu-devel@nongnu.org; Fri, 23 May 2014 11:18:30 -0400
Received: from mx1.redhat.com ([209.132.183.28]:7630)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <stefanha@redhat.com>) id 1WnrEm-0003vX-Gn
	for qemu-devel@nongnu.org; Fri, 23 May 2014 11:18:24 -0400
Date: Fri, 23 May 2014 17:18:20 +0200
From: Stefan Hajnoczi <stefanha@redhat.com>
Message-ID: <20140523151820.GA4585@stefanha-thinkpad.redhat.com>
References: <1400851788-32588-1-git-send-email-famz@redhat.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <1400851788-32588-1-git-send-email-famz@redhat.com>
Subject: Re: [Qemu-devel] [PATCH v2 0/8] block: Drop in_use with op blocker
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Fam Zheng <famz@redhat.com>
Cc: Kevin Wolf <kwolf@redhat.com>, Benoit Canet <benoit@irqsave.net>, jcody@redhat.com, qemu-devel@nongnu.org, Max Reitz <mreitz@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>

On Fri, May 23, 2014 at 09:29:40PM +0800, Fam Zheng wrote:
> This is split from series "[Qemu-devel] [PATCH v20 00/15] Drop in_use from
> BlockDriverState and enable point-in-time snapshot exporting over NBD", with a
> new patch 06 to avoid backing_blocker assertion violation.
> 
> v2: Address Stefan's comments on last two patches from v1, and add a patch 8 to
>     drop one more redundant bdrv_refresh_limits() after bdrv_set_backing_hd().
> 
> Thanks for reviewing!
> 
> Fam
> 
> 
> Fam Zheng (8):
>   block: Add BlockOpType enum
>   block: Introduce op_blockers to BlockDriverState
>   block: Replace in_use with operation blocker
>   block: Move op_blocker check from block_job_create to its caller
>   block: Add bdrv_set_backing_hd()
>   block: Use bdrv_set_backing_hd everywhere
>   block: Add backing_blocker in BlockDriverState
>   block: Drop redundant bdrv_refresh_limits
> 
>  block-migration.c               |   7 +-
>  block.c                         | 152 +++++++++++++++++++++++++++++++---------
>  block/mirror.c                  |   2 +-
>  block/stream.c                  |   4 +-
>  block/vvfat.c                   |   2 +-
>  blockdev.c                      |  27 ++++---
>  blockjob.c                      |  14 ++--
>  hw/block/dataplane/virtio-blk.c |  18 +++--
>  include/block/block.h           |  29 +++++++-
>  include/block/block_int.h       |   9 ++-
>  include/block/blockjob.h        |   3 +
>  11 files changed, 204 insertions(+), 63 deletions(-)

Thanks, applied to my block tree:
https://github.com/stefanha/qemu/commits/block

Stefan