On 3/9/21 6:18 AM, Kevin Wolf wrote:
> bdrv_close_all() asserts that no jobs are running any more, so we need
> to cancel all jobs first to avoid failing the assertion.
> 
> Fixes: b55a3c8860b763b62b2cc2f4a6f55379977bbde5
> Reported-by: Nini Gu <n...@redhat.com>
> Signed-off-by: Kevin Wolf <kw...@redhat.com>
> ---
>  storage-daemon/qemu-storage-daemon.c  |  1 +
>  tests/qemu-iotests/tests/qsd-jobs     | 66 +++++++++++++++++++++++++++
>  tests/qemu-iotests/tests/qsd-jobs.out | 22 +++++++++
>  3 files changed, 89 insertions(+)
>  create mode 100755 tests/qemu-iotests/tests/qsd-jobs
>  create mode 100644 tests/qemu-iotests/tests/qsd-jobs.out

Reviewed-by: Eric Blake <ebl...@redhat.com>

> +++ b/tests/qemu-iotests/tests/qsd-jobs

Yay for nicer iotest names ;)

> +seq="$(basename $0)"
> +echo "QA output created by $seq"

$seq is a bit of a misnomer, now, but such is life.

-- 
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3226
Virtualization:  qemu.org | libvirt.org


Reply via email to