Re: [PATCH 03/26] perf vendor events: Update metrics for Broadwell DE

["Liang, Kan" <kan.liang@linux.intel.com>]

On 1/29/2022 3:09 AM, Ian Rogers wrote:
> Based on TMA_metrics-full.csv version 4.3 at 01.org:
>      https://download.01.org/perfmon/
> Events are still at version 7:
>      https://download.01.org/perfmon/BDW-DE
> Json files generated by:
>      https://github.com/intel/event-converter-for-linux-perf
> 
> This adds TopdownL1_SMT metrics to bdwde-metrics.json as
> generated by the extract-tmam.py script.
> 
> Tested:
> ...
>    6: Parse event definition strings                                  : Ok
>    7: Simple expression parser                                        : Ok
> ...
>    9: Parse perf pmu format                                           : Ok
>   10: PMU events                                                      :
>   10.1: PMU event table sanity                                        : Ok
>   10.2: PMU event map aliases                                         : Ok
>   10.3: Parsing of PMU event table metrics                            : Skip (some metrics failed)
>   10.4: Parsing of PMU event table metrics with fake PMUs             : Ok
> ...
>   68: Parse and process metrics                                       : Ok
> ...
>   88: perf stat metrics (shadow stat) test                            : Ok
>   89: perf all metricgroups test                                      : FAILED!
>   90: perf all metrics test                                           : FAILED!
>   91: perf all PMU test                                               : Ok
> ...
> The failures/skips relate to:
> event syntax error: '{arb/event=0x84,umask=0x1,metric-id=arb!3event!20x84!0umask!20x1!3/,arb/even..'
>                       \___ Cannot find PMU `arb'. Missing kernel support?
> 

ARB is an uncore unit for client platforms. Broadwell DE should be a 
server platform. It looks like an issue of TMA_metrics-full.csv.
I will check and see whether we can fix the TMA file.

Thanks,
Kan

> Signed-off-by: Ian Rogers<irogers@google.com>
> ---
>   .../arch/x86/broadwellde/bdwde-metrics.json   |  401 +++-
>   .../arch/x86/broadwellde/cache.json           | 1122 +++++-----
>   .../arch/x86/broadwellde/floating-point.json  |  222 +-
>   .../arch/x86/broadwellde/frontend.json        |  335 +--
>   .../arch/x86/broadwellde/memory.json          |  608 +++---
>   .../arch/x86/broadwellde/other.json           |   28 +-
>   .../arch/x86/broadwellde/pipeline.json        | 1892 ++++++++---------
>   .../arch/x86/broadwellde/virtual-memory.json  |  394 ++--
>   8 files changed, 2646 insertions(+), 2356 deletions(-)