Skip to content

Fix qemu-iotests failures

Kevin Wolf qemu-devel 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest4 patches 22 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/4Revert "coroutine: abort if we try to schedule or enter a pending coroutine"include/qemu/coroutine_int.h util/async.c util/qemu-coroutine-sleep.c +1 more+3 -497 replies
2/4Revert "blockjob: do not allow coroutine double entry or entry-after-completion"blockjob.c include/block/blockjob_int.h+3 -71 reply
3/4coroutine: Cancel aio_co_schedule() on direct entryinclude/qemu/coroutine_int.h util/async.c util/qemu-coroutine.c+32 -111 replies
4/4block: Expect graph changes in bdrv_parent_drained_begin/endblock/io.c+4 -41 reply

Reviews trailers per patch

3 of 4 patches have a Reviewed-by or Acked-by of their own; the series has one for all of them.

PatchJeff CodyStefan Hajnoczi
SeriesT-b R-b
1/4R-b
2/4R-b
3/4
4/4R-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [Qemu-devel] [PATCH for-2.11 0/4] Fix qemu-iotests failures

This reverts the recent fix for the case of qemu-iotests 200 (block job completion races when using a separate I/O thread) that only fixed a symptom and caused many regressions, and replaces it with a new solution that hopefully attacks the right root cause.

The series also contains a bonus fix for a bug that was uncovered by the different timing with the patches that are reverted here.

Kevin Wolf (4):
Revert "coroutine: abort if we try to schedule or enter a pending
coroutine"
Revert "blockjob: do not allow coroutine double entry or
entry-after-completion"
coroutine: Cancel aio_co_schedule() on direct entry
block: Expect graph changes in bdrv_parent_drained_begin/end

 include/block/blockjob_int.h |  3 +--
 include/qemu/coroutine_int.h | 14 ++++----------
 block/io.c                   |  8 ++++----
 blockjob.c                   |  7 ++-----
 util/async.c                 | 26 +++++++++++++-------------
 util/qemu-coroutine-sleep.c  | 12 ------------
 util/qemu-coroutine.c        | 31 +++++++++++++++++--------------
 7 files changed, 41 insertions(+), 60 deletions(-)
Signature · 1 line
2.13.6