Re: [Qemu-devel] [PATCH] mirror: Confirm we're quiesced only if the job is paused or cancelled

2019-03-07 Thread Eric Blake
On 3/7/19 8:03 AM, Sergio Lopez wrote: > While child_job_drained_begin() calls to job_pause(), the job doesn't > actually transition between states until it runs again and reaches a > pause point. This means bdrv_drained_begin() may return with some jobs > using the node still having 'busy == true'

[Qemu-devel] [PATCH] mirror: Confirm we're quiesced only if the job is paused or cancelled

2019-03-07 Thread Sergio Lopez
While child_job_drained_begin() calls to job_pause(), the job doesn't actually transition between states until it runs again and reaches a pause point. This means bdrv_drained_begin() may return with some jobs using the node still having 'busy == true'. As a consequence, block_job_detach_aio_conte