diff options
author | Stefan Hajnoczi <stefanha@redhat.com> | 2024-02-06 14:06:09 -0500 |
---|---|---|
committer | Kevin Wolf <kwolf@redhat.com> | 2024-02-07 14:44:19 +0100 |
commit | 8802f4f4e3a3e7e0ffc778cab853b14349574a7b (patch) | |
tree | 2611c3eeeca020e2149a4072880efa664d38d576 | |
parent | f2eea93c6b410efccc18f4b47090b4a4e842afbd (diff) | |
download | qemu-8802f4f4e3a3e7e0ffc778cab853b14349574a7b.zip qemu-8802f4f4e3a3e7e0ffc778cab853b14349574a7b.tar.gz qemu-8802f4f4e3a3e7e0ffc778cab853b14349574a7b.tar.bz2 |
virtio-blk: declare VirtIOBlock::rq with a type
The VirtIOBlock::rq field has had the type void * since its introduction
in commit 869a5c6df19a ("Stop VM on error in virtio-blk. (Gleb
Natapov)").
Perhaps this was done to avoid the forward declaration of
VirtIOBlockReq.
Hanna Czenczek <hreitz@redhat.com> pointed out the missing type. Specify
the actual type because there is no need to use void * here.
Suggested-by: Hanna Czenczek <hreitz@redhat.com>
Reviewed-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
Reviewed-by: Hanna Czenczek <hreitz@redhat.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
Message-ID: <20240206190610.107963-5-stefanha@redhat.com>
Reviewed-by: Kevin Wolf <kwolf@redhat.com>
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
-rw-r--r-- | include/hw/virtio/virtio-blk.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/hw/virtio/virtio-blk.h b/include/hw/virtio/virtio-blk.h index 833a9a3..5c14110 100644 --- a/include/hw/virtio/virtio-blk.h +++ b/include/hw/virtio/virtio-blk.h @@ -55,7 +55,7 @@ struct VirtIOBlock { VirtIODevice parent_obj; BlockBackend *blk; QemuMutex rq_lock; - void *rq; /* protected by rq_lock */ + struct VirtIOBlockReq *rq; /* protected by rq_lock */ VirtIOBlkConf conf; unsigned short sector_mask; bool original_wce; |