From mboxrd@z Thu Jan 1 00:00:00 1970 From: will.deacon@arm.com (Will Deacon) Date: Mon, 26 Feb 2018 18:05:05 +0000 Subject: [PATCH v2 3/4] iommu/arm-smmu-v3: Support 52-bit physical address In-Reply-To: <9d2a2eb4527987aec520e112163a178d92fc946b.1512038236.git.robin.murphy@arm.com> References: <9d2a2eb4527987aec520e112163a178d92fc946b.1512038236.git.robin.murphy@arm.com> Message-ID: <20180226180504.GD26147@arm.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Thu, Dec 14, 2017 at 04:58:52PM +0000, Robin Murphy wrote: > Implement SMMUv3.1 support for 52-bit physical addresses. Since a 52-bit > OAS implies 64KB translation granule support, permitting level 1 block > entries there is simple, and the rest is just extending address fields. > > Tested-by: Nate Watterson > Signed-off-by: Robin Murphy > --- > > v2: No change > > drivers/iommu/arm-smmu-v3.c | 35 ++++++++++++++++++++--------------- > 1 file changed, 20 insertions(+), 15 deletions(-) > > diff --git a/drivers/iommu/arm-smmu-v3.c b/drivers/iommu/arm-smmu-v3.c > index 52cad776b31b..c9c4e6132e27 100644 > --- a/drivers/iommu/arm-smmu-v3.c > +++ b/drivers/iommu/arm-smmu-v3.c > @@ -101,6 +101,7 @@ > #define IDR5_OAS_42_BIT (3 << IDR5_OAS_SHIFT) > #define IDR5_OAS_44_BIT (4 << IDR5_OAS_SHIFT) > #define IDR5_OAS_48_BIT (5 << IDR5_OAS_SHIFT) > +#define IDR5_OAS_52_BIT (6 << IDR5_OAS_SHIFT) > > #define ARM_SMMU_CR0 0x20 > #define CR0_CMDQEN (1 << 3) > @@ -159,7 +160,7 @@ > > #define ARM_SMMU_STRTAB_BASE 0x80 > #define STRTAB_BASE_RA (1UL << 62) > -#define STRTAB_BASE_ADDR_MASK GENMASK_ULL(47, 6) > +#define STRTAB_BASE_ADDR_MASK GENMASK_ULL(51, 6) > > #define ARM_SMMU_STRTAB_BASE_CFG 0x88 > #define STRTAB_BASE_CFG_LOG2SIZE_SHIFT 0 > @@ -190,7 +191,7 @@ > #define ARM_SMMU_PRIQ_IRQ_CFG2 0xdc > > /* Common MSI config fields */ > -#define MSI_CFG0_ADDR_MASK GENMASK_ULL(47, 2) > +#define MSI_CFG0_ADDR_MASK GENMASK_ULL(51, 2) > #define MSI_CFG2_SH_SHIFT 4 > #define MSI_CFG2_SH_NSH (0UL << MSI_CFG2_SH_SHIFT) > #define MSI_CFG2_SH_OSH (2UL << MSI_CFG2_SH_SHIFT) > @@ -206,7 +207,7 @@ > Q_IDX(q, p) * (q)->ent_dwords) > > #define Q_BASE_RWA (1UL << 62) > -#define Q_BASE_ADDR_MASK GENMASK_ULL(47, 5) > +#define Q_BASE_ADDR_MASK GENMASK_ULL(51, 5) > #define Q_BASE_LOG2SIZE_SHIFT 0 > #define Q_BASE_LOG2SIZE_MASK 0x1fUL > > @@ -223,7 +224,7 @@ > #define STRTAB_L1_DESC_DWORDS 1 > #define STRTAB_L1_DESC_SPAN_SHIFT 0 > #define STRTAB_L1_DESC_SPAN_MASK 0x1fUL > -#define STRTAB_L1_DESC_L2PTR_MASK GENMASK_ULL(47, 6) > +#define STRTAB_L1_DESC_L2PTR_MASK GENMASK_ULL(51, 6) > > #define STRTAB_STE_DWORDS 8 > #define STRTAB_STE_0_V (1UL << 0) > @@ -236,7 +237,7 @@ > > #define STRTAB_STE_0_S1FMT_SHIFT 4 > #define STRTAB_STE_0_S1FMT_LINEAR (0UL << STRTAB_STE_0_S1FMT_SHIFT) > -#define STRTAB_STE_0_S1CTXPTR_MASK GENMASK_ULL(47, 6) > +#define STRTAB_STE_0_S1CTXPTR_MASK GENMASK_ULL(51, 6) > #define STRTAB_STE_0_S1CDMAX_SHIFT 59 > #define STRTAB_STE_0_S1CDMAX_MASK 0x1fUL > > @@ -274,7 +275,7 @@ > #define STRTAB_STE_2_S2PTW (1UL << 54) > #define STRTAB_STE_2_S2R (1UL << 58) > > -#define STRTAB_STE_3_S2TTB_MASK GENMASK_ULL(47, 4) > +#define STRTAB_STE_3_S2TTB_MASK GENMASK_ULL(51, 4) > > /* Context descriptor (stage-1 only) */ > #define CTXDESC_CD_DWORDS 8 > @@ -320,7 +321,7 @@ > #define CTXDESC_CD_0_ASID_SHIFT 48 > #define CTXDESC_CD_0_ASID_MASK 0xffffUL > > -#define CTXDESC_CD_1_TTB0_MASK GENMASK_ULL(47, 4) > +#define CTXDESC_CD_1_TTB0_MASK GENMASK_ULL(51, 4) > > #define CTXDESC_CD_3_MAIR_SHIFT 0 > > @@ -357,7 +358,7 @@ > #define CMDQ_TLBI_0_ASID_SHIFT 48 > #define CMDQ_TLBI_1_LEAF (1UL << 0) > #define CMDQ_TLBI_1_VA_MASK GENMASK_ULL(63, 12) > -#define CMDQ_TLBI_1_IPA_MASK GENMASK_ULL(47, 12) > +#define CMDQ_TLBI_1_IPA_MASK GENMASK_ULL(51, 12) > > #define CMDQ_PRI_0_SSID_SHIFT 12 > #define CMDQ_PRI_0_SSID_MASK 0xfffffUL > @@ -380,7 +381,7 @@ > #define CMDQ_SYNC_0_MSIATTR_OIWB (0xfUL << CMDQ_SYNC_0_MSIATTR_SHIFT) > #define CMDQ_SYNC_0_MSIDATA_SHIFT 32 > #define CMDQ_SYNC_0_MSIDATA_MASK 0xffffffffUL > -#define CMDQ_SYNC_1_MSIADDR_MASK GENMASK_ULL(47, 2) > +#define CMDQ_SYNC_1_MSIADDR_MASK GENMASK_ULL(51, 2) > > /* Event queue */ > #define EVTQ_ENT_DWORDS 4 > @@ -1686,7 +1687,7 @@ static int arm_smmu_domain_finalise(struct iommu_domain *domain) > return -ENOMEM; > > domain->pgsize_bitmap = pgtbl_cfg.pgsize_bitmap; > - domain->geometry.aperture_end = (1UL << ias) - 1; > + domain->geometry.aperture_end = (1UL << pgtbl_cfg.ias) - 1; Ah good, you fixed this :) Will