Re: [PATCH 0/4] iommu/amd: SVA Support (part 2) - deprecate iommu_v2 module

[Jason Gunthorpe <jgg@ziepe.ca>]

On Tue, Aug 15, 2023 at 10:32:51AM +0000, Vasant Hegde wrote:
> This is part 2 of the 4-part series to introduce Share Virtual Address
> (SVA) support. This series removes iommu_v2 module support.
> 
> AMD GPU driver which was the only in-kernel user of iommu_v2 module is
> removing dependency on iommu_v2 module [1].
> 
> [1] https://patchwork.freedesktop.org/series/122124/
> 
> Also subsequent series will add SVA spuport in AMD IOMMU driver. Device
> drivers are expected to use common SVA framework to enable device
> PASID/PRI features.
> 
> Hence this series removes iommu_v2 module and the functions used to support
> iommu_v2 modue. Note that there are certain functions (ex: PPR interrupt
> handling code) which will be used in subsequent series for SVA support.
> Hence not removed those functions.
> 
> This patch series is based on top of SVA Part 1 [1] :
>   https://lore.kernel.org/linux-iommu/20230815102202.565012-1-vasant.hegde@amd.com/T/#t
> 
>   Also to build kernel with this series either we need above AMD GPU
>   related changes [1] -OR- compile with kernel config 'CONFIG_HSA_AMD=n'.

Why did you order it this way? Wouldn't part 1 be nicer if it doesn't
have to worry about this code? Especially since part 1 is now not
changing this code?

Jason