[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
RE: [RFC v2 6/7] hw/vfio: Remove memory_region_iommu_set_iova_ranges() c
From: |
Duan, Zhenzhong |
Subject: |
RE: [RFC v2 6/7] hw/vfio: Remove memory_region_iommu_set_iova_ranges() call |
Date: |
Tue, 11 Jun 2024 03:23:27 +0000 |
>-----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
- [RFC v2 3/7] HostIOMMUDevice: Introduce get_iova_ranges callback, (continued)
- [RFC v2 3/7] HostIOMMUDevice: Introduce get_iova_ranges callback, Eric Auger, 2024/06/07
- [RFC v2 4/7] virtio-iommu: Compute host reserved regions, Eric Auger, 2024/06/07
- [RFC v2 5/7] virtio-iommu: Remove the implementation of iommu_set_iova_range, Eric Auger, 2024/06/07
- [RFC v2 6/7] hw/vfio: Remove memory_region_iommu_set_iova_ranges() call, Eric Auger, 2024/06/07
- RE: [RFC v2 6/7] hw/vfio: Remove memory_region_iommu_set_iova_ranges() call,
Duan, Zhenzhong <=
- [RFC v2 7/7] memory: Remove IOMMU MR iommu_set_iova_range API, Eric Auger, 2024/06/07