[PATCH v8 00/14] Update SMMUv3 to the modern iommu API (part 2b/3)

Nicolin Chen nicolinc at nvidia.com
Tue Jun 4 01:45:01 PDT 2024


On Mon, Jun 03, 2024 at 09:15:45PM -0300, Jason Gunthorpe wrote:

> This is on github:
> https://github.com/jgunthorpe/linux/commits/smmuv3_newapi
> 
> v8:
>  - Rebase on v6.10-rc2

It seems that the branch above isn't updated yet from 6.9-rc7 :)

FWIW, it's missing a patch from Robin fixing a regression:
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?h=v6.10-rc2&id=8b80549f1bc6

>  - Make arm_smmu_sva_domain_alloc NULL when SVA is disabled so the core
>    code sees a NULL function pointer
>  - Update comments around arm_smmu_attach_prepare()
>  - Rename struct attach_state -> arm_smmu_attach_state and document
>    better, include more common function paramters in state
>  - Consistently use ats_enabled everywhere, replacing disable_ats in state
>  - Move the note about ATS bypass/abort to arm_smmu_attach_prepare()
>  - Remove temporary cd_table.in_ste check in arm_smmu_sva_set_dev_pasid()
>  - Improve comments and clarity of logic in arm_smmu_attach_commit()
>  - Shorten arm_smmu_mmu_notifier_free()
>  - Fix domain -> sid_domain typo in arm_smmu_remove_dev_pasid()
 
> Cc: Nicolin Chen <nicolinc at nvidia.com>

With that, I retested with SVA sanity running on S1DSS.SSID0 and
S1DSS.BYPASS modes. Both work well. The kunit test passes also.

So, my previous Tested-by still stands.

Thanks
Nicolin

> Cc: Michael Shavit <mshavit at google.com>
> Cc: Jean-Philippe Brucker <jean-philippe at linaro.org>
> Cc: Shameerali Kolothum Thodi <shameerali.kolothum.thodi at huawei.com>
> Cc: Eric Auger <eric.auger at redhat.com>
> Cc: Moritz Fischer <mdf at kernel.org>
> Signed-off-by: Jason Gunthorpe <jgg at nvidia.com>




More information about the linux-arm-kernel mailing list