On 14.07.2016 19:06, Eric Blake wrote:
On 07/14/2016 06:29 AM, Denis V. Lunev wrote:
From: Evgeny Yakovlev <eyakov...@virtuozzo.com>
bdrv_co_flush is now skipping flushes in case underlying media has no
actual changes. This affected some blkdebug testcases that were
expecting error logs from failure-injected flushes which are now
skipped entirely.
This change removes expected flush error logs from block tests 026 071 089
Signed-off-by: Evgeny Yakovlev <eyakov...@virtuozzo.com>
Signed-off-by: Denis V. Lunev <d...@openvz.org>
CC: Kevin Wolf <kw...@redhat.com>
CC: Max Reitz <mre...@redhat.com>
CC: Stefan Hajnoczi <stefa...@redhat.com>
CC: Fam Zheng <f...@redhat.com>
CC: John Snow <js...@redhat.com>
---
tests/qemu-iotests/026.out.nocache | 50 --------------------------------------
tests/qemu-iotests/071.out | 8 ------
tests/qemu-iotests/089.out | 2 --
3 files changed, 60 deletions(-)
If the previous patch broke the testsuite, then this should be squashed
in with that patch so that bisection doesn't land on a broken testsuite.
Seeing the testsuite change alongside code change is just fine; it
proves what impact the code change has.
Will do.