Re: [PATCH v2] ftrace: Add FTRACE_MCOUNT_MAX_OFFSET to avoid adding weak function

From: Ingo Molnar
Date: Fri May 27 2022 - 06:04:25 EST



* Steven Rostedt <rostedt@xxxxxxxxxxx> wrote:

> On Thu, 26 May 2022 04:50:17 +0200
> Ingo Molnar <mingo@xxxxxxxxxx> wrote:
>
>
> > > The real fix would be to fix kallsyms to not show address of weak
> > > functions as the function before it. But that would require adding code in
> > > the build to add function size to kallsyms so that it can know when the
> > > function ends instead of just using the start of the next known symbol.
> >
> > Yeah, so I actually have a (prototype...) objtool based kallsyms
> > implementation in the (way too large) fast-headers tree that is both faster
> > & allows such details in principle:
>
> Nice.
>
> Will this work for other architectures too?

For those which implement objtool, it certainly should: as we parse through
each object file during the build, generating kallsyms data structures is
relatively straightforward.

Objtool availability is a big gating condition though. :-/

[ ... and still Acked-by on -v4 too. ]

Thanks,

Ingo