[PATCH] coresight: etm4x: Fix save/restore during cpu idle

Mathieu Poirier mathieu.poirier at linaro.org
Mon Jul 6 12:51:17 EDT 2020


On Wed, 1 Jul 2020 at 17:17, Suzuki K Poulose <suzuki.poulose at arm.com> wrote:
>
> The ETM state save/restore incorrectly reads/writes some of the 64bit
> registers (e.g, address comparators, vmid/cid comparators etc.) using
> 32bit accesses. Ensure we use the appropriate width accessors for
> the registers.
>
> Fixes: f188b5e76aae ("coresight: etm4x: Save/restore state across CPU low power states")
> Cc: Mathieu Poirier <mathieu.poirier at linaro.org>
> Cc: Mike Leach <mike.leach at linaro.org>
> Signed-off-by: Suzuki K Poulose <suzuki.poulose at arm.com>
> ---
>  drivers/hwtracing/coresight/coresight-etm4x.c | 16 ++++++++--------
>  drivers/hwtracing/coresight/coresight-etm4x.h |  2 +-
>  2 files changed, 9 insertions(+), 9 deletions(-)

Applied - thanks.
Mathieu

>
> diff --git a/drivers/hwtracing/coresight/coresight-etm4x.c b/drivers/hwtracing/coresight/coresight-etm4x.c
> index 82fc2fab072a..be990457a8ea 100644
> --- a/drivers/hwtracing/coresight/coresight-etm4x.c
> +++ b/drivers/hwtracing/coresight/coresight-etm4x.c
> @@ -1206,8 +1206,8 @@ static int etm4_cpu_save(struct etmv4_drvdata *drvdata)
>         }
>
>         for (i = 0; i < drvdata->nr_addr_cmp * 2; i++) {
> -               state->trcacvr[i] = readl(drvdata->base + TRCACVRn(i));
> -               state->trcacatr[i] = readl(drvdata->base + TRCACATRn(i));
> +               state->trcacvr[i] = readq(drvdata->base + TRCACVRn(i));
> +               state->trcacatr[i] = readq(drvdata->base + TRCACATRn(i));
>         }
>
>         /*
> @@ -1218,10 +1218,10 @@ static int etm4_cpu_save(struct etmv4_drvdata *drvdata)
>          */
>
>         for (i = 0; i < drvdata->numcidc; i++)
> -               state->trccidcvr[i] = readl(drvdata->base + TRCCIDCVRn(i));
> +               state->trccidcvr[i] = readq(drvdata->base + TRCCIDCVRn(i));
>
>         for (i = 0; i < drvdata->numvmidc; i++)
> -               state->trcvmidcvr[i] = readl(drvdata->base + TRCVMIDCVRn(i));
> +               state->trcvmidcvr[i] = readq(drvdata->base + TRCVMIDCVRn(i));
>
>         state->trccidcctlr0 = readl(drvdata->base + TRCCIDCCTLR0);
>         state->trccidcctlr1 = readl(drvdata->base + TRCCIDCCTLR1);
> @@ -1319,18 +1319,18 @@ static void etm4_cpu_restore(struct etmv4_drvdata *drvdata)
>         }
>
>         for (i = 0; i < drvdata->nr_addr_cmp * 2; i++) {
> -               writel_relaxed(state->trcacvr[i],
> +               writeq_relaxed(state->trcacvr[i],
>                                drvdata->base + TRCACVRn(i));
> -               writel_relaxed(state->trcacatr[i],
> +               writeq_relaxed(state->trcacatr[i],
>                                drvdata->base + TRCACATRn(i));
>         }
>
>         for (i = 0; i < drvdata->numcidc; i++)
> -               writel_relaxed(state->trccidcvr[i],
> +               writeq_relaxed(state->trccidcvr[i],
>                                drvdata->base + TRCCIDCVRn(i));
>
>         for (i = 0; i < drvdata->numvmidc; i++)
> -               writel_relaxed(state->trcvmidcvr[i],
> +               writeq_relaxed(state->trcvmidcvr[i],
>                                drvdata->base + TRCVMIDCVRn(i));
>
>         writel_relaxed(state->trccidcctlr0, drvdata->base + TRCCIDCCTLR0);
> diff --git a/drivers/hwtracing/coresight/coresight-etm4x.h b/drivers/hwtracing/coresight/coresight-etm4x.h
> index 7da022e87218..b8283e1d6d88 100644
> --- a/drivers/hwtracing/coresight/coresight-etm4x.h
> +++ b/drivers/hwtracing/coresight/coresight-etm4x.h
> @@ -334,7 +334,7 @@ struct etmv4_save_state {
>         u64     trcacvr[ETM_MAX_SINGLE_ADDR_CMP];
>         u64     trcacatr[ETM_MAX_SINGLE_ADDR_CMP];
>         u64     trccidcvr[ETMv4_MAX_CTXID_CMP];
> -       u32     trcvmidcvr[ETM_MAX_VMID_CMP];
> +       u64     trcvmidcvr[ETM_MAX_VMID_CMP];
>         u32     trccidcctlr0;
>         u32     trccidcctlr1;
>         u32     trcvmidcctlr0;
> --
> 2.24.1
>



More information about the linux-arm-kernel mailing list