[RFC v2 6/7] hw/vfio: Remove memory_region_iommu_set_iova_ranges() call

Eric Auger posted 7 patches 5 months, 2 weeks ago
There is a newer version of this series
[RFC v2 6/7] hw/vfio: Remove memory_region_iommu_set_iova_ranges() call
Posted by Eric Auger 5 months, 2 weeks ago
As we have just removed the only implementation of
iommu_set_iova_ranges IOMMU MR callback in the virtio-iommu,
let's remove the call to the memory wrapper. Usable IOVA ranges
are now conveyed through the PCIIOMMUOps in VFIO-PCI.

Signed-off-by: Eric Auger <eric.auger@redhat.com>
---
 hw/vfio/common.c | 10 ----------
 1 file changed, 10 deletions(-)

diff --git a/hw/vfio/common.c b/hw/vfio/common.c
index f20a7b5bba..9e4c0cc95f 100644
--- a/hw/vfio/common.c
+++ b/hw/vfio/common.c
@@ -630,16 +630,6 @@ static void vfio_listener_region_add(MemoryListener *listener,
             goto fail;
         }
 
-        if (bcontainer->iova_ranges) {
-            ret = memory_region_iommu_set_iova_ranges(giommu->iommu_mr,
-                                                      bcontainer->iova_ranges,
-                                                      &err);
-            if (ret) {
-                g_free(giommu);
-                goto fail;
-            }
-        }
-
         ret = memory_region_register_iommu_notifier(section->mr, &giommu->n,
                                                     &err);
         if (ret) {
-- 
2.41.0
RE: [RFC v2 6/7] hw/vfio: Remove memory_region_iommu_set_iova_ranges() call
Posted by Duan, Zhenzhong 5 months, 2 weeks ago

>-----Original Message-----
>From: Eric Auger <eric.auger@redhat.com>
>Subject: [RFC v2 6/7] hw/vfio: Remove
>memory_region_iommu_set_iova_ranges() call
>
>As we have just removed the only implementation of
>iommu_set_iova_ranges IOMMU MR callback in the virtio-iommu,
>let's remove the call to the memory wrapper. Usable IOVA ranges
>are now conveyed through the PCIIOMMUOps in VFIO-PCI.
>
>Signed-off-by: Eric Auger <eric.auger@redhat.com>

Reviewed-by: Zhenzhong Duan <zhenzhong.duan@intel.com> 

Thanks
Zhenzhong

>---
> hw/vfio/common.c | 10 ----------
> 1 file changed, 10 deletions(-)
>
>diff --git a/hw/vfio/common.c b/hw/vfio/common.c
>index f20a7b5bba..9e4c0cc95f 100644
>--- a/hw/vfio/common.c
>+++ b/hw/vfio/common.c
>@@ -630,16 +630,6 @@ static void
>vfio_listener_region_add(MemoryListener *listener,
>             goto fail;
>         }
>
>-        if (bcontainer->iova_ranges) {
>-            ret = memory_region_iommu_set_iova_ranges(giommu-
>>iommu_mr,
>-                                                      bcontainer->iova_ranges,
>-                                                      &err);
>-            if (ret) {
>-                g_free(giommu);
>-                goto fail;
>-            }
>-        }
>-
>         ret = memory_region_register_iommu_notifier(section->mr,
>&giommu->n,
>                                                     &err);
>         if (ret) {
>--
>2.41.0