aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorArd Biesheuvel <ard.biesheuvel@linaro.org>2015-10-23 10:48:14 -0400
committerCatalin Marinas <catalin.marinas@arm.com>2015-10-30 12:02:52 -0400
commitbf457786f569cc480629d7855cac1fd1173ac009 (patch)
treef4f2eea9505db363050ae55eb80b2317b2446eb9
parentcb083816ab5ac3d10a9417527f07fc5962cc3808 (diff)
arm64/efi: move arm64 specific stub C code to libstub
Now that we added special handling to the C files in libstub, move the one remaining arm64 specific EFI stub C file to libstub as well, so that it gets the same treatment. This should prevent future changes from resulting in binaries that may execute incorrectly in UEFI context. With efi-entry.S the only remaining EFI stub source file under arch/arm64, we can also simplify the Makefile logic somewhat. Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org> Reviewed-by: Matt Fleming <matt@codeblueprint.co.uk> Tested-by: Jeremy Linton <jeremy.linton@arm.com> Signed-off-by: Catalin Marinas <catalin.marinas@arm.com>
-rw-r--r--arch/arm64/kernel/Makefile9
-rw-r--r--drivers/firmware/efi/libstub/Makefile3
-rw-r--r--drivers/firmware/efi/libstub/arm64-stub.c (renamed from arch/arm64/kernel/efi-stub.c)0
3 files changed, 5 insertions, 7 deletions
diff --git a/arch/arm64/kernel/Makefile b/arch/arm64/kernel/Makefile
index 1b6bda2ff102..474691f8b13a 100644
--- a/arch/arm64/kernel/Makefile
+++ b/arch/arm64/kernel/Makefile
@@ -4,11 +4,8 @@
4 4
5CPPFLAGS_vmlinux.lds := -DTEXT_OFFSET=$(TEXT_OFFSET) 5CPPFLAGS_vmlinux.lds := -DTEXT_OFFSET=$(TEXT_OFFSET)
6AFLAGS_head.o := -DTEXT_OFFSET=$(TEXT_OFFSET) 6AFLAGS_head.o := -DTEXT_OFFSET=$(TEXT_OFFSET)
7CFLAGS_efi-stub.o := -DTEXT_OFFSET=$(TEXT_OFFSET)
8CFLAGS_armv8_deprecated.o := -I$(src) 7CFLAGS_armv8_deprecated.o := -I$(src)
9 8
10KASAN_SANITIZE_efi-stub.o := n
11
12CFLAGS_REMOVE_ftrace.o = -pg 9CFLAGS_REMOVE_ftrace.o = -pg
13CFLAGS_REMOVE_insn.o = -pg 10CFLAGS_REMOVE_insn.o = -pg
14CFLAGS_REMOVE_return_address.o = -pg 11CFLAGS_REMOVE_return_address.o = -pg
@@ -22,9 +19,7 @@ arm64-obj-y := debug-monitors.o entry.o irq.o fpsimd.o \
22 cpufeature.o alternative.o cacheinfo.o \ 19 cpufeature.o alternative.o cacheinfo.o \
23 smp.o smp_spin_table.o topology.o 20 smp.o smp_spin_table.o topology.o
24 21
25stub-obj := efi-stub.o efi-entry.o 22extra-$(CONFIG_EFI) := efi-entry.o
26extra-y := $(stub-obj)
27stub-obj := $(patsubst %.o,%.stub.o,$(stub-obj))
28 23
29OBJCOPYFLAGS := --prefix-symbols=__efistub_ 24OBJCOPYFLAGS := --prefix-symbols=__efistub_
30$(obj)/%.stub.o: $(obj)/%.o FORCE 25$(obj)/%.stub.o: $(obj)/%.o FORCE
@@ -42,7 +37,7 @@ arm64-obj-$(CONFIG_CPU_PM) += sleep.o suspend.o
42arm64-obj-$(CONFIG_CPU_IDLE) += cpuidle.o 37arm64-obj-$(CONFIG_CPU_IDLE) += cpuidle.o
43arm64-obj-$(CONFIG_JUMP_LABEL) += jump_label.o 38arm64-obj-$(CONFIG_JUMP_LABEL) += jump_label.o
44arm64-obj-$(CONFIG_KGDB) += kgdb.o 39arm64-obj-$(CONFIG_KGDB) += kgdb.o
45arm64-obj-$(CONFIG_EFI) += efi.o $(stub-obj) 40arm64-obj-$(CONFIG_EFI) += efi.o efi-entry.stub.o
46arm64-obj-$(CONFIG_PCI) += pci.o 41arm64-obj-$(CONFIG_PCI) += pci.o
47arm64-obj-$(CONFIG_ARMV8_DEPRECATED) += armv8_deprecated.o 42arm64-obj-$(CONFIG_ARMV8_DEPRECATED) += armv8_deprecated.o
48arm64-obj-$(CONFIG_ACPI) += acpi.o 43arm64-obj-$(CONFIG_ACPI) += acpi.o
diff --git a/drivers/firmware/efi/libstub/Makefile b/drivers/firmware/efi/libstub/Makefile
index bca9a76cbd33..92ae557abbbc 100644
--- a/drivers/firmware/efi/libstub/Makefile
+++ b/drivers/firmware/efi/libstub/Makefile
@@ -34,6 +34,9 @@ $(obj)/lib-%.o: $(srctree)/lib/%.c FORCE
34lib-$(CONFIG_EFI_ARMSTUB) += arm-stub.o fdt.o string.o \ 34lib-$(CONFIG_EFI_ARMSTUB) += arm-stub.o fdt.o string.o \
35 $(patsubst %.c,lib-%.o,$(arm-deps)) 35 $(patsubst %.c,lib-%.o,$(arm-deps))
36 36
37lib-$(CONFIG_ARM64) += arm64-stub.o
38CFLAGS_arm64-stub.o := -DTEXT_OFFSET=$(TEXT_OFFSET)
39
37# 40#
38# arm64 puts the stub in the kernel proper, which will unnecessarily retain all 41# arm64 puts the stub in the kernel proper, which will unnecessarily retain all
39# code indefinitely unless it is annotated as __init/__initdata/__initconst etc. 42# code indefinitely unless it is annotated as __init/__initdata/__initconst etc.
diff --git a/arch/arm64/kernel/efi-stub.c b/drivers/firmware/efi/libstub/arm64-stub.c
index 816120ece6bc..816120ece6bc 100644
--- a/arch/arm64/kernel/efi-stub.c
+++ b/drivers/firmware/efi/libstub/arm64-stub.c