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.au...@redhat.com>


Reviewed-by: Cédric Le Goater <c...@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;
  }


Reply via email to