diff options
author | Masahiro Yamada <masahiroy@kernel.org> | 2023-06-01 21:09:57 +0900 |
---|---|---|
committer | Masahiro Yamada <masahiroy@kernel.org> | 2023-06-02 17:59:52 +0900 |
commit | 12ca2c67d742d390c0aa1f8c1cfc49469df55ddf (patch) | |
tree | 797c448de29989a38f9b094babe7e0f473f1e2e0 /lib/uuid.c | |
parent | 56a24b8ce6a7f9c4a21b2276a8644f6f3d8fc14d (diff) | |
download | linux-stable-12ca2c67d742d390c0aa1f8c1cfc49469df55ddf.tar.gz linux-stable-12ca2c67d742d390c0aa1f8c1cfc49469df55ddf.tar.bz2 linux-stable-12ca2c67d742d390c0aa1f8c1cfc49469df55ddf.zip |
modpost: detect section mismatch for R_ARM_{MOVW_ABS_NC,MOVT_ABS}
For ARM defconfig (i.e. multi_v7_defconfig), modpost fails to detect
some types of section mismatches.
[test code]
#include <linux/init.h>
int __initdata foo;
int get_foo(void) { return foo; }
It is apparently a bad reference, but modpost does not report anything.
The test code above produces the following relocations.
Relocation section '.rel.text' at offset 0x200 contains 2 entries:
Offset Info Type Sym.Value Sym. Name
00000000 0000062b R_ARM_MOVW_ABS_NC 00000000 .LANCHOR0
00000004 0000062c R_ARM_MOVT_ABS 00000000 .LANCHOR0
Currently, R_ARM_MOVW_ABS_NC and R_ARM_MOVT_ABS are just skipped.
Add code to handle them. I checked arch/arm/kernel/module.c to learn
how the offset is encoded in the instruction.
The referenced symbol in relocation might be a local anchor.
If is_valid_name() returns false, let's search for a better symbol name.
Signed-off-by: Masahiro Yamada <masahiroy@kernel.org>
Diffstat (limited to 'lib/uuid.c')
0 files changed, 0 insertions, 0 deletions