linux-kernel - Re: [PATCH] perf tools: Dump callchain context marker names

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: Windows password security audit tool. GUI, reports in PDF.

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

Message-ID: <CAP-5=fUmbpDb0S33qb82Wvh2PjSGOWYYFi-70FZSWD6L7TuwLw@mail.gmail.com>
Date: Fri, 2 Jan 2026 04:18:37 -0800
From: Ian Rogers <irogers@...gle.com>
To: James Clark <james.clark@...aro.org>
Cc: Peter Zijlstra <peterz@...radead.org>, Ingo Molnar <mingo@...hat.com>, 
	Arnaldo Carvalho de Melo <acme@...nel.org>, Namhyung Kim <namhyung@...nel.org>, 
	Mark Rutland <mark.rutland@....com>, 
	Alexander Shishkin <alexander.shishkin@...ux.intel.com>, Jiri Olsa <jolsa@...nel.org>, 
	Adrian Hunter <adrian.hunter@...el.com>, Leo Yan <leo.yan@....com>, 
	linux-perf-users@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] perf tools: Dump callchain context marker names

On Fri, Jan 2, 2026 at 4:16 AM James Clark <james.clark@...aro.org> wrote:
>
> These are hard to interpret in the raw output because they are printed
> as hex but are defined in perf_event.h as decimal. Make it much easier
> to read the raw callchains by just printing their names.
>
> For example:
>
>   $ perf report -D
>
>   1798195372321 0x4638 [0xb0]: PERF_RECORD_SAMPLE(IP, 0x4002): 44922/44922: 0x7c8046dd3400 period: 120218 addr: 0
>   ... FP chain: nr:12
>   .....  0: fffffffffffffe00 (PERF_CONTEXT_USER)
>   .....  1: 00007c8046dd3400
>   .....  2: 00007c8046db86d3
>
> Signed-off-by: James Clark <james.clark@...aro.org>

Reviewed-by: Ian Rogers <irogers@...gle.com>

Thanks,
Ian

> ---
>  tools/perf/util/session.c | 25 +++++++++++++++++++++++--
>  1 file changed, 23 insertions(+), 2 deletions(-)
>
> diff --git a/tools/perf/util/session.c b/tools/perf/util/session.c
> index 4236503c8f6c..e812f8954075 100644
> --- a/tools/perf/util/session.c
> +++ b/tools/perf/util/session.c
> @@ -841,6 +841,26 @@ static void callchain__lbr_callstack_printf(struct perf_sample *sample)
>         }
>  }
>
> +static const char *callchain_context_str(u64 ip)
> +{
> +       switch (ip) {
> +       case PERF_CONTEXT_HV:
> +               return " (PERF_CONTEXT_HV)";
> +       case PERF_CONTEXT_KERNEL:
> +               return " (PERF_CONTEXT_KERNEL)";
> +       case PERF_CONTEXT_USER:
> +               return " (PERF_CONTEXT_USER)";
> +       case PERF_CONTEXT_GUEST:
> +               return " (PERF_CONTEXT_GUEST)";
> +       case PERF_CONTEXT_GUEST_KERNEL:
> +               return " (PERF_CONTEXT_GUEST_KERNEL)";
> +       case PERF_CONTEXT_GUEST_USER:
> +               return " (PERF_CONTEXT_GUEST_USER)";
> +       default:
> +               return "";
> +       }
> +}
> +
>  static void callchain__printf(struct evsel *evsel,
>                               struct perf_sample *sample)
>  {
> @@ -853,8 +873,9 @@ static void callchain__printf(struct evsel *evsel,
>         printf("... FP chain: nr:%" PRIu64 "\n", callchain->nr);
>
>         for (i = 0; i < callchain->nr; i++)
> -               printf("..... %2d: %016" PRIx64 "\n",
> -                      i, callchain->ips[i]);
> +               printf("..... %2d: %016" PRIx64 "%s\n",
> +                      i, callchain->ips[i],
> +                      callchain_context_str(callchain->ips[i]));
>
>         if (sample->deferred_callchain)
>                 printf("...... (deferred)\n");
>
> ---
> base-commit: cbd41c6d4c26c161a2b0e70ad411d3885ff13507
> change-id: 20260102-james-perf-dump-callchain-markers-64d175b108c8
>
> Best regards,
> --
> James Clark <james.clark@...aro.org>
>