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