[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 5/5] hw/arm/smmuv3: Report F_STE_FETCH fault address in corre
From: |
Auger Eric |
Subject: |
Re: [PATCH 5/5] hw/arm/smmuv3: Report F_STE_FETCH fault address in correct word position |
Date: |
Thu, 5 Dec 2019 09:39:11 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.4.0 |
Hi Simon,
On 12/4/19 2:55 PM, Simon Veith wrote:
> The smmuv3_record_event() function that generates the F_STE_FETCH error
> uses the EVT_SET_ADDR macro to record the fetch address, placing it in
> 32-bit words 4 and 5.
>
> The correct position for this address is in words 6 and 7, per the
> SMMUv3 Architecture Specification.
>
> Update the function to use the EVT_SET_ADDR2 macro instead, which is the
> macro intended for writing to these words.
>
> ref. ARM IHI 0070C, section 7.3.4.
>
> Signed-off-by: Simon Veith <address@hidden>
> Cc: Eric Auger <address@hidden>
> Cc: address@hidden
> Cc: address@hidden
> ---
> hw/arm/smmuv3.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/hw/arm/smmuv3.c b/hw/arm/smmuv3.c
> index 2d6c275..125e47d 100644
> --- a/hw/arm/smmuv3.c
> +++ b/hw/arm/smmuv3.c
> @@ -172,7 +172,7 @@ void smmuv3_record_event(SMMUv3State *s, SMMUEventInfo
> *info)
> case SMMU_EVT_F_STE_FETCH:
> EVT_SET_SSID(&evt, info->u.f_ste_fetch.ssid);
> EVT_SET_SSV(&evt, info->u.f_ste_fetch.ssv);
> - EVT_SET_ADDR(&evt, info->u.f_ste_fetch.addr);
> + EVT_SET_ADDR2(&evt, info->u.f_ste_fetch.addr);
> break;
> case SMMU_EVT_C_BAD_STE:
> EVT_SET_SSID(&evt, info->u.c_bad_ste.ssid);
>
Acked-by: Eric Auger <address@hidden>
Thanks
Eric
- Re: [PATCH 1/5] hw/arm/smmuv3: Apply address mask to linear strtab base address, (continued)
[PATCH 3/5] hw/arm/smmuv3: Align stream table base address to table size, Simon Veith, 2019/12/04
[PATCH 2/5] hw/arm/smmuv3: Check stream IDs against actual table LOG2SIZE, Simon Veith, 2019/12/04
[PATCH 4/5] hw/arm/smmuv3: Use correct bit positions in EVT_SET_ADDR2 macro, Simon Veith, 2019/12/04
[PATCH 5/5] hw/arm/smmuv3: Report F_STE_FETCH fault address in correct word position, Simon Veith, 2019/12/04
- Re: [PATCH 5/5] hw/arm/smmuv3: Report F_STE_FETCH fault address in correct word position,
Auger Eric <=