On Thu, 11 Jun 2020 at 17:16, Eric Auger <eric.au...@redhat.com> wrote:
>
> Let's introduce an helper for S1 IOVA range invalidation.
> This will be used for NH_VA and NH_VAA commands. It decodes
> the same fields, trace, calls the UNMAP notifiers and
> invalidate the corresponding IOTLB entries.
>
> At the moment, we do not support 3.2 range invalidation yet.
> So it reduces to a single IOVA invalidation.
>
> Signed-off-by: Eric Auger <eric.au...@redhat.com>
> ---
> hw/arm/smmuv3.c | 36 +++++++++++++++++-------------------
> hw/arm/trace-events | 3 +--
> 2 files changed, 18 insertions(+), 21 deletions(-)
>
> diff --git a/hw/arm/smmuv3.c b/hw/arm/smmuv3.c
> index 931a2b6872..4eda16be7b 100644
> --- a/hw/arm/smmuv3.c
> +++ b/hw/arm/smmuv3.c
> @@ -840,6 +840,22 @@ static void smmuv3_inv_notifiers_iova(SMMUState *s, int
> asid, dma_addr_t iova)
> }
> }
>
> +static void smmuv3_s1_range_inval(SMMUState *s, Cmd *cmd)
> +{
> + dma_addr_t addr = CMD_ADDR(cmd);
> + uint8_t type = CMD_TYPE(cmd);
> + uint16_t vmid = CMD_VMID(cmd);
> + bool leaf = CMD_LEAF(cmd);
We used to only read the leaf bit for
CMD_TLBI_NH_VA, but now we read it also for
CMD_TLBI_NH_VAA. That's OK because:
* the leaf bit really does exist on both commands
* the only thing we do with it is print it in the trace
message, so this isn't fixing a bug
But you could mention it in the commit message.
Otherwise
Reviewed-by: Peter Maydell <peter.mayd...@linaro.org>
thanks
-- PMM
