[PATCH v3 0/3] Remove bad uncore filter/events

Ian Rogers posted 3 patches 3 years, 8 months ago
.../arch/x86/broadwellde/uncore-cache.json    | 97 -------------------
.../arch/x86/broadwellde/uncore-other.json    | 13 ---
.../arch/x86/ivytown/uncore-cache.json        | 90 -----------------
.../arch/x86/ivytown/uncore-interconnect.json |  1 -
.../arch/x86/ivytown/uncore-other.json        | 13 ---
.../arch/x86/ivytown/uncore-power.json        | 19 ----
.../arch/x86/jaketown/uncore-cache.json       | 30 ------
.../arch/x86/jaketown/uncore-other.json       | 13 ---
.../arch/x86/jaketown/uncore-power.json       | 11 ---
9 files changed, 287 deletions(-)
[PATCH v3 0/3] Remove bad uncore filter/events
Posted by Ian Rogers 3 years, 8 months ago
The event converter scripts at:
https://github.com/intel/event-converter-for-linux-perf
passes Filter values from data on 01.org that is bogus in a perf command
line and can cause perf to infinitely recurse in parse events. Remove
such events or filter using the updated patch:
https://github.com/intel/event-converter-for-linux-perf/pull/15/commits/afd779df99ee41aac646eae1ae5ae651cda3394d

v3. Removed filters rather than events in some cases as Suggested-by:
    Liang, Kan <kan.liang@linux.intel.com>
v2. Manually fixed a broken \\Inbound\\ ivytown event.

Ian Rogers (3):
  perf vendor events: Remove bad broadwellde uncore events
  perf vendor events: Remove bad ivytown uncore events
  perf vendor events: Remove bad jaketown uncore events

 .../arch/x86/broadwellde/uncore-cache.json    | 97 -------------------
 .../arch/x86/broadwellde/uncore-other.json    | 13 ---
 .../arch/x86/ivytown/uncore-cache.json        | 90 -----------------
 .../arch/x86/ivytown/uncore-interconnect.json |  1 -
 .../arch/x86/ivytown/uncore-other.json        | 13 ---
 .../arch/x86/ivytown/uncore-power.json        | 19 ----
 .../arch/x86/jaketown/uncore-cache.json       | 30 ------
 .../arch/x86/jaketown/uncore-other.json       | 13 ---
 .../arch/x86/jaketown/uncore-power.json       | 11 ---
 9 files changed, 287 deletions(-)

-- 
2.37.1.559.g78731f0fdb-goog
Re: [PATCH v3 0/3] Remove bad uncore filter/events
Posted by Liang, Kan 3 years, 8 months ago

On 2022-08-04 9:38 p.m., Ian Rogers wrote:
> The event converter scripts at:
> https://github.com/intel/event-converter-for-linux-perf
> passes Filter values from data on 01.org that is bogus in a perf command
> line and can cause perf to infinitely recurse in parse events. Remove
> such events or filter using the updated patch:
> https://github.com/intel/event-converter-for-linux-perf/pull/15/commits/afd779df99ee41aac646eae1ae5ae651cda3394d
> 
> v3. Removed filters rather than events in some cases as Suggested-by:
>     Liang, Kan <kan.liang@linux.intel.com>
> v2. Manually fixed a broken \\Inbound\\ ivytown event.
> 
> Ian Rogers (3):
>   perf vendor events: Remove bad broadwellde uncore events
>   perf vendor events: Remove bad ivytown uncore events
>   perf vendor events: Remove bad jaketown uncore events
> 

Reviewed-by: Kan Liang <kan.liang@linux.intel.com>

Thanks,
Kan

>  .../arch/x86/broadwellde/uncore-cache.json    | 97 -------------------
>  .../arch/x86/broadwellde/uncore-other.json    | 13 ---
>  .../arch/x86/ivytown/uncore-cache.json        | 90 -----------------
>  .../arch/x86/ivytown/uncore-interconnect.json |  1 -
>  .../arch/x86/ivytown/uncore-other.json        | 13 ---
>  .../arch/x86/ivytown/uncore-power.json        | 19 ----
>  .../arch/x86/jaketown/uncore-cache.json       | 30 ------
>  .../arch/x86/jaketown/uncore-other.json       | 13 ---
>  .../arch/x86/jaketown/uncore-power.json       | 11 ---
>  9 files changed, 287 deletions(-)
>
Re: [PATCH v3 0/3] Remove bad uncore filter/events
Posted by Arnaldo Carvalho de Melo 3 years, 8 months ago
Em Fri, Aug 05, 2022 at 10:45:31AM -0400, Liang, Kan escreveu:
> 
> 
> On 2022-08-04 9:38 p.m., Ian Rogers wrote:
> > The event converter scripts at:
> > https://github.com/intel/event-converter-for-linux-perf
> > passes Filter values from data on 01.org that is bogus in a perf command
> > line and can cause perf to infinitely recurse in parse events. Remove
> > such events or filter using the updated patch:
> > https://github.com/intel/event-converter-for-linux-perf/pull/15/commits/afd779df99ee41aac646eae1ae5ae651cda3394d
> > 
> > v3. Removed filters rather than events in some cases as Suggested-by:
> >     Liang, Kan <kan.liang@linux.intel.com>
> > v2. Manually fixed a broken \\Inbound\\ ivytown event.
> > 
> > Ian Rogers (3):
> >   perf vendor events: Remove bad broadwellde uncore events
> >   perf vendor events: Remove bad ivytown uncore events
> >   perf vendor events: Remove bad jaketown uncore events
> > 
> 
> Reviewed-by: Kan Liang <kan.liang@linux.intel.com>

Thanks, applied.

- Arnaldo

 
> Thanks,
> Kan
> 
> >  .../arch/x86/broadwellde/uncore-cache.json    | 97 -------------------
> >  .../arch/x86/broadwellde/uncore-other.json    | 13 ---
> >  .../arch/x86/ivytown/uncore-cache.json        | 90 -----------------
> >  .../arch/x86/ivytown/uncore-interconnect.json |  1 -
> >  .../arch/x86/ivytown/uncore-other.json        | 13 ---
> >  .../arch/x86/ivytown/uncore-power.json        | 19 ----
> >  .../arch/x86/jaketown/uncore-cache.json       | 30 ------
> >  .../arch/x86/jaketown/uncore-other.json       | 13 ---
> >  .../arch/x86/jaketown/uncore-power.json       | 11 ---
> >  9 files changed, 287 deletions(-)
> > 

-- 

- Arnaldo