On Wed, Nov 12, 2025 at 6:21 AM Petr Mladek <[email protected]> wrote:
>
> bpf_address_lookup() has been used only in kallsyms_lookup_buildid().
> It was supposed to set @modname and @modbuildid when the symbol was
> in a module.
>
> But it always just cleared @modname because BPF symbols were never in
> a module. And it did not clear @modbuildid because the pointer was
> not passed.
>
> The wrapper is not longer needed. Both @modname and @modbuildid
is no longer
> are newly always initialized to NULL in kallsyms_lookup_buildid().
are now?
> Remove the wrapper and rename __bpf_address_lookup() to
> bpf_address_lookup() because this variant is used everywhere.
>
> Fixes: 9294523e3768 ("module: add printk formats to add module build ID to
> stacktraces")
> Signed-off-by: Petr Mladek <[email protected]>
other than typos in the commit log it lgtm.
Acked-by: Alexei Starovoitov <[email protected]>
