Skip to contentSkip to conversation

modpost: add .text.fixup to OTHER_TEXT_SECTIONS

PATCH In review Arnd Bergmann 1 participant 2 messages Apr 15, 2015 Kbuild

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 modpost: add .text.fixup to OTHER_TEXT_SECTIONS

Two correct patches for the 4.1 release clashed in a way that caused modpost to fail badly, with hundreds warnings like

WARNING: vmlinux.o(__ex_table+0x1014): Section mismatch in reference from the (unknown reference) (unknown) to the (unknown reference) .text.fixup:(unknown)

The fix, as suggested by the warnings, is to add the new .text.fixup section into the OTHER_TEXT_SECTIONS list.

Fixes 52dc0595d540 ("modpost: handle relocations mismatch in __ex_table")
Fixes c4a84ae39b4a ("ARM: 8322/1: keep .text and .fixup regions closer together")

---
The mostpost patch has not made it into 4.1 yet, so it's probably best to add this fix on top of that to avoid breaking when Linus pulls it in.

scripts/mod/modpost.c +1 -1
index cbd53e08769d..dd7efba020a1 100644
@@ -876,7 +876,7 @@ static void check_section(const char *modname, struct elf_info *elf,
 #define TEXT_SECTIONS ".text", ".text.unlikely", ".sched.text", \
 		".kprobes.text"
 #define OTHER_TEXT_SECTIONS ".ref.text", ".head.text", ".spinlock.text", \
-		".fixup", ".entry.text"
+		".fixup", ".text.fixup", ".entry.text"
 
 #define INIT_SECTIONS      ".init.*"
 #define MEM_INIT_SECTIONS  ".meminit.*"
#1 lore raw
Quoted 8 lines · On Wednesday 15 April 2015 11:48:31 Arnd Bergmann wrote:

Two correct patches for the 4.1 release clashed in a way that caused modpost to fail badly, with hundreds warnings like

WARNING: vmlinux.o(__ex_table+0x1014): Section mismatch in reference from the (unknown reference) (unknown) to the (unknown reference) .text.fixup:(unknown)

The fix, as suggested by the warnings, is to add the new .text.fixup section into the OTHER_TEXT_SECTIONS list.

Quoted 7 lines
Fixes 52dc0595d540 ("modpost: handle relocations mismatch in __ex_table")
Fixes c4a84ae39b4a ("ARM: 8322/1: keep .text and .fixup regions closer together")

---
The mostpost patch has not made it into 4.1 yet, so it's probably best to add this fix on top of that to avoid breaking when Linus pulls it in.

Nevermind this patch, I'm way behind the times apparently, and multiple improvements of the original code have made it in since it was broken.

Arnd