On Thu, 26 Dec 2024 at 13:49, Steven Rostedt <rostedt@xxxxxxxxxxx> wrote:
>
> But then, when the linker removes these functions because they were
> overridden, the code does not disappear, leaving the pointers in the
> __mcount_loc locations.
This seems entirely unrelated to weak functions, and will be true for
any other "linker removed it" (which can happen for other reasons
too).
So your "fix" seems to be hacking around a symptom.
And honestly, the kallsyms argument seems bogus too. The problem with
kallsyms is that it looks up the size the wrong way. Making up new
function names doesn't fix the problem, it - once again - just hacks
around the symptom of doing something wrong.
Christ, kallsyms looking at nm output and going by "next symbol" was
always bogus, but I think that's how the old a.out format worked
originally.
But "nm" literally takes a "-S" argument. We just don't use it.
So I think the fix is literally to just make kallsysms have the size
data. Of course, very annoyingly out /proc/kallsyms file format also
tracks the (legacy) nm output that doesn't have size information.
But I do think that if you hit real problems, you need to fix the
*source* of the issue, not add another ugly hack around things.
Linus
