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.


Reply via email to