On Fri, May 22, 2020 at 1:19 PM Stefan Hajnoczi <stefa...@redhat.com> wrote:
>
> Let vhost-scsi and vhost-user-scsi device backends determine whether
> VIRTIO 1.0 and packed virtqueues are supported. It doesn't make sense to
> handle these feature bits in QEMU since the device backend needs to
> support them if we want to use them.
>
> Signed-off-by: Stefan Hajnoczi <stefa...@redhat.com>
Reviewed-by: Raphael Norwitz <raphael.norw...@nutanix.com>
> ---
> hw/scsi/vhost-scsi.c | 2 ++
> hw/scsi/vhost-user-scsi.c | 2 ++
> 2 files changed, 4 insertions(+)
>
> diff --git a/hw/scsi/vhost-scsi.c b/hw/scsi/vhost-scsi.c
> index c1b012aea4..a7fb788af5 100644
> --- a/hw/scsi/vhost-scsi.c
> +++ b/hw/scsi/vhost-scsi.c
> @@ -34,6 +34,8 @@
>
> /* Features supported by host kernel. */
> static const int kernel_feature_bits[] = {
> + VIRTIO_F_VERSION_1,
> + VIRTIO_F_RING_PACKED,
> VIRTIO_F_NOTIFY_ON_EMPTY,
> VIRTIO_RING_F_INDIRECT_DESC,
> VIRTIO_RING_F_EVENT_IDX,
> diff --git a/hw/scsi/vhost-user-scsi.c b/hw/scsi/vhost-user-scsi.c
> index cbb5d97599..6aa0d5ded2 100644
> --- a/hw/scsi/vhost-user-scsi.c
> +++ b/hw/scsi/vhost-user-scsi.c
> @@ -32,6 +32,8 @@
>
> /* Features supported by the host application */
> static const int user_feature_bits[] = {
> + VIRTIO_F_VERSION_1,
> + VIRTIO_F_RING_PACKED,
> VIRTIO_F_NOTIFY_ON_EMPTY,
> VIRTIO_RING_F_INDIRECT_DESC,
> VIRTIO_RING_F_EVENT_IDX,
> --
> 2.25.3
>
