|
 |
357786 |
From 8c657cb165e931b4eb6f3675495240437b0c1cb1 Mon Sep 17 00:00:00 2001
|
|
|
357786 |
From: Kevin Wolf <kwolf@redhat.com>
|
|
|
357786 |
Date: Fri, 14 Sep 2018 10:55:38 +0200
|
|
|
357786 |
Subject: [PATCH 47/49] test-bdrv-drain: Test nested poll in
|
|
|
357786 |
bdrv_drain_poll_top_level()
|
|
|
357786 |
|
|
|
357786 |
RH-Author: Kevin Wolf <kwolf@redhat.com>
|
|
|
357786 |
Message-id: <20180914105540.18077-41-kwolf@redhat.com>
|
|
|
357786 |
Patchwork-id: 82189
|
|
|
357786 |
O-Subject: [RHV-7.6 qemu-kvm-rhev PATCH 40/42] test-bdrv-drain: Test nested poll in bdrv_drain_poll_top_level()
|
|
|
357786 |
Bugzilla: 1601212
|
|
|
357786 |
RH-Acked-by: John Snow <jsnow@redhat.com>
|
|
|
357786 |
RH-Acked-by: Max Reitz <mreitz@redhat.com>
|
|
|
357786 |
RH-Acked-by: Fam Zheng <famz@redhat.com>
|
|
|
357786 |
|
|
|
357786 |
This is a regression test for a deadlock that could occur in callbacks
|
|
|
357786 |
called from the aio_poll() in bdrv_drain_poll_top_level(). The
|
|
|
357786 |
AioContext lock wasn't released and therefore would be taken a second
|
|
|
357786 |
time in the callback. This would cause a possible AIO_WAIT_WHILE() in
|
|
|
357786 |
the callback to hang.
|
|
|
357786 |
|
|
|
357786 |
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
|
|
|
357786 |
Reviewed-by: Fam Zheng <famz@redhat.com>
|
|
|
357786 |
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
357786 |
---
|
|
|
357786 |
tests/test-bdrv-drain.c | 13 +++++++++++++
|
|
|
357786 |
1 file changed, 13 insertions(+)
|
|
|
357786 |
|
|
|
357786 |
diff --git a/tests/test-bdrv-drain.c b/tests/test-bdrv-drain.c
|
|
|
357786 |
index 05f3b55..f4b57f7 100644
|
|
|
357786 |
--- a/tests/test-bdrv-drain.c
|
|
|
357786 |
+++ b/tests/test-bdrv-drain.c
|
|
|
357786 |
@@ -636,6 +636,17 @@ static void test_iothread_aio_cb(void *opaque, int ret)
|
|
|
357786 |
qemu_event_set(&done_event);
|
|
|
357786 |
}
|
|
|
357786 |
|
|
|
357786 |
+static void test_iothread_main_thread_bh(void *opaque)
|
|
|
357786 |
+{
|
|
|
357786 |
+ struct test_iothread_data *data = opaque;
|
|
|
357786 |
+
|
|
|
357786 |
+ /* Test that the AioContext is not yet locked in a random BH that is
|
|
|
357786 |
+ * executed during drain, otherwise this would deadlock. */
|
|
|
357786 |
+ aio_context_acquire(bdrv_get_aio_context(data->bs));
|
|
|
357786 |
+ bdrv_flush(data->bs);
|
|
|
357786 |
+ aio_context_release(bdrv_get_aio_context(data->bs));
|
|
|
357786 |
+}
|
|
|
357786 |
+
|
|
|
357786 |
/*
|
|
|
357786 |
* Starts an AIO request on a BDS that runs in the AioContext of iothread 1.
|
|
|
357786 |
* The request involves a BH on iothread 2 before it can complete.
|
|
|
357786 |
@@ -705,6 +716,8 @@ static void test_iothread_common(enum drain_type drain_type, int drain_thread)
|
|
|
357786 |
aio_context_acquire(ctx_a);
|
|
|
357786 |
}
|
|
|
357786 |
|
|
|
357786 |
+ aio_bh_schedule_oneshot(ctx_a, test_iothread_main_thread_bh, &data);
|
|
|
357786 |
+
|
|
|
357786 |
/* The request is running on the IOThread a. Draining its block device
|
|
|
357786 |
* will make sure that it has completed as far as the BDS is concerned,
|
|
|
357786 |
* but the drain in this thread can continue immediately after
|
|
|
357786 |
--
|
|
|
357786 |
1.8.3.1
|
|
|
357786 |
|