linux-kernel - Re: [PATCH 2/3] perf ftrace: Add --graph-opts option to profile subcommand

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <d6cc9e4b-7a38-4673-9c2a-d97301a7fdfa@linaro.org>
Date: Tue, 7 Jan 2025 14:48:58 +0000
From: James Clark <james.clark@...aro.org>
To: Namhyung Kim <namhyung@...nel.org>
Cc: Jiri Olsa <jolsa@...nel.org>, Adrian Hunter <adrian.hunter@...el.com>,
 Peter Zijlstra <peterz@...radead.org>, Ingo Molnar <mingo@...nel.org>,
 LKML <linux-kernel@...r.kernel.org>, linux-perf-users@...r.kernel.org,
 Arnaldo Carvalho de Melo <acme@...nel.org>, Ian Rogers <irogers@...gle.com>,
 Kan Liang <kan.liang@...ux.intel.com>
Subject: Re: [PATCH 2/3] perf ftrace: Add --graph-opts option to profile
 subcommand



On 03/01/2025 11:16 pm, Namhyung Kim wrote:
> Like trace subcommand, it should be able to pass some options to control
> the tracing behavior for the function graph tracer.  But some options
> are limited in order to maintain the internal behavior.
> 
> For example, it can limit the function call depth like below:
> 
>    # perf ftrace profile --graph-opts depth=5 -- myprog
> 
> Signed-off-by: Namhyung Kim <namhyung@...nel.org>
> ---
>   tools/perf/Documentation/perf-ftrace.txt | 8 ++++++++
>   tools/perf/builtin-ftrace.c              | 4 ++++
>   2 files changed, 12 insertions(+)
> 

Reviewed-by: James Clark <james.clark@...aro.org>

> diff --git a/tools/perf/Documentation/perf-ftrace.txt b/tools/perf/Documentation/perf-ftrace.txt
> index 16b067eb5d84746d..4a69a1135f07341b 100644
> --- a/tools/perf/Documentation/perf-ftrace.txt
> +++ b/tools/perf/Documentation/perf-ftrace.txt
> @@ -205,6 +205,14 @@ OPTIONS for 'perf ftrace profile'
>   	Sort the result by the given field.  Available values are:
>   	total, avg, max, count, name.  Default is 'total'.
>   
> +--graph-opts::
> +	List of options allowed to set:
> +
> +	  - nosleep-time - Measure on-CPU time only for function_graph tracer.
> +	  - noirqs       - Ignore functions that happen inside interrupt.
> +	  - thresh=<n>   - Setup trace duration threshold in microseconds.
> +	  - depth=<n>    - Set max depth for function graph tracer to follow.
> +
>   
>   SEE ALSO
>   --------
> diff --git a/tools/perf/builtin-ftrace.c b/tools/perf/builtin-ftrace.c
> index 0ac2b20b272bd790..d747cca2403e494e 100644
> --- a/tools/perf/builtin-ftrace.c
> +++ b/tools/perf/builtin-ftrace.c
> @@ -1072,6 +1072,7 @@ static int prepare_func_profile(struct perf_ftrace *ftrace)
>   {
>   	ftrace->tracer = "function_graph";
>   	ftrace->graph_tail = 1;
> +	ftrace->graph_verbose = 0;
>   
>   	ftrace->profile_hash = hashmap__new(profile_hash, profile_equal, NULL);
>   	if (ftrace->profile_hash == NULL)
> @@ -1660,6 +1661,9 @@ int cmd_ftrace(int argc, const char **argv)
>   	OPT_CALLBACK('s', "sort", &profile_sort, "key",
>   		     "Sort result by key: total (default), avg, max, count, name.",
>   		     parse_sort_key),
> +	OPT_CALLBACK(0, "graph-opts", &ftrace, "options",
> +		     "Graph tracer options, available options: nosleep-time,noirqs,thresh=<n>,depth=<n>",
> +		     parse_graph_tracer_opts),
>   	OPT_PARENT(common_options),
>   	};
>   	const struct option *options = ftrace_options;