]> git.baikalelectronics.ru Git - uboot.git/commit
x86: Fix car_uninit weak symbol definition
authorBin Meng <bmeng.cn@gmail.com>
Thu, 25 Oct 2018 10:05:37 +0000 (03:05 -0700)
committerBin Meng <bmeng.cn@gmail.com>
Sun, 28 Oct 2018 13:02:15 +0000 (21:02 +0800)
commit216e8f7f94bbb300d1d2c8145776fc2581b3483e
tree6529f26302c486df7345d8e300fd6527475f1082
parente755c0bf5febb4e5667ef2c49700510cfb0eb36f
x86: Fix car_uninit weak symbol definition

Since commit d2ef0c3b624f ("x86: detect unsupported relocation types"),
an error message is seen on QEMU x86 target during boot:

do_elf_reloc_fixups32: unsupported relocation type 0x1 at fff841f0, offset = 0xfff00087
do_elf_reloc_fixups32: unsupported relocation type 0x2 at fff841f8, offset = 0xfff00091

Check offset 0xfff00087 and 0xfff00091 in the u-boot ELF image,

fff00087  000df401 R_386_32          00000000   car_uninit
fff00091  000df402 R_386_PC32        00000000   car_uninit

we see R_386_32 and R_386_PC32 relocation type is generated for
symbol car_uninit, which is declared as a weak symbol in start.S.

However the actual weak symbol implementation ends up nowhere. As
we can see below, it's *UND*.

$ objdump -t u-boot | grep car_uninit
00000000  w      *UND*  00000000 car_uninit

With this fix, it is normal now.

$ objdump -t u-boot | grep car_uninit
fff00094  w    F .text.start    00000001 car_uninit

Reported-by: Hannes Schmelzer <hannes@schmelzer.or.at>
Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
Tested-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
Reviewed-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
Tested-by: Hannes Schmelzer <oe5hpm@oevsv.at>
arch/x86/cpu/start.S