Messages in this thread | | | From | Namhyung Kim <> | Date | Thu, 10 Sep 2020 14:38:24 +0900 | Subject | Re: [PATCH v2 1/3] perf stat: Remove dead code |
| |
Hi Ian,
On Thu, Sep 10, 2020 at 12:26 PM Ian Rogers <irogers@google.com> wrote: > > No need to set os.evsel twice. > > Signed-off-by: Ian Rogers <irogers@google.com>
Acked-by: Namhyung Kim <namhyung@kernel.org>
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 >
| |