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

Ilkka Koskinen posted 2 patches 2 weeks ago
.../util/arm-spe-decoder/arm-spe-decoder.h    |  9 ++
tools/perf/util/arm-spe.c                     | 86 ++++++++++++++++---
2 files changed, 83 insertions(+), 12 deletions(-)
[PATCH v5 0/2] [PATCH v4 0/2] perf arm-spe: Add support for SPE Data Source packet on AmpereOne
Posted by Ilkka Koskinen 2 weeks ago
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(-)

-- 
2.47.0
Re: [PATCH v5 0/2] [PATCH v4 0/2] perf arm-spe: Add support for SPE Data Source packet on AmpereOne
Posted by James Clark 1 week, 3 days ago

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>