From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:49986) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1atUrk-0001tG-4q for qemu-devel@nongnu.org; Fri, 22 Apr 2016 02:47:00 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1atUrj-000698-CD for qemu-devel@nongnu.org; Fri, 22 Apr 2016 02:47:00 -0400 Date: Fri, 22 Apr 2016 14:46:57 +0800 From: Fam Zheng Message-ID: <20160422064657.GA4437@ad.usersys.redhat.com> References: <1461306907-2837-1-git-send-email-famz@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1461306907-2837-1-git-send-email-famz@redhat.com> Subject: Re: [Qemu-devel] [PATCH for-2.6 0/3] block: Fix assertion failure at mirror exit List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Kevin Wolf , qemu-block@nongnu.org, Jeff Cody , Max Reitz , stefanha@redhat.com, pbonzini@redhat.com, qemu-stable@nongnu.org On Fri, 04/22 14:35, Fam Zheng wrote: > This supersedes the "virtio: Register host notifier handler as external" patch > from yesterday. > > The bug was initially reported by Matthew Schumacher as LaunchPad Bug 1570134, > and nicely bisected by Max Reitz. See patch 2 for the analysis. > > We are late for 2.6 so the fix is done locally in mirror.c. In 2.7 we should > revisit this and let bdrv_drained_begin do the work. > > Fam Zheng (3): > mirror: Extract mirror_replace > mirror: Skip BH for mirror_exit if in main loop > block: Update comment of bdrv_drained_begin > > block/mirror.c | 43 ++++++++++++++++++++++++++++++------------- > include/block/block.h | 5 +++++ > 2 files changed, 35 insertions(+), 13 deletions(-) > > -- > 2.8.0 > > Cc: qemu-stable@nongnu.org