Re: [PATCH v4] perf stat: Fix wrong skipping for per-die aggregation

Hi Jiri, On 1/12/2021 6:04 PM, Jiri Olsa wrote: On Tue, Jan 05, 2021 at 10:36:15AM +0800, Jin Yao wrote: SNIP diff --git a/tools/perf/util/stat.c b/tools/perf/util/stat.c index 8ce1479c98f0..a658e0ffaf2a 100644 --- a/tools/perf/util/stat.c +++ b/tools/perf/util/stat.c @@ -13,6 +13,7 @@ #inc

Re: [PATCH v4] perf stat: Fix wrong skipping for per-die aggregation

On Tue, Jan 05, 2021 at 10:36:15AM +0800, Jin Yao wrote: SNIP > diff --git a/tools/perf/util/stat.c b/tools/perf/util/stat.c > index 8ce1479c98f0..a658e0ffaf2a 100644 > --- a/tools/perf/util/stat.c > +++ b/tools/perf/util/stat.c > @@ -13,6 +13,7 @@ > #include "evlist.h" > #include "evsel.h" >

[PATCH v4] perf stat: Fix wrong skipping for per-die aggregation

Uncore becomes die-scope on Xeon Cascade Lake-AP and perf has supported --per-die aggregation yet. One issue is found in check_per_pkg() for uncore events running on AP system. On cascade Lake-AP, we have: S0-D0 S0-D1 S1-D0 S1-D1 But in check_per_pkg(), S0-D1 and S1-D1 are skipped because the ma