Hi Ian, On Thu, Sep 10, 2020 at 12:26 PM Ian Rogers <[email protected]> wrote: > > No need to set os.evsel twice. > > Signed-off-by: Ian Rogers <[email protected]>
Acked-by: Namhyung Kim <[email protected]> Btw, I think setting the 'out' variable can be moved out of the loop. Thanks Namhyung > --- > tools/perf/util/stat-display.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/tools/perf/util/stat-display.c b/tools/perf/util/stat-display.c > index 493ec372fdec..4b57c0c07632 100644 > --- a/tools/perf/util/stat-display.c > +++ b/tools/perf/util/stat-display.c > @@ -946,7 +946,6 @@ static void print_metric_headers(struct perf_stat_config > *config, > out.print_metric = print_metric_header; > out.new_line = new_line_metric; > out.force_header = true; > - os.evsel = counter; > perf_stat__print_shadow_stats(config, counter, 0, > 0, > &out, > -- > 2.28.0.526.ge36021eeef-goog >
