Re: [GIT PULL] tracing: Updates for 7.2
From: Yury Norov
Date: Fri Jun 19 2026 - 18:19:00 EST
On Fri, Jun 19, 2026 at 08:40:12AM -0700, Linus Torvalds wrote:
> On Fri, 19 Jun 2026 at 08:19, Yury Norov <yury.norov@xxxxxxxxx> wrote:
> >
> > The Steven's point is the convenience for developers to throw
> > trace_printk() here and there without including any headers.
>
> Steven's point isn't a point. It's just fantasy.
>
> trace_printk() isn't special. It's not even common. And people who add
> debugging printk's can add a header file to their own hackery, rather
> than hurt everybody else.
OK, then I'll resend my patch. I tested it with build bots against
various architectures and configs, so I believe it's complete.
Your version includes trace_printk.h in linux/trace.h. It looks
reasonable, and it allows to drop explicit trace_printk.h
inclusion from:
- kernel/trace/trace.c;
- kernel/trace/trace_events_trigger.c;
- kernel/trace/trace_functions.c;
- kernel/trace/trace_printk.c.
With that, the total number of files including the trace_printk.h
is 30, which looks reasonable.
I need a couple days to get feedback from build bots, so not sure I'll
submit it before the end of the week.
If somebody wants to give it a try, check here:
https://github.com/norov/linux/tree/trace_printk
Thanks,
Yury