[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] iotests: adapt test output for new qemu_cleanup() behavior
From: |
Kevin Wolf |
Subject: |
Re: [PATCH] iotests: adapt test output for new qemu_cleanup() behavior |
Date: |
Fri, 18 Aug 2023 16:56:41 +0200 |
Am 17.08.2023 um 13:25 hat Fiona Ebner geschrieben:
> Since commit ca2a5e630d ("qemu_cleanup: begin drained section after
> vm_shutdown()"), there will be an additional pause for jobs during
> qemu_cleanup(). The reason is that the bdrv_drain_all() call in
> do_vm_stop() is not inside the drained section used by qemu_cleanup()
> anymore. I.e., there is a second drained section now that ends before
> the final one in qemu_cleanup() starts. Thus, job_pause() is called
> twice during cleanup (via child_job_drained_begin()).
>
> Test 185 needs to be adapted directly too, because it waits for a
> specific number of JOB_STATUS_CHANGE events before the
> BLOCK_JOB_CANCELLED event.
>
> Reported-by: Kevin Wolf <kwolf@redhat.com>
> Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>
Thanks, applied to the block branch.
Kevin