Re: [PATCH v8 0/9] Refactor the SMMU's CD table ownership

From: Jason Gunthorpe
Date: Wed Oct 11 2023 - 19:49:00 EST


On Fri, Sep 15, 2023 at 09:17:31PM +0800, Michael Shavit wrote:
>
> Hi all,
>
> This series refactors stage 1 domains so that they describe a single CD
> entry. These entries are now inserted into a CD table that is owned by
> the arm_smmu_master instead of the domain.
> This is conceptually cleaner and unblocks other features, such as
> attaching domains with PASID (for unmanaged/dma domains).
>
> This patch series was originally part of a larger patch series that
> implemented the set_dev_pasid callback for non-SVA domains but is now
> split into a distinct series.
>
> This patch series is also available on gerrit with Jean's SMMU test
> engine patches cherry-picked on top for testing:
> https://linux-review.googlesource.com/c/linux/kernel/git/torvalds/linux/+/24742/16
>
> Thanks,
> Michael Shavit
>
> Michael Shavit (9):
> iommu/arm-smmu-v3: Move ctx_desc out of s1_cfg
> iommu/arm-smmu-v3: Replace s1_cfg with cdtab_cfg
> iommu/arm-smmu-v3: Encapsulate ctx_desc_cfg init in alloc_cd_tables
> iommu/arm-smmu-v3: move stall_enabled to the cd table
> iommu/arm-smmu-v3: Refactor write_ctx_desc
> iommu/arm-smmu-v3: Move CD table to arm_smmu_master
> iommu/arm-smmu-v3: Cleanup arm_smmu_domain_finalise
> iommu/arm-smmu-v3: Update comment about STE liveness
> iommu/arm-smmu-v3: Rename cdcfg to cd_table

Will, can you take this please?

Thanks,
Jason