Re: [PATCH 2/3] soc: sifive: Add SiFive private L2 cache PMU driver

From: Conor Dooley
Date: Fri Jun 16 2023 - 06:13:33 EST


On Fri, Jun 16, 2023 at 02:32:09PM +0800, Eric Lin wrote:
> From: Greentime Hu <greentime.hu@xxxxxxxxxx>
>
> This adds SiFive private L2 cache PMU driver. User
> can use perf tool to profile by event name and event id.
>
> Example:
> $ perf stat -C 0 -e /sifive_pl2_pmu/inner_acquire_block_btot/
> -e /sifive_pl2_pmu/inner_acquire_block_ntob/
> -e /sifive_pl2_pmu/inner_acquire_block_ntot/ ls
>
> Performance counter stats for 'CPU(s) 0':
>
> 300 sifive_pl2_pmu/inner_acquire_block_btot/
> 17801 sifive_pl2_pmu/inner_acquire_block_ntob/
> 5253 sifive_pl2_pmu/inner_acquire_block_ntot/
>
> 0.088917326 seconds time elapsed
>
> $ perf stat -C 0 -e /sifive_pl2_pmu/event=0x10001/
> -e /sifive_pl2_pmu/event=0x4001/
> -e /sifive_pl2_pmu/event=0x8001/ ls
>
> Performance counter stats for 'CPU(s) 0':
>
> 251 sifive_pl2_pmu/event=0x10001/
> 2620 sifive_pl2_pmu/event=0x4001/
> 644 sifive_pl2_pmu/event=0x8001/
>
> 0.092827110 seconds time elapsed
>
> Signed-off-by: Greentime Hu <greentime.hu@xxxxxxxxxx>
> Signed-off-by: Eric Lin <eric.lin@xxxxxxxxxx>
> Reviewed-by: Zong Li <zong.li@xxxxxxxxxx>
> Reviewed-by: Nick Hu <nick.hu@xxxxxxxxxx>
> ---
> drivers/soc/sifive/Kconfig | 9 +
> drivers/soc/sifive/Makefile | 1 +
> drivers/soc/sifive/sifive_pl2.h | 20 +
> drivers/soc/sifive/sifive_pl2_cache.c | 16 +
> drivers/soc/sifive/sifive_pl2_pmu.c | 669 ++++++++++++++++++++++++++

Perf drivers should be in drivers/perf, no?

Cheers,
Conor.

Attachment: signature.asc
Description: PGP signature