Hi Mikhail,
On Wed, Mar 11, 2020 at 5:34 AM Mikhail Petrov <Mikhail.Petrov@xxxxxxx> wrote:
>
> There is the code in the read_symbol function in 'scripts/kallsyms.c':
>
> if (is_ignored_symbol(name, type))
> return NULL;
>
> /* Ignore most absolute/undefined (?) symbols. */
> if (strcmp(name, "_text") == 0)
> _text = addr;
>
> But the is_ignored_symbol function returns true for name="_text" and type='a'. So the next condition is not executed and the _text variable is always zero.
>
> It makes the wrong kallsyms_relative_base symbol as a result of the code:
>
> if (base_relative) {
> output_label("kallsyms_relative_base");
> output_address(relative_base);
> printf("\n");
> }
>
> Because the output_address function uses the _text variable.
>
> So the kallsyms_lookup function and all related functions in the kernel do not work properly. For example, the stack trace in oops:
>
> Call Trace:
> [aa095e58] [809feab8] kobj_ns_ops_tbl+0x7ff09ac8/0x7ff1c1c4 (unreliable)
> [aa095e98] [80002b64] kobj_ns_ops_tbl+0x7f50db74/0x80000010
> [aa095ef8] [809c3d24] kobj_ns_ops_tbl+0x7feced34/0x7ff1c1c4
> [aa095f28] [80002ed0] kobj_ns_ops_tbl+0x7f50dee0/0x80000010
> [aa095f38] [8000f238] kobj_ns_ops_tbl+0x7f51a248/0x80000010
>
> The right stack trace:
>
> Call Trace:
> [aa095e58] [809feab8] module_vdu_video_init+0x2fc/0x3bc (unreliable)
> [aa095e98] [80002b64] do_one_initcall+0x40/0x1f0
> [aa095ef8] [809c3d24] kernel_init_freeable+0x164/0x1d8
> [aa095f28] [80002ed0] kernel_init+0x14/0x124
> [aa095f38] [8000f238] ret_from_kernel_thread+0x14/0x1c
>
> Signed-off-by: Mikhail Petrov <Mikhail.Petrov@xxxxxxx>
>
> ---
Thanks for the patch.
Just for curiosity, on which architecrure
did you see name="_text" and type='a' case ?
Could you wrap the commit log to avoid
this checkpatch warning?
WARNING: Possible unwrapped commit description (prefer a maximum 75
chars per line)
Also, could you shorten the patch subject
to make it fit in this limit?
Thanks.
--
Best Regards
Masahiro Yamada
