On Thu, 6 Mar 2025 11:37:38 +0800 Shijie Huang <[email protected]> wrote:
> diff --git a/kernel/tracepoint.c b/kernel/tracepoint.c > index abfd0ac1177f..5a5041f32cc8 100644 > --- a/kernel/tracepoint.c > +++ b/kernel/tracepoint.c > @@ -127,7 +127,8 @@ static void debug_print_probes(struct > tracepoint_func *funcs) > return; > > for (i = 0; funcs[i].func; i++) > - printk(KERN_DEBUG "Probe %d : %p\n", i, funcs[i].func); > + printk(KERN_DEBUG "Probe %d : [ %p ] %pS\n", > + i, funcs[i].func, funcs[i].func); > } > > > The output will look like: > > [ 63.818829] Probe 0 : [ 0000000032848d41 ] > perf_trace_sched_wakeup_template+0x0/0x20 > [ 63.819287] Probe 0 : [ 00000000fe8cca4d ] > perf_trace_sched_switch+0x0/0x20 > [ 65.325638] Probe 0 : [ 0000000032848d41 ] > perf_trace_sched_wakeup_template+0x0/0x20 > [ 65.695631] Probe 0 : [ 00000000fe8cca4d ] > perf_trace_sched_switch+0x0/0x20 I'm fine either way. Mathieu, what's your preference? Although the above shows a hashed %p. Is that even useful? -- Steve
