Hi Joerg,

On 2/14/22 10:14 PM, Joerg Roedel wrote:
On Mon, Feb 14, 2022 at 09:55:28AM +0800, Lu Baolu wrote:
v3:
  - Remove ops check when dev_iommu_ops() is used.
  - This version of series is available on github:
    https://github.com/LuBaolu/intel-iommu/commits/iommu-domain-ops-v3

Lu Baolu (10):
   iommu/vt-d: Remove guest pasid related callbacks
   iommu: Remove guest pasid related interfaces and definitions
   iommu/vt-d: Remove aux-domain related callbacks
   iommu: Remove aux-domain related interfaces and iommu_ops
   iommu: Remove apply_resv_region
   drm/nouveau/device: Get right pgsize_bitmap of iommu_domain
   iommu: Use right way to retrieve iommu_ops
   iommu: Remove unused argument in is_attach_deferred
   iommu: Use dev_iommu_ops() helper
   iommu: Split struct iommu_ops

Really cool, thanks for doing this, Baolu! I am currently running some
tests on this series to make sure there are no build breakages. And I
agree with Jason that:

        iommu: Use right way to retrieve iommu_ops and
        iommu: Use dev_iommu_ops() helper

can be combined into one patch. Splitting iommu_ops was something I have
been thinking about for some time, so thanks again for doing this.

You are welcome!

Do you want me to send a new version with below changes:

- Remove WARN_ON() in dev_iommu_ops();
- Merge above two patches.

Or, you can change above yourself?


Regards,

        Joerg

Best regards,
baolu
_______________________________________________
iommu mailing list
iommu@lists.linux-foundation.org
https://lists.linuxfoundation.org/mailman/listinfo/iommu

Reply via email to