|
 |
ed5979 |
From e790b4c20a5124239fe93e91fbc87745e5f2cea6 Mon Sep 17 00:00:00 2001
|
|
|
ed5979 |
From: Kevin Wolf <kwolf@redhat.com>
|
|
|
ed5979 |
Date: Fri, 18 Nov 2022 18:41:01 +0100
|
|
|
ed5979 |
Subject: [PATCH 19/31] block: Fix locking for bdrv_reopen_queue_child()
|
|
|
ed5979 |
|
|
|
ed5979 |
RH-Author: Stefano Garzarella <sgarzare@redhat.com>
|
|
|
ed5979 |
RH-MergeRequest: 135: block: Simplify drain to prevent QEMU from crashing during snapshot
|
|
|
ed5979 |
RH-Bugzilla: 2155112
|
|
|
ed5979 |
RH-Acked-by: Emanuele Giuseppe Esposito <eesposit@redhat.com>
|
|
|
ed5979 |
RH-Acked-by: Hanna Czenczek <hreitz@redhat.com>
|
|
|
ed5979 |
RH-Acked-by: Kevin Wolf <kwolf@redhat.com>
|
|
|
ed5979 |
RH-Commit: [7/16] 46bb54506c4400b9a1bf66b6bd7987ff67260003 (sgarzarella/qemu-kvm-c-9-s)
|
|
|
ed5979 |
|
|
|
ed5979 |
Callers don't agree whether bdrv_reopen_queue_child() should be called
|
|
|
ed5979 |
with the AioContext lock held or not. Standardise on holding the lock
|
|
|
ed5979 |
(as done by QMP blockdev-reopen and the replication block driver) and
|
|
|
ed5979 |
fix bdrv_reopen() to do the same.
|
|
|
ed5979 |
|
|
|
ed5979 |
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
|
|
|
ed5979 |
Message-Id: <20221118174110.55183-7-kwolf@redhat.com>
|
|
|
ed5979 |
Reviewed-by: Hanna Reitz <hreitz@redhat.com>
|
|
|
ed5979 |
Reviewed-by: Vladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru>
|
|
|
ed5979 |
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
|
|
|
ed5979 |
(cherry picked from commit 2e117866d7c96cc17e84cd2946fee1bf3292d814)
|
|
|
ed5979 |
Signed-off-by: Stefano Garzarella <sgarzare@redhat.com>
|
|
|
ed5979 |
---
|
|
|
ed5979 |
block.c | 7 +++++--
|
|
|
ed5979 |
1 file changed, 5 insertions(+), 2 deletions(-)
|
|
|
ed5979 |
|
|
|
ed5979 |
diff --git a/block.c b/block.c
|
|
|
ed5979 |
index 7999fd08c5..46df410b07 100644
|
|
|
ed5979 |
--- a/block.c
|
|
|
ed5979 |
+++ b/block.c
|
|
|
ed5979 |
@@ -4151,6 +4151,8 @@ static bool bdrv_recurse_has_child(BlockDriverState *bs,
|
|
|
ed5979 |
* bs_queue, or the existing bs_queue being used.
|
|
|
ed5979 |
*
|
|
|
ed5979 |
* bs must be drained between bdrv_reopen_queue() and bdrv_reopen_multiple().
|
|
|
ed5979 |
+ *
|
|
|
ed5979 |
+ * To be called with bs->aio_context locked.
|
|
|
ed5979 |
*/
|
|
|
ed5979 |
static BlockReopenQueue *bdrv_reopen_queue_child(BlockReopenQueue *bs_queue,
|
|
|
ed5979 |
BlockDriverState *bs,
|
|
|
ed5979 |
@@ -4309,6 +4311,7 @@ static BlockReopenQueue *bdrv_reopen_queue_child(BlockReopenQueue *bs_queue,
|
|
|
ed5979 |
return bs_queue;
|
|
|
ed5979 |
}
|
|
|
ed5979 |
|
|
|
ed5979 |
+/* To be called with bs->aio_context locked */
|
|
|
ed5979 |
BlockReopenQueue *bdrv_reopen_queue(BlockReopenQueue *bs_queue,
|
|
|
ed5979 |
BlockDriverState *bs,
|
|
|
ed5979 |
QDict *options, bool keep_old_opts)
|
|
|
ed5979 |
@@ -4473,11 +4476,11 @@ int bdrv_reopen(BlockDriverState *bs, QDict *opts, bool keep_old_opts,
|
|
|
ed5979 |
GLOBAL_STATE_CODE();
|
|
|
ed5979 |
|
|
|
ed5979 |
bdrv_subtree_drained_begin(bs);
|
|
|
ed5979 |
+ queue = bdrv_reopen_queue(NULL, bs, opts, keep_old_opts);
|
|
|
ed5979 |
+
|
|
|
ed5979 |
if (ctx != qemu_get_aio_context()) {
|
|
|
ed5979 |
aio_context_release(ctx);
|
|
|
ed5979 |
}
|
|
|
ed5979 |
-
|
|
|
ed5979 |
- queue = bdrv_reopen_queue(NULL, bs, opts, keep_old_opts);
|
|
|
ed5979 |
ret = bdrv_reopen_multiple(queue, errp);
|
|
|
ed5979 |
|
|
|
ed5979 |
if (ctx != qemu_get_aio_context()) {
|
|
|
ed5979 |
--
|
|
|
ed5979 |
2.31.1
|
|
|
ed5979 |
|