|
 |
902636 |
From 73d83d8880e85eedc22c9651b67d1eacd5de5ff4 Mon Sep 17 00:00:00 2001
|
|
|
902636 |
From: Sergio Lopez Pascual <slp@redhat.com>
|
|
|
902636 |
Date: Wed, 24 Jun 2020 13:24:52 -0400
|
|
|
902636 |
Subject: [PATCH 10/12] virtio-blk: Refactor the code that processes queued
|
|
|
902636 |
requests
|
|
|
902636 |
|
|
|
902636 |
RH-Author: Sergio Lopez Pascual <slp@redhat.com>
|
|
|
902636 |
Message-id: <20200624132453.111276-2-slp@redhat.com>
|
|
|
902636 |
Patchwork-id: 97797
|
|
|
902636 |
O-Subject: [RHEL-AV-8.2.1 qemu-kvm PATCH 1/2] virtio-blk: Refactor the code that processes queued requests
|
|
|
902636 |
Bugzilla:
|
|
|
902636 |
RH-Acked-by: John Snow <jsnow@redhat.com>
|
|
|
902636 |
RH-Acked-by: Michael S. Tsirkin <mst@redhat.com>
|
|
|
902636 |
RH-Acked-by: Kevin Wolf <kwolf@redhat.com>
|
|
|
902636 |
|
|
|
902636 |
Move the code that processes queued requests from
|
|
|
902636 |
virtio_blk_dma_restart_bh() to its own, non-static, function. This
|
|
|
902636 |
will allow us to call it from the virtio_blk_data_plane_start() in a
|
|
|
902636 |
future patch.
|
|
|
902636 |
|
|
|
902636 |
Signed-off-by: Sergio Lopez <slp@redhat.com>
|
|
|
902636 |
Message-Id: <20200603093240.40489-2-slp@redhat.com>
|
|
|
902636 |
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
|
|
|
902636 |
(cherry picked from commit 7aa1c247b466870b0704d3ccdc3755e5e7394dca)
|
|
|
902636 |
Signed-off-by: Sergio Lopez <slp@redhat.com>
|
|
|
902636 |
Signed-off-by: Danilo C. L. de Paula <ddepaula@redhat.com>
|
|
|
902636 |
---
|
|
|
902636 |
hw/block/virtio-blk.c | 16 +++++++++++-----
|
|
|
902636 |
include/hw/virtio/virtio-blk.h | 1 +
|
|
|
902636 |
2 files changed, 12 insertions(+), 5 deletions(-)
|
|
|
902636 |
|
|
|
902636 |
diff --git a/hw/block/virtio-blk.c b/hw/block/virtio-blk.c
|
|
|
902636 |
index c4e55fb3de..6ff29a05d6 100644
|
|
|
902636 |
--- a/hw/block/virtio-blk.c
|
|
|
902636 |
+++ b/hw/block/virtio-blk.c
|
|
|
902636 |
@@ -819,15 +819,11 @@ static void virtio_blk_handle_output(VirtIODevice *vdev, VirtQueue *vq)
|
|
|
902636 |
virtio_blk_handle_output_do(s, vq);
|
|
|
902636 |
}
|
|
|
902636 |
|
|
|
902636 |
-static void virtio_blk_dma_restart_bh(void *opaque)
|
|
|
902636 |
+void virtio_blk_process_queued_requests(VirtIOBlock *s)
|
|
|
902636 |
{
|
|
|
902636 |
- VirtIOBlock *s = opaque;
|
|
|
902636 |
VirtIOBlockReq *req = s->rq;
|
|
|
902636 |
MultiReqBuffer mrb = {};
|
|
|
902636 |
|
|
|
902636 |
- qemu_bh_delete(s->bh);
|
|
|
902636 |
- s->bh = NULL;
|
|
|
902636 |
-
|
|
|
902636 |
s->rq = NULL;
|
|
|
902636 |
|
|
|
902636 |
aio_context_acquire(blk_get_aio_context(s->conf.conf.blk));
|
|
|
902636 |
@@ -855,6 +851,16 @@ static void virtio_blk_dma_restart_bh(void *opaque)
|
|
|
902636 |
aio_context_release(blk_get_aio_context(s->conf.conf.blk));
|
|
|
902636 |
}
|
|
|
902636 |
|
|
|
902636 |
+static void virtio_blk_dma_restart_bh(void *opaque)
|
|
|
902636 |
+{
|
|
|
902636 |
+ VirtIOBlock *s = opaque;
|
|
|
902636 |
+
|
|
|
902636 |
+ qemu_bh_delete(s->bh);
|
|
|
902636 |
+ s->bh = NULL;
|
|
|
902636 |
+
|
|
|
902636 |
+ virtio_blk_process_queued_requests(s);
|
|
|
902636 |
+}
|
|
|
902636 |
+
|
|
|
902636 |
static void virtio_blk_dma_restart_cb(void *opaque, int running,
|
|
|
902636 |
RunState state)
|
|
|
902636 |
{
|
|
|
902636 |
diff --git a/include/hw/virtio/virtio-blk.h b/include/hw/virtio/virtio-blk.h
|
|
|
902636 |
index cddcfbebe9..cf8eea2f58 100644
|
|
|
902636 |
--- a/include/hw/virtio/virtio-blk.h
|
|
|
902636 |
+++ b/include/hw/virtio/virtio-blk.h
|
|
|
902636 |
@@ -84,5 +84,6 @@ typedef struct MultiReqBuffer {
|
|
|
902636 |
} MultiReqBuffer;
|
|
|
902636 |
|
|
|
902636 |
bool virtio_blk_handle_vq(VirtIOBlock *s, VirtQueue *vq);
|
|
|
902636 |
+void virtio_blk_process_queued_requests(VirtIOBlock *s);
|
|
|
902636 |
|
|
|
902636 |
#endif
|
|
|
902636 |
--
|
|
|
902636 |
2.27.0
|
|
|
902636 |
|