modpost: detect section mismatch for R_ARM_{MOVW_ABS_NC,MOVT_ABS}
authorMasahiro Yamada <masahiroy@kernel.org>
Thu, 1 Jun 2023 12:09:57 +0000 (21:09 +0900)
committerMasahiro Yamada <masahiroy@kernel.org>
Fri, 2 Jun 2023 08:59:52 +0000 (17:59 +0900)
commit12ca2c67d742d390c0aa1f8c1cfc49469df55ddf
tree797c448de29989a38f9b094babe7e0f473f1e2e0
parent56a24b8ce6a7f9c4a21b2276a8644f6f3d8fc14d
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>
scripts/mod/modpost.c