[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 7/8] dataplane: avoid reentrancy during virtio_blk_d
From: |
Stefan Hajnoczi |
Subject: |
[Qemu-devel] [PATCH 7/8] dataplane: avoid reentrancy during virtio_blk_data_plane_stop() |
Date: |
Fri, 18 Jan 2013 17:28:40 +0100 |
When dataplane is stopping, the s->vdev->binding->set_host_notifier(...,
false) call can invoke the virtqueue handler if an ioeventfd
notification is pending. This causes hw/virtio-blk.c to invoke
virtio_blk_data_plane_start() before virtio_blk_data_plane_stop()
returns!
The result is that we try to restart dataplane while trying to stop it
and the following assertion is raised:
msix_set_mask_notifier: Assertion `!dev->msix_mask_notifier' failed.
Although the code was intended to prevent this scenario, the s->started
boolean isn't enough. Add s->stopping so that we can postpone clearing
s->started until we've completely stopped dataplane.
This way, virtqueue handler calls during virtio_blk_data_plane_stop()
are ignored. When dataplane is legitimately started again later we
already self-kick ourselves to resume processing.
Signed-off-by: Stefan Hajnoczi <address@hidden>
---
hw/dataplane/virtio-blk.c | 9 ++++++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/hw/dataplane/virtio-blk.c b/hw/dataplane/virtio-blk.c
index 4b26faa..3f2da22 100644
--- a/hw/dataplane/virtio-blk.c
+++ b/hw/dataplane/virtio-blk.c
@@ -40,6 +40,7 @@ typedef struct {
struct VirtIOBlockDataPlane {
bool started;
+ bool stopping;
QEMUBH *start_bh;
QemuThread thread;
@@ -357,7 +358,7 @@ static void *data_plane_thread(void *opaque)
do {
event_poll(&s->event_poll);
- } while (s->started || s->num_reqs > 0);
+ } while (!s->stopping || s->num_reqs > 0);
return NULL;
}
@@ -486,10 +487,10 @@ void virtio_blk_data_plane_start(VirtIOBlockDataPlane *s)
void virtio_blk_data_plane_stop(VirtIOBlockDataPlane *s)
{
- if (!s->started) {
+ if (!s->started || s->stopping) {
return;
}
- s->started = false;
+ s->stopping = true;
trace_virtio_blk_data_plane_stop(s);
/* Stop thread or cancel pending thread creation BH */
@@ -511,4 +512,6 @@ void virtio_blk_data_plane_stop(VirtIOBlockDataPlane *s)
s->vdev->binding->set_guest_notifiers(s->vdev->binding_opaque, 1, false);
vring_teardown(&s->vring);
+ s->started = false;
+ s->stopping = false;
}
--
1.8.0.2
- [Qemu-devel] [PULL for-1.4 0/8] Block patches, Stefan Hajnoczi, 2013/01/18
- [Qemu-devel] [PATCH 2/8] ide: Remove wrong assertion, Stefan Hajnoczi, 2013/01/18
- [Qemu-devel] [PATCH 1/8] block: fix null-pointer bug on error case in block commit, Stefan Hajnoczi, 2013/01/18
- [Qemu-devel] [PATCH 4/8] win32-aio: Fix vectored reads, Stefan Hajnoczi, 2013/01/18
- [Qemu-devel] [PATCH 5/8] win32-aio: Fix memory leak, Stefan Hajnoczi, 2013/01/18
- [Qemu-devel] [PATCH 6/8] win32-aio: use iov utility functions instead of open-coding them, Stefan Hajnoczi, 2013/01/18
- [Qemu-devel] [PATCH 3/8] aio: Fix return value of aio_poll(), Stefan Hajnoczi, 2013/01/18
- [Qemu-devel] [PATCH 7/8] dataplane: avoid reentrancy during virtio_blk_data_plane_stop(),
Stefan Hajnoczi <=
- [Qemu-devel] [PATCH 8/8] dataplane: support viostor virtio-pci status bit setting, Stefan Hajnoczi, 2013/01/18
- Re: [Qemu-devel] [PULL for-1.4 0/8] Block patches, Anthony Liguori, 2013/01/20