Hello,

This is to remove arbitrary restrictions in the hierarchy mode.

The -F/--fields option is to specify output fields and sort keys in perf
report exactly what users want.  It was not allowed in the hierarchy mode
because it missed to handle the field correctly.

This patchset addresses that problem and allows the option.  It'd make
the output more flexible.  For example, this is possible:

  $ perf report -F overhead,comm,dso -H

Thanks,
Namhyung


Namhyung Kim (4):
  perf hist: Remove formats in hierarchy when cancel children
  perf hist: Remove formats in hierarchy when cancel latency
  perf hist: Set levels in output_field_add()
  perf hist: Allow custom output fields in hierarchy mode

 tools/perf/builtin-c2c.c    |  3 ++-
 tools/perf/builtin-report.c | 16 ++++++----------
 tools/perf/builtin-top.c    |  2 +-
 tools/perf/ui/hist.c        | 35 +++++++++++++++++++++++++++++++++--
 tools/perf/util/hist.h      |  4 ++--
 tools/perf/util/sort.c      | 36 +++++++++++++++++++++++-------------
 tools/perf/util/sort.h      |  2 +-
 7 files changed, 68 insertions(+), 30 deletions(-)


base-commit: 35d13f841a3d8159ef20d5e32a9ed3faa27875bc
-- 
2.49.0

On Mon, Mar 31, 2025 at 12:37:18AM -0700, Namhyung Kim wrote:
> Hello,
> 
> This is to remove arbitrary restrictions in the hierarchy mode.
> 
> The -F/--fields option is to specify output fields and sort keys in perf
> report exactly what users want.  It was not allowed in the hierarchy mode
> because it missed to handle the field correctly.
> 
> This patchset addresses that problem and allows the option.  It'd make
> the output more flexible.  For example, this is possible:
> 
>   $ perf report -F overhead,comm,dso -H

Thanks, tested and applied to perf-tools-next.

- Arnaldo