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