Re: [Qemu-devel] [PATCH for 2.6 1/1] iotests: fix the broken 026.nocache output

[Kevin Wolf]

Am 06.04.2016 um 08:08 hat Denis V. Lunev geschrieben:
> From: Pavel Butsykin <address@hidden>
> 
> 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 <address@hidden>
>     Date:   Tue Dec 2 18:32:51 2014 +0100
>     qcow2: Flushing the caches in qcow2_close may fail
> 
>     commit a069e2f1372a0a823ab506fc019852a2a652aa54
>     Author: John Snow <address@hidden>
>     Date:   Fri Feb 6 16:26:17 2015 -0500
>     blkdebug: fix "once" rule
> 
>     commit b106ad9185f35fc4ad669555ad0e79e276083bd7
>     Author: Kevin Wolf <address@hidden>
>     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 <address@hidden>
> Signed-off-by: Denis V. Lunev <address@hidden>
> CC: Max Reitz <address@hidden>
> CC: John Snow <address@hidden>
> CC: Kevin Wolf <address@hidden>

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