[PATCHSET 0/7] perf stat: Small random cleanups (v2)

Namhyung Kim posted 7 patches 1 year, 6 months ago
tools/perf/builtin-stat.c      |  54 ------
tools/perf/tests/topology.c    |  10 +-
tools/perf/util/cpumap.c       |   8 +-
tools/perf/util/cpumap.h       |   2 +-
tools/perf/util/stat-display.c |  32 ++--
tools/perf/util/stat-shadow.c  | 320 ++++++++++++++++-----------------
tools/perf/util/stat.c         |  20 +--
tools/perf/util/stat.h         |  10 +-
8 files changed, 189 insertions(+), 267 deletions(-)
[PATCHSET 0/7] perf stat: Small random cleanups (v2)
Posted by Namhyung Kim 1 year, 6 months ago
Hello,

I'm working on perf stat and I found some items to clean up.  This time
I removed runtime stats for per-thread aggregation mode which we can simply
use thread map index to compare the shadow stat values in the rt_stat.

Changes from v1)
 * renaming missing parts in headers  (James)
 * add a comment for map_idx  (Ian)
 * add Reviewed-by from James

The code is available at 'perf/stat-cleanup-v2' branch in

  git://git.kernel.org/pub/scm/linux/kernel/git/namhyung/linux-perf.git

Thanks,
Namhyung


Namhyung Kim (7):
  perf stat: Convert perf_stat_evsel.res_stats array
  perf stat: Don't call perf_stat_evsel_id_init() repeatedly
  perf stat: Rename saved_value->cpu_map_idx
  perf stat: Use thread map index for shadow stat
  perf stat: Kill unused per-thread runtime stats
  perf stat: Don't compare runtime stat for shadow stats
  perf stat: Rename to aggr_cpu_id.thread_idx

 tools/perf/builtin-stat.c      |  54 ------
 tools/perf/tests/topology.c    |  10 +-
 tools/perf/util/cpumap.c       |   8 +-
 tools/perf/util/cpumap.h       |   2 +-
 tools/perf/util/stat-display.c |  32 ++--
 tools/perf/util/stat-shadow.c  | 320 ++++++++++++++++-----------------
 tools/perf/util/stat.c         |  20 +--
 tools/perf/util/stat.h         |  10 +-
 8 files changed, 189 insertions(+), 267 deletions(-)


base-commit: 62e64c9d2fd12839c02f1b3e8b873e7cb34e8720
-- 
2.38.0.rc1.362.ged0d419d3c-goog
Re: [PATCHSET 0/7] perf stat: Small random cleanups (v2)
Posted by Arnaldo Carvalho de Melo 1 year, 5 months ago
Em Fri, Sep 30, 2022 at 01:21:03PM -0700, Namhyung Kim escreveu:
> Hello,
> 
> I'm working on perf stat and I found some items to clean up.  This time
> I removed runtime stats for per-thread aggregation mode which we can simply
> use thread map index to compare the shadow stat values in the rt_stat.
> 
> Changes from v1)
>  * renaming missing parts in headers  (James)
>  * add a comment for map_idx  (Ian)
>  * add Reviewed-by from James
> 
> The code is available at 'perf/stat-cleanup-v2' branch in
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/namhyung/linux-perf.git

Applied locally, testing.

- Arnaldo
 
> Thanks,
> Namhyung
> 
> 
> Namhyung Kim (7):
>   perf stat: Convert perf_stat_evsel.res_stats array
>   perf stat: Don't call perf_stat_evsel_id_init() repeatedly
>   perf stat: Rename saved_value->cpu_map_idx
>   perf stat: Use thread map index for shadow stat
>   perf stat: Kill unused per-thread runtime stats
>   perf stat: Don't compare runtime stat for shadow stats
>   perf stat: Rename to aggr_cpu_id.thread_idx
> 
>  tools/perf/builtin-stat.c      |  54 ------
>  tools/perf/tests/topology.c    |  10 +-
>  tools/perf/util/cpumap.c       |   8 +-
>  tools/perf/util/cpumap.h       |   2 +-
>  tools/perf/util/stat-display.c |  32 ++--
>  tools/perf/util/stat-shadow.c  | 320 ++++++++++++++++-----------------
>  tools/perf/util/stat.c         |  20 +--
>  tools/perf/util/stat.h         |  10 +-
>  8 files changed, 189 insertions(+), 267 deletions(-)
> 
> 
> base-commit: 62e64c9d2fd12839c02f1b3e8b873e7cb34e8720
> -- 
> 2.38.0.rc1.362.ged0d419d3c-goog

-- 

- Arnaldo