[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 4/6] hw/arm/smmuv3: Align stream table base address to tab
From: |
Auger Eric |
Subject: |
Re: [PATCH v3 4/6] hw/arm/smmuv3: Align stream table base address to table size |
Date: |
Tue, 17 Dec 2019 10:40:16 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.4.0 |
Hi Simon,
On 12/16/19 4:15 PM, Simon Veith wrote:
> Per the specification, and as observed in hardware, the SMMUv3 aligns
> the SMMU_STRTAB_BASE address to the size of the table by masking out the
> respective least significant bits in the ADDR field.
>
> Apply this masking logic to our smmu_find_ste() lookup function per the
> specification.
>
> ref. ARM IHI 0070C, section 6.3.23.
>
> Signed-off-by: Simon Veith <address@hidden>
> Cc: Eric Auger <address@hidden>
> Cc: address@hidden
> Cc: address@hidden
Looks good to me.
Acked-by: Eric Auger <address@hidden>
Thanks
Eric
> ---
> Changed in v2:
>
> * Now using MAKE_64BIT_MASK()
> * Eliminated unnecessary branches by using MAX()
> * Removed unnecessary range check against DMA_ADDR_BITS
>
> hw/arm/smmuv3.c | 18 ++++++++++++++----
> 1 file changed, 14 insertions(+), 4 deletions(-)
>
> diff --git a/hw/arm/smmuv3.c b/hw/arm/smmuv3.c
> index 727558b..31ac3ca 100644
> --- a/hw/arm/smmuv3.c
> +++ b/hw/arm/smmuv3.c
> @@ -376,8 +376,9 @@ bad_ste:
> static int smmu_find_ste(SMMUv3State *s, uint32_t sid, STE *ste,
> SMMUEventInfo *event)
> {
> - dma_addr_t addr;
> + dma_addr_t addr, strtab_base;
> uint32_t log2size;
> + int strtab_size_shift;
> int ret;
>
> trace_smmuv3_find_ste(sid, s->features, s->sid_split);
> @@ -391,10 +392,16 @@ static int smmu_find_ste(SMMUv3State *s, uint32_t sid,
> STE *ste,
> }
> if (s->features & SMMU_FEATURE_2LVL_STE) {
> int l1_ste_offset, l2_ste_offset, max_l2_ste, span;
> - dma_addr_t strtab_base, l1ptr, l2ptr;
> + dma_addr_t l1ptr, l2ptr;
> STEDesc l1std;
>
> - strtab_base = s->strtab_base & SMMU_BASE_ADDR_MASK;
> + /*
> + * Align strtab base address to table size. For this purpose, assume
> it
> + * is not bounded by SMMU_IDR1_SIDSIZE.
> + */
> + strtab_size_shift = MAX(5, (int)log2size - s->sid_split - 1 + 3);
> + strtab_base = s->strtab_base & SMMU_BASE_ADDR_MASK &
> + ~MAKE_64BIT_MASK(0, strtab_size_shift);
> l1_ste_offset = sid >> s->sid_split;
> l2_ste_offset = sid & ((1 << s->sid_split) - 1);
> l1ptr = (dma_addr_t)(strtab_base + l1_ste_offset * sizeof(l1std));
> @@ -433,7 +440,10 @@ static int smmu_find_ste(SMMUv3State *s, uint32_t sid,
> STE *ste,
> }
> addr = l2ptr + l2_ste_offset * sizeof(*ste);
> } else {
> - addr = (s->strtab_base & SMMU_BASE_ADDR_MASK) + sid * sizeof(*ste);
> + strtab_size_shift = log2size + 5;
> + strtab_base = s->strtab_base & SMMU_BASE_ADDR_MASK &
> + ~MAKE_64BIT_MASK(0, strtab_size_shift);
> + addr = strtab_base + sid * sizeof(*ste);
> }
>
> if (smmu_get_ste(s, addr, ste, event)) {
>
- [PATCH v3 0/6] hw/arm/smmuv3: Correct stream ID and event address handling, Simon Veith, 2019/12/16
- [PATCH v3 2/6] hw/arm/smmuv3: Correct SMMU_BASE_ADDR_MASK value, Simon Veith, 2019/12/16
- [PATCH v3 1/6] hw/arm/smmuv3: Apply address mask to linear strtab base address, Simon Veith, 2019/12/16
- [PATCH v3 3/6] hw/arm/smmuv3: Check stream IDs against actual table LOG2SIZE, Simon Veith, 2019/12/16
- [PATCH v3 5/6] hw/arm/smmuv3: Use correct bit positions in EVT_SET_ADDR2 macro, Simon Veith, 2019/12/16
- [PATCH v3 6/6] hw/arm/smmuv3: Report F_STE_FETCH fault address in correct word position, Simon Veith, 2019/12/16
- [PATCH v3 4/6] hw/arm/smmuv3: Align stream table base address to table size, Simon Veith, 2019/12/16
- Re: [PATCH v3 4/6] hw/arm/smmuv3: Align stream table base address to table size,
Auger Eric <=
- Re: [PATCH v3 0/6] hw/arm/smmuv3: Correct stream ID and event address handling, Auger Eric, 2019/12/17