From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:45351)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <kwolf@redhat.com>) id 1ankPW-0007ig-Fi
	for qemu-devel@nongnu.org; Wed, 06 Apr 2016 06:10:07 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <kwolf@redhat.com>) id 1ankPR-0000Jt-98
	for qemu-devel@nongnu.org; Wed, 06 Apr 2016 06:10:06 -0400
Received: from mx1.redhat.com ([209.132.183.28]:43090)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <kwolf@redhat.com>) id 1ankPR-0000Is-4S
	for qemu-devel@nongnu.org; Wed, 06 Apr 2016 06:10:01 -0400
Date: Wed, 6 Apr 2016 12:09:57 +0200
From: Kevin Wolf <kwolf@redhat.com>
Message-ID: <20160406100957.GC5098@noname.redhat.com>
References: <1459922912-31715-1-git-send-email-den@openvz.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <1459922912-31715-1-git-send-email-den@openvz.org>
Subject: Re: [Qemu-devel] [PATCH for 2.6 1/1] iotests: fix the broken
	026.nocache output
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: "Denis V. Lunev" <den@openvz.org>
Cc: Max Reitz <mreitz@redhat.com>, John Snow <jsnow@redhat.com>, qemu-devel@nongnu.org, Pavel Butsykin <pbutsykin@virtuozzo.com>

Am 06.04.2016 um 08:08 hat Denis V. Lunev geschrieben:
> From: Pavel Butsykin <pbutsykin@virtuozzo.com>
> 
> This patch fixes longstanding issue with 026 iotest. Unfortunately,
> this test contains 2 versions of the correct output, one for cached
> writes and one for non-cached ones. People tends to fix only one
> version of output of the test and thus noncached version becomes
> broken. Unfortunately, it is default in tests/check-block.sh
> 
> The following problematic commits were made:
>     commit 3b5e14c76a6bb142bf250ddf99e24a0ac8c7bc12
>     Author: Max Reitz <mreitz@redhat.com>
>     Date:   Tue Dec 2 18:32:51 2014 +0100
>     qcow2: Flushing the caches in qcow2_close may fail
> 
>     commit a069e2f1372a0a823ab506fc019852a2a652aa54
>     Author: John Snow <jsnow@redhat.com>
>     Date:   Fri Feb 6 16:26:17 2015 -0500
>     blkdebug: fix "once" rule
> 
>     commit b106ad9185f35fc4ad669555ad0e79e276083bd7
>     Author: Kevin Wolf <kwolf@redhat.com>
>     Date:   Fri Mar 28 18:06:31 2014 +0100
>     qcow2: Don't rely on free_cluster_index in alloc_refcount_block()
> 
> Signed-off-by: Pavel Butsykin <pbutsykin@virtuozzo.com>
> Signed-off-by: Denis V. Lunev <den@openvz.org>
> CC: Max Reitz <mreitz@redhat.com>
> CC: John Snow <jsnow@redhat.com>
> CC: Kevin Wolf <kwolf@redhat.com>

Thanks for tracking this down! It's always amazing to have a patch where
the commit message is by far more work than the actual patch, and this
is certainly one of them. With these references to older commits, the
changes look all good.

Applied to the block branch.

Kevin