Skip to contentSkip to conversation

ARM: Remove unused .fixup section in boot stub

PATCH In review Kees Cook 3 participants 3 messages Feb 8, 2020 Arm (32-bit)

lore
Share & tools
lore

Kees Cook

author
#0 lore raw

PATCH 1/1 ARM: Remove unused .fixup section in boot stub

The boot stub does not emit a .fixup section at all anymore, so remove it.

Suggested-by Nick Desaulniers <[email protected]>
Link https://lore.kernel.org/lkml/CAKwvOdnRhx=SgtcUCyX2ZOGATM8OzG6hSOY9wGQZcwtp+P5WBQ@mail.gmail.com open here
Signed-off-by Kees Cook <[email protected]>
---
 arch/arm/boot/compressed/vmlinux.lds.S | 1 -
 1 file changed, 1 deletion(-)
arch/arm/boot/compressed/vmlinux.lds.S +0 -1
index fc7ed03d8b93..b247f399de71 100644
@@ -36,7 +36,6 @@ SECTIONS
     *(.start)
     *(.text)
     *(.text.*)
-    *(.fixup)
     *(.gnu.warning)
     *(.glue_7t)
     *(.glue_7)
Signature · 5 lines
2.20.1


-- 
Kees Cook
Reviewed-by on v1 1/1
#1 lore raw
Quoted 7 lines · Kees Cook

The boot stub does not emit a .fixup section at all anymore, so remove it.

Suggested-by Nick Desaulniers <[email protected]>
Link https://lore.kernel.org/lkml/CAKwvOdnRhx=SgtcUCyX2ZOGATM8OzG6hSOY9wGQZcwtp+P5WBQ@mail.gmail.com open here
Signed-off-by Kees Cook <[email protected]>
Reviewed-by Ard Biesheuvel <[email protected]>

I'd assume these are uncontroversial enough to go straight into the patch system.

Quoted 22 lines
---
 arch/arm/boot/compressed/vmlinux.lds.S | 1 -
 1 file changed, 1 deletion(-)
arch/arm/boot/compressed/vmlinux.lds.S +0 -2
index fc7ed03d8b93..b247f399de71 100644
@@ -36,7 +36,6 @@ SECTIONS
     *(.start)
     *(.text)
     *(.text.*)
-    *(.fixup)
     *(.gnu.warning)
     *(.glue_7t)
     *(.glue_7)
--

2.20.1

--
Kees Cook

Reviewed-by on v1 1/1
#2 lore raw
In reply to Kees Cook
Quoted 7 lines · Kees Cook

The boot stub does not emit a .fixup section at all anymore, so remove it.

Suggested-by Nick Desaulniers <[email protected]>
Link https://lore.kernel.org/lkml/CAKwvOdnRhx=SgtcUCyX2ZOGATM8OzG6hSOY9wGQZcwtp+P5WBQ@mail.gmail.com open here
Signed-off-by Kees Cook <[email protected]>

thanks for the patch!

Reviewed-by Nick Desaulniers <[email protected]>
Quoted 22 lines
---
 arch/arm/boot/compressed/vmlinux.lds.S | 1 -
 1 file changed, 1 deletion(-)
arch/arm/boot/compressed/vmlinux.lds.S +0 -2
index fc7ed03d8b93..b247f399de71 100644
@@ -36,7 +36,6 @@ SECTIONS
     *(.start)
     *(.text)
     *(.text.*)
-    *(.fixup)
     *(.gnu.warning)
     *(.glue_7t)
     *(.glue_7)
--

2.20.1

--
Kees Cook

Signature · 2 lines
Thanks,
~Nick Desaulniers