Message ID | 20230915132051.2646055-1-mshavit@google.com (mailing list archive) |
---|---|
Headers | show |
Series | Refactor the SMMU's CD table ownership | expand |
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 > > Changes in v8: > - Rebased off of 6.6-rc1 > - Drive-by update of the "Move ctx_desc out of s1_cfg" commit message to be less > vague. > - Link to v7: https://lore.kernel.org/linux-iommu/20230831174536.103472-1-mshavit@google.com/ Hmm. I recall that I gave Tested-by to v7, yet some of the patches in this v8 don't include that. Anyway, I retested with this v8: Tested-by: Nicolin Chen <nicolinc@nvidia.com>
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
On Wed, Oct 11, 2023 at 08:48:48PM -0300, Jason Gunthorpe wrote: > 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? On my list to look at today! Will
On Fri, 15 Sep 2023 21:17:31 +0800, Michael Shavit wrote: > 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. > > [...] Applied to will (for-joerg/arm-smmu/updates), thanks! [1/9] iommu/arm-smmu-v3: Move ctx_desc out of s1_cfg https://git.kernel.org/will/c/987a878e09c6 [2/9] iommu/arm-smmu-v3: Replace s1_cfg with cdtab_cfg https://git.kernel.org/will/c/1f8588834016 [3/9] iommu/arm-smmu-v3: Encapsulate ctx_desc_cfg init in alloc_cd_tables https://git.kernel.org/will/c/e3aad74c51a7 [4/9] iommu/arm-smmu-v3: move stall_enabled to the cd table https://git.kernel.org/will/c/1228cc509fc6 [5/9] iommu/arm-smmu-v3: Refactor write_ctx_desc https://git.kernel.org/will/c/24503148c545 [6/9] iommu/arm-smmu-v3: Move CD table to arm_smmu_master https://git.kernel.org/will/c/10e4968cd511 [7/9] iommu/arm-smmu-v3: Cleanup arm_smmu_domain_finalise https://git.kernel.org/will/c/5e14313df2c8 [8/9] iommu/arm-smmu-v3: Update comment about STE liveness https://git.kernel.org/will/c/6032f58498b7 [9/9] iommu/arm-smmu-v3: Rename cdcfg to cd_table https://git.kernel.org/will/c/475918e9c4eb Cheers,