On Wed, Nov 9, 2022 at 1:08 AM Eugenio Pérez <eperezma@xxxxxxxxxx> wrote:
>
> This function used to trust in v->shadow_vqs != NULL to know if it must
> start svq or not.
>
> This is not going to be valid anymore, as qemu is going to allocate svq
> unconditionally (but it will only start them conditionally).
It might be a waste of memory if we did this. Any reason for this?
Thanks
>
> Signed-off-by: Eugenio Pérez <eperezma@xxxxxxxxxx>
> ---
> hw/virtio/vhost-vdpa.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/hw/virtio/vhost-vdpa.c b/hw/virtio/vhost-vdpa.c
> index 7468e44b87..7f0ff4df5b 100644
> --- a/hw/virtio/vhost-vdpa.c
> +++ b/hw/virtio/vhost-vdpa.c
> @@ -1029,7 +1029,7 @@ static bool vhost_vdpa_svqs_start(struct vhost_dev *dev)
> Error *err = NULL;
> unsigned i;
>
> - if (!v->shadow_vqs) {
> + if (!v->shadow_vqs_enabled) {
> return true;
> }
>
> @@ -1082,7 +1082,7 @@ static void vhost_vdpa_svqs_stop(struct vhost_dev *dev)
> {
> struct vhost_vdpa *v = dev->opaque;
>
> - if (!v->shadow_vqs) {
> + if (!v->shadow_vqs_enabled) {
> return;
> }
>
> --
> 2.31.1
>
