Re: [PATCH] perf report: Clarify in help that --children is default

From: Arnaldo Carvalho de Melo
Date: Mon Jan 06 2020 - 17:16:54 EST


Em Fri, Jan 03, 2020 at 10:36:43AM -0800, Andi Kleen escreveu:
> From: Andi Kleen <ak@xxxxxxxxxxxxxxx>
>
> Refer to --no-children, which is what most people probably
> want.

Thanks, applied.

- Arnaldo

> Signed-off-by: Andi Kleen <ak@xxxxxxxxxxxxxxx>
> ---
> tools/perf/builtin-report.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/tools/perf/builtin-report.c b/tools/perf/builtin-report.c
> index 387311c67264..059639f437e5 100644
> --- a/tools/perf/builtin-report.c
> +++ b/tools/perf/builtin-report.c
> @@ -1163,7 +1163,8 @@ int cmd_report(int argc, const char **argv)
> report_callchain_help, &report_parse_callchain_opt,
> callchain_default_opt),
> OPT_BOOLEAN(0, "children", &symbol_conf.cumulate_callchain,
> - "Accumulate callchains of children and show total overhead as well"),
> + "Accumulate callchains of children and show total overhead as well. "
> + "Enabled by default, use --no-children to disable."),
> OPT_INTEGER(0, "max-stack", &report.max_stack,
> "Set the maximum stack depth when parsing the callchain, "
> "anything beyond the specified depth will be ignored. "
> --
> 2.23.0

--

- Arnaldo