On 1/5/22 15:02, Emanuele Giuseppe Esposito wrote:
Now that they are not nop anymore, remove the aiocontext to avoid deadlocks.
Ok, that should have been in patch 1, together with a description of the deadlocks. :) Disregard that review.
There is only one JobDriver callback, ->free() that assumes that the aiocontext lock is held (because it calls bdrv_unref), so for now keep that under aiocontext lock.
This explains the issue with detach_child in the other series, too. Paolo