This serie aims to remove and clean up some bugs that came up when trying to replace the AioContext lock and still protect BlockDriverState fields.
They were part of the serie "Removal of Aiocontext lock through drains: protect bdrv_replace_child_noperm", but since that serie is still a work in progress and these fixes are pretty much independent, I split that in two separate series. --- v2: * change comments in patch 2 and 3, .attach() and .detach() callbacks. * remove job_sleep_ns patch, as it was causing random deadlocks in test 030 Emanuele Giuseppe Esposito (10): drains: create bh only when polling bdrv_parent_drained_begin_single: handle calls from coroutine context block/io.c: fix bdrv_child_cb_drained_begin invocations from a coroutine block.c: bdrv_replace_child_noperm: first remove the child, and then call ->detach() block.c: bdrv_replace_child_noperm: first call ->attach(), and then add child test-bdrv-drain.c: adapt test to the coming subtree drains test-bdrv-drain.c: remove test_detach_by_parent_cb() tests/unit/test-bdrv-drain.c: graph setup functions can't run in coroutines child_job_drained_poll: override polling condition only when in home thread tests/qemu-iotests/030: test_stream_parallel should use auto_finalize=False block.c | 41 ++++++--- block/io.c | 137 +++++++++++++++++++++++++--- blockjob.c | 13 ++- include/block/block-io.h | 20 +++-- tests/qemu-iotests/030 | 12 +-- tests/unit/test-bdrv-drain.c | 169 +++++++++++++++++++---------------- 6 files changed, 267 insertions(+), 125 deletions(-) -- 2.31.1