Messages in this thread | | | Date | Thu, 26 May 2022 09:11:06 -0400 | From | Steven Rostedt <> | Subject | Re: [PATCH v2] ftrace: Add FTRACE_MCOUNT_MAX_OFFSET to avoid adding weak function |
| |
On Thu, 26 May 2022 04:50:17 +0200 Ingo Molnar <mingo@kernel.org> 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?
> > If CONFIG_HAVE_FENTRY is defined for x86, define FTRACE_MCOUNT_MAX_OFFSET > > to zero, which will have ftrace ignore all locations that are not at the > > start of the function. > > > > [1] https://lore.kernel.org/all/20220412094923.0abe90955e5db486b7bca279@kernel.org/ > > > > Signed-off-by: Steven Rostedt (Google) <rostedt@goodmis.org> > > LGTM. > > I suppose you'd like to merge this via the tracing tree? If so:
Yeah, I'll pull it through my tree.
> > Acked-by: Ingo Molnar <mingo@kernel.org>
Thanks a lot Ingo for reviewing it. I really appreciate it!
-- Steve
| |