Re: [PATCH] printf: mark errptr() noinline

From: Petr Mladek

Date: Wed Apr 08 2026 - 03:25:54 EST


On Tue 2026-04-07 16:08:09, David Laight wrote:
> On Mon, 6 Apr 2026 12:32:32 -0400
> Steven Rostedt <rostedt@xxxxxxxxxxx> wrote:
>
> > On Mon, 6 Apr 2026 11:21:39 -0400
> > Tamir Duberstein <tamird@xxxxxxxxxx> wrote:
> >
> > > Thanks Steve. IMO that is a very big hammer and not warranted in this
> > > case. There's been talk of encouraging distros to enable CONFIG_KUNIT
> > > by default [0], which would probably interact poorly with the change
> > > you propose.
> > >
> >
> > Branch profiling is really just a niche that is enabled specifically for
> > seeing all branches taken in the kernel. It hooks to all "if" statements!
> > As you can imagine, it causes a rather large overhead in performance.
> >
> > This option is only used by developers doing special analysis of their code
> > (namely me ;-).
>
> Is there any way to stop randconfig picking up options like these?
> It is rather a waste of brain-cycles trying to fix them.
> If you want the option to test a specific bit of code it is easy to
> hack/disable any problematic parts.
>
> Even having the KASAN/KMSAN code compiled into allmodconfig is a PITA
> when you are trying to check that code compiles to something sensible.

This does not look like a good idea. KASAN/KMSAN are very useful
features. People will want to keep them working. Removing them from
randconfig would just postpone detection of the problem. We would
need to deal with it sooner or later anyway.

Best Regards,
Petr