Re: [PATCH 4/6] virtio-iommu: Remove the end point on detach

[Cédric Le Goater]

On 7/16/24 11:45, Eric Auger wrote:
> We currently miss the removal of the endpoint in case of detach.
>
> Signed-off-by: Eric Auger <eric.auger@redhat.com>


Reviewed-by: Cédric Le Goater <clg@redhat.com>

Thanks,

C.


> ---
>   hw/virtio/virtio-iommu.c | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/hw/virtio/virtio-iommu.c b/hw/virtio/virtio-iommu.c
> index 2de41ab412..440dfa6e92 100644
> --- a/hw/virtio/virtio-iommu.c
> +++ b/hw/virtio/virtio-iommu.c
> @@ -786,6 +786,7 @@ static int virtio_iommu_detach(VirtIOIOMMU *s,
>       if (QLIST_EMPTY(&domain->endpoint_list)) {
>           g_tree_remove(s->domains, GUINT_TO_POINTER(domain->id));
>       }
> +    g_tree_remove(s->endpoints, GUINT_TO_POINTER(ep_id));
>       return VIRTIO_IOMMU_S_OK;
>   }
>