[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL 4/5] block: Dec. drained_end_counter before bdrv_wake
From: |
Max Reitz |
Subject: |
[Qemu-devel] [PULL 4/5] block: Dec. drained_end_counter before bdrv_wakeup |
Date: |
Mon, 22 Jul 2019 19:26:15 +0200 |
Decrementing drained_end_counter after bdrv_dec_in_flight() (which in
turn invokes bdrv_wakeup() and thus aio_wait_kick()) is not very clever.
We should decrement it beforehand, so that any waiting aio_poll() that
is woken by bdrv_dec_in_flight() sees the decremented
drained_end_counter.
Because the time window between decrementing drained_end_counter and
aio_wait_kick() is very small, I cannot supply a reliable regression
test. However, running e.g. the /bdrv-drain/blockjob/iothread/drain_all
test in test-bdrv-drain has a small chance of hanging without this
patch (about 1/200 or so; it gets to nearly 100 % if you add e.g. an
fputc(' ', stderr); after the bdrv_dec_in_flight()).
Fixes: e037c09c78520cbdb6da7cfc6ad0256d5870b814
Signed-off-by: Max Reitz <address@hidden>
Message-id: address@hidden
Signed-off-by: Max Reitz <address@hidden>
---
block/io.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/block/io.c b/block/io.c
index b89e155d21..06305c6ea6 100644
--- a/block/io.c
+++ b/block/io.c
@@ -217,13 +217,12 @@ static void coroutine_fn bdrv_drain_invoke_entry(void
*opaque)
bs->drv->bdrv_co_drain_end(bs);
}
- /* Set data->done before reading bs->wakeup. */
+ /* Set data->done and decrement drained_end_counter before bdrv_wakeup() */
atomic_mb_set(&data->done, true);
- bdrv_dec_in_flight(bs);
-
if (!data->begin) {
atomic_dec(data->drained_end_counter);
}
+ bdrv_dec_in_flight(bs);
g_free(data);
}
--
2.21.0
- [Qemu-devel] [PULL 0/5] Block patches for 4.1.0-rc2, Max Reitz, 2019/07/22
- [Qemu-devel] [PULL 1/5] block/nvme: fix doorbell stride, Max Reitz, 2019/07/22
- [Qemu-devel] [PULL 2/5] block/nvme: support larger that 512 bytes sector devices, Max Reitz, 2019/07/22
- [Qemu-devel] [PULL 3/5] block/nvme: don't touch the completion entries, Max Reitz, 2019/07/22
- [Qemu-devel] [PULL 4/5] block: Dec. drained_end_counter before bdrv_wakeup,
Max Reitz <=
- [Qemu-devel] [PULL 5/5] block: Only the main loop can change AioContexts, Max Reitz, 2019/07/22
- Re: [Qemu-devel] [PULL 0/5] Block patches for 4.1.0-rc2, Peter Maydell, 2019/07/23