Re: [PATCH v5 0/2] [PATCH v4 0/2] perf arm-spe: Add support for SPE Data Source packet on AmpereOne

[James Clark <james.clark@linaro.org>]

On 08/11/2024 8:29 pm, Ilkka Koskinen wrote:
> v1:
> 	* https://lore.kernel.org/all/20241024233035.7979-1-ilkka@os.amperecomputing.com/
> 
> v2:
> 	* Doesn't use read_cpuid_implementor() anymore as that was broken and
> 	  unnecessary.
> 	* Convert AmpereOne source field to matching common source fields to
> 	  avoid duplicating the code.
> 	* Rebased on top of perf-tools-next/perf-tools-next (ba993e5ada1d)
> 	* https://lore.kernel.org/all/20241031213533.11148-1-ilkka@os.amperecomputing.com/
> 
> v3:
> 	* Changed source mapping to simple switch statement
> 	* Dropped is_xyz() stuff
> 	* Added table to map midr to data source decoding function
> 	* https://lore.kernel.org/all/20241106193740.6159-1-ilkka@os.amperecomputing.com/
> 
> v4:
> 	* Split midr/decoding function table
> 	* Made AmpereOne DS decoding function to ignore unknown sources
> 	* https://lore.kernel.org/all/20241108010911.58412-1-ilkka@os.amperecomputing.com/
> 
> v5:
> 	* Moved data_source_handles[] to fix the build issue
> 
> Ilkka Koskinen (2):
>    perf arm-spe: Prepare for adding data source packet implementations
>      for other cores
>    perf arm-spe: Add support for SPE Data Source packet on AmpereOne
> 
>   .../util/arm-spe-decoder/arm-spe-decoder.h    |  9 ++
>   tools/perf/util/arm-spe.c                     | 86 ++++++++++++++++---
>   2 files changed, 83 insertions(+), 12 deletions(-)
> 

Reviewed-by: James Clark <james.clark@linaro.org>