On Mon, Mar 24, 2025 at 3:00 PM Sahil Siddiq <icegambi...@gmail.com> wrote:
>
> Validate transport device features required for utilizing packed SVQ
> that both guests can use with the SVQ and SVQs can use with vdpa.
>
> Signed-off-by: Sahil Siddiq <sahil...@proton.me>
> ---
> Changes from v4 -> v5:
> - Split from commit #2 in v4.
>
> hw/virtio/vhost-shadow-virtqueue.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/hw/virtio/vhost-shadow-virtqueue.c
> b/hw/virtio/vhost-shadow-virtqueue.c
> index 8430b3c94a..035ab1e66f 100644
> --- a/hw/virtio/vhost-shadow-virtqueue.c
> +++ b/hw/virtio/vhost-shadow-virtqueue.c
> @@ -33,6 +33,9 @@ bool vhost_svq_valid_features(uint64_t features, Error
> **errp)
> ++b) {
> switch (b) {
> case VIRTIO_F_ANY_LAYOUT:
> + case VIRTIO_F_RING_PACKED:
> + case VIRTIO_F_RING_RESET:
> + case VIRTIO_RING_F_INDIRECT_DESC:
This should only enable _F_RING_PACKED, there is no code supporting
either reset or indirect descriptors.
> case VIRTIO_RING_F_EVENT_IDX:
> continue;
>
> --
> 2.48.1
>
