Re: [PATCH v3 11/12] iommu/amd: Add IO page fault notifier handler

[Vasant Hegde <vasant.hegde@amd.com>]

On 11/7/2023 4:50 AM, Jason Gunthorpe wrote:
> On Mon, Oct 16, 2023 at 10:43:50AM +0000, Vasant Hegde wrote:
>> From: Wei Huang <wei.huang2@amd.com>
>>
>> Whenever there is a page fault IOMMU logs entry to ppr log and sends
>> interrupt to host. We have to handle the page fault and respond to IOMMU.
>>
>> Add support to validate page fault request and hook it to core iommu
>> page fault handler.
>>
>> Signed-off-by: Wei Huang <wei.huang2@amd.com>
>> Co-developed-by: Suravee Suthikulpanit <suravee.suthikulpanit@amd.com>
>> Signed-off-by: Suravee Suthikulpanit <suravee.suthikulpanit@amd.com>
>> Co-developed-by: Vasant Hegde <vasant.hegde@amd.com>
>> Signed-off-by: Vasant Hegde <vasant.hegde@amd.com>
>> ---
>>  drivers/iommu/amd/amd_iommu_types.h |   8 +++
>>  drivers/iommu/amd/ppr.c             | 100 +++++++++++++++++++++++++++-
>>  2 files changed, 107 insertions(+), 1 deletion(-)
> 
> These patches don't seem ordered quite right, the patch creating the
> SVA domain should be last, it should be organized so the SVA domain
> doesn't work then following patches fix it.

We support PASID without PRI support. Hence we have SVA related patches first
and then PRI enablement series.

-Vasant