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

From: Steven Rostedt

Date: Mon Apr 06 2026 - 12:32:52 EST


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 ;-).

The only real concern I would have is if the kunit test developers would
want to use the branch profiling on their code, in which case my suggestion
would prevent that.

-- Steve