* [PATCH] perf tools: Dump callchain context marker names
@ 2026-01-02 12:15 James Clark
2026-01-02 12:18 ` Ian Rogers
2026-01-02 22:40 ` Namhyung Kim
0 siblings, 2 replies; 5+ messages in thread
From: James Clark @ 2026-01-02 12:15 UTC (permalink / raw)
To: Peter Zijlstra, Ingo Molnar, Arnaldo Carvalho de Melo,
Namhyung Kim, Mark Rutland, Alexander Shishkin, Jiri Olsa,
Ian Rogers, Adrian Hunter, Leo Yan
Cc: linux-perf-users, linux-kernel, James Clark
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@linaro.org>
---
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@linaro.org>
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [PATCH] perf tools: Dump callchain context marker names
2026-01-02 12:15 [PATCH] perf tools: Dump callchain context marker names James Clark
@ 2026-01-02 12:18 ` Ian Rogers
2026-01-13 20:33 ` Arnaldo Carvalho de Melo
2026-01-02 22:40 ` Namhyung Kim
1 sibling, 1 reply; 5+ messages in thread
From: Ian Rogers @ 2026-01-02 12:18 UTC (permalink / raw)
To: James Clark
Cc: Peter Zijlstra, Ingo Molnar, Arnaldo Carvalho de Melo,
Namhyung Kim, Mark Rutland, Alexander Shishkin, Jiri Olsa,
Adrian Hunter, Leo Yan, linux-perf-users, linux-kernel
On Fri, Jan 2, 2026 at 4:16 AM James Clark <james.clark@linaro.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@linaro.org>
Reviewed-by: Ian Rogers <irogers@google.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@linaro.org>
>
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH] perf tools: Dump callchain context marker names
2026-01-02 12:18 ` Ian Rogers
@ 2026-01-13 20:33 ` Arnaldo Carvalho de Melo
0 siblings, 0 replies; 5+ messages in thread
From: Arnaldo Carvalho de Melo @ 2026-01-13 20:33 UTC (permalink / raw)
To: Ian Rogers
Cc: James Clark, Peter Zijlstra, Ingo Molnar, Namhyung Kim,
Mark Rutland, Alexander Shishkin, Jiri Olsa, Adrian Hunter,
Leo Yan, linux-perf-users, linux-kernel
On Fri, Jan 02, 2026 at 04:18:37AM -0800, Ian Rogers wrote:
> On Fri, Jan 2, 2026 at 4:16 AM James Clark <james.clark@linaro.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@linaro.org>
>
> Reviewed-by: Ian Rogers <irogers@google.com>
Thanks, applied to perf-tools-next,
- Arnaldo
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] perf tools: Dump callchain context marker names
2026-01-02 12:15 [PATCH] perf tools: Dump callchain context marker names James Clark
2026-01-02 12:18 ` Ian Rogers
@ 2026-01-02 22:40 ` Namhyung Kim
2026-01-13 20:37 ` Arnaldo Carvalho de Melo
1 sibling, 1 reply; 5+ messages in thread
From: Namhyung Kim @ 2026-01-02 22:40 UTC (permalink / raw)
To: James Clark
Cc: Peter Zijlstra, Ingo Molnar, Arnaldo Carvalho de Melo,
Mark Rutland, Alexander Shishkin, Jiri Olsa, Ian Rogers,
Adrian Hunter, Leo Yan, linux-perf-users, linux-kernel
Hi James,
Happy new year!
On Fri, Jan 02, 2026 at 12:15:43PM +0000, James Clark 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@linaro.org>
> ---
> 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)";
We now have PERF_CONTEXT_USER_DEFERRED too.
Thanks,
Namhyung
> + 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@linaro.org>
>
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH] perf tools: Dump callchain context marker names
2026-01-02 22:40 ` Namhyung Kim
@ 2026-01-13 20:37 ` Arnaldo Carvalho de Melo
0 siblings, 0 replies; 5+ messages in thread
From: Arnaldo Carvalho de Melo @ 2026-01-13 20:37 UTC (permalink / raw)
To: Namhyung Kim
Cc: James Clark, Peter Zijlstra, Ingo Molnar, Mark Rutland,
Alexander Shishkin, Jiri Olsa, Ian Rogers, Adrian Hunter, Leo Yan,
linux-perf-users, linux-kernel
On Fri, Jan 02, 2026 at 02:40:05PM -0800, Namhyung Kim wrote:
> Hi James,
>
> Happy new year!
>
> On Fri, Jan 02, 2026 at 12:15:43PM +0000, James Clark 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@linaro.org>
> > ---
> > 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)";
>
> We now have PERF_CONTEXT_USER_DEFERRED too.
I ammended the patch with:
diff --git a/tools/perf/util/session.c b/tools/perf/util/session.c
index e812f89540755f31..65fa9bdff1b8cdf2 100644
--- a/tools/perf/util/session.c
+++ b/tools/perf/util/session.c
@@ -856,6 +856,8 @@ static const char *callchain_context_str(u64 ip)
return " (PERF_CONTEXT_GUEST_KERNEL)";
case PERF_CONTEXT_GUEST_USER:
return " (PERF_CONTEXT_GUEST_USER)";
+ case PERF_CONTEXT_USER_DEFERRED:
+ return " (PERF_CONTEXT_USER_DEFERRED)";
default:
return "";
}
⬢ [acme@toolbx perf-tools-next]$
And added a note about being Namhyung who made the suggestion, thanks.
- Arnaldo
^ permalink raw reply related [flat|nested] 5+ messages in thread
end of thread, other threads:[~2026-01-13 20:37 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-01-02 12:15 [PATCH] perf tools: Dump callchain context marker names James Clark
2026-01-02 12:18 ` Ian Rogers
2026-01-13 20:33 ` Arnaldo Carvalho de Melo
2026-01-02 22:40 ` Namhyung Kim
2026-01-13 20:37 ` Arnaldo Carvalho de Melo
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox