Skip to content

Commit

Permalink
vdpa_sim_blk: add support for VIRTIO_BLK_T_FLUSH
Browse files Browse the repository at this point in the history
The simulator behaves like a ramdisk, so we don't have to do
anything when a VIRTIO_BLK_T_FLUSH request is received, but it
could be useful to test driver behavior.

Let's expose the VIRTIO_BLK_F_FLUSH feature to inform the driver
that we support the flush command.

Signed-off-by: Stefano Garzarella <sgarzare@redhat.com>
Message-Id: <20220811083632.77525-4-sgarzare@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
  • Loading branch information
Stefano Garzarella authored and Michael S. Tsirkin committed Aug 11, 2022
1 parent ac926e1 commit 518083d
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions drivers/vdpa/vdpa_sim/vdpa_sim_blk.c
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@
#define DRV_LICENSE "GPL v2"

#define VDPASIM_BLK_FEATURES (VDPASIM_FEATURES | \
(1ULL << VIRTIO_BLK_F_FLUSH) | \
(1ULL << VIRTIO_BLK_F_SIZE_MAX) | \
(1ULL << VIRTIO_BLK_F_SEG_MAX) | \
(1ULL << VIRTIO_BLK_F_BLK_SIZE) | \
Expand Down Expand Up @@ -188,6 +189,10 @@ static bool vdpasim_blk_handle_req(struct vdpasim *vdpasim,
pushed += bytes;
break;

case VIRTIO_BLK_T_FLUSH:
/* nothing to do */
break;

default:
dev_dbg(&vdpasim->vdpa.dev,
"Unsupported request type %d\n", type);
Expand Down

0 comments on commit 518083d

Please sign in to comment.