On Wed, Nov 16, 2022 at 11:06 PM Eugenio Pérez <eperezma@xxxxxxxxxx> wrote:
>
> SVQ may run or not in a device depending on runtime conditions (for
> example, if the device can move CVQ to its own group or not).
>
> Allocate the SVQ array unconditionally at startup, since its hard to
> move this allocation elsewhere.
>
> Signed-off-by: Eugenio Pérez <eperezma@xxxxxxxxxx>
Acked-by: Jason Wang <jasowang@xxxxxxxxxx>
Thanks
> ---
> hw/virtio/vhost-vdpa.c | 4 ----
> 1 file changed, 4 deletions(-)
>
> diff --git a/hw/virtio/vhost-vdpa.c b/hw/virtio/vhost-vdpa.c
> index 146f0dcb40..23efb8f49d 100644
> --- a/hw/virtio/vhost-vdpa.c
> +++ b/hw/virtio/vhost-vdpa.c
> @@ -547,10 +547,6 @@ static void vhost_vdpa_svq_cleanup(struct vhost_dev *dev)
> struct vhost_vdpa *v = dev->opaque;
> size_t idx;
>
> - if (!v->shadow_vqs) {
> - return;
> - }
> -
> for (idx = 0; idx < v->shadow_vqs->len; ++idx) {
> vhost_svq_stop(g_ptr_array_index(v->shadow_vqs, idx));
> }
> --
> 2.31.1
>
