[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v2 3/7] virtio-blk: Make request completion function
From: |
Fam Zheng |
Subject: |
[Qemu-devel] [PATCH v2 3/7] virtio-blk: Make request completion function virtual |
Date: |
Tue, 17 Jun 2014 14:32:06 +0800 |
virtio_blk_req_complete will call VirtIOBlock.complete_request() to push
data and notify guest. No functional change.
Later, this will allow dataplane to provide it's own (vring_) version.
Signed-off-by: Fam Zheng <address@hidden>
---
hw/block/virtio-blk.c | 9 ++++++++-
include/hw/virtio/virtio-blk.h | 3 +++
2 files changed, 11 insertions(+), 1 deletion(-)
diff --git a/hw/block/virtio-blk.c b/hw/block/virtio-blk.c
index b9d0d0a..aed580c 100644
--- a/hw/block/virtio-blk.c
+++ b/hw/block/virtio-blk.c
@@ -44,7 +44,8 @@ static void virtio_blk_free_request(VirtIOBlockReq *req)
}
}
-static void virtio_blk_req_complete(VirtIOBlockReq *req, int status)
+static void virtio_blk_complete_request(VirtIOBlockReq *req,
+ unsigned char status)
{
VirtIOBlock *s = req->dev;
VirtIODevice *vdev = VIRTIO_DEVICE(s);
@@ -56,6 +57,11 @@ static void virtio_blk_req_complete(VirtIOBlockReq *req, int
status)
virtio_notify(vdev, s->vq);
}
+static void virtio_blk_req_complete(VirtIOBlockReq *req, unsigned char status)
+{
+ req->dev->complete_request(req, status);
+}
+
static int virtio_blk_handle_rw_error(VirtIOBlockReq *req, int error,
bool is_read)
{
@@ -740,6 +746,7 @@ static void virtio_blk_device_realize(DeviceState *dev,
Error **errp)
s->sector_mask = (s->conf->logical_block_size / BDRV_SECTOR_SIZE) - 1;
s->vq = virtio_add_queue(vdev, 128, virtio_blk_handle_output);
+ s->complete_request = virtio_blk_complete_request;
#ifdef CONFIG_VIRTIO_BLK_DATA_PLANE
virtio_blk_data_plane_create(vdev, blk, &s->dataplane, &err);
if (err != NULL) {
diff --git a/include/hw/virtio/virtio-blk.h b/include/hw/virtio/virtio-blk.h
index 2571e96..0398f4c 100644
--- a/include/hw/virtio/virtio-blk.h
+++ b/include/hw/virtio/virtio-blk.h
@@ -117,6 +117,7 @@ struct VirtIOBlkConf
struct VirtIOBlockDataPlane;
+struct VirtIOBlockReq;
typedef struct VirtIOBlock {
VirtIODevice parent_obj;
BlockDriverState *bs;
@@ -128,6 +129,8 @@ typedef struct VirtIOBlock {
unsigned short sector_mask;
bool original_wce;
VMChangeStateEntry *change;
+ /* Function to push to vq and notify guest */
+ void (*complete_request)(struct VirtIOBlockReq *req, unsigned char status);
#ifdef CONFIG_VIRTIO_BLK_DATA_PLANE
Notifier migration_state_notifier;
struct VirtIOBlockDataPlane *dataplane;
--
2.0.0
- [Qemu-devel] [PATCH v2 0/7] virtio-blk: Unify request handling of dataplane, Fam Zheng, 2014/06/17
- [Qemu-devel] [PATCH v2 1/7] block: make bdrv_query_stats() static, Fam Zheng, 2014/06/17
- [Qemu-devel] [PATCH v2 2/7] block: acquire AioContext in qmp_query_blockstats(), Fam Zheng, 2014/06/17
- [Qemu-devel] [PATCH v2 3/7] virtio-blk: Make request completion function virtual,
Fam Zheng <=
- [Qemu-devel] [PATCH v2 4/7] virtio-blk: Export request handling functions to dataplane, Fam Zheng, 2014/06/17
- [Qemu-devel] [PATCH v2 5/7] virtio-blk: Schedule BH in the right context, Fam Zheng, 2014/06/17
- [Qemu-devel] [PATCH v2 6/7] virtio-blk: Unify {non-, }dataplane's request handlings, Fam Zheng, 2014/06/17
- [Qemu-devel] [PATCH v2 7/7] virtio-blk: Rename complete_request_early to complete_request_vring, Fam Zheng, 2014/06/17
- Re: [Qemu-devel] [PATCH v2 0/7] virtio-blk: Unify request handling of dataplane, Paolo Bonzini, 2014/06/17
- Re: [Qemu-devel] [PATCH v2 0/7] virtio-blk: Unify request handling of dataplane, Stefan Hajnoczi, 2014/06/27