diff options
author | Ard Biesheuvel <ard.biesheuvel@linaro.org> | 2015-10-23 10:48:14 -0400 |
---|---|---|
committer | Catalin Marinas <catalin.marinas@arm.com> | 2015-10-30 12:02:52 -0400 |
commit | bf457786f569cc480629d7855cac1fd1173ac009 (patch) | |
tree | f4f2eea9505db363050ae55eb80b2317b2446eb9 | |
parent | cb083816ab5ac3d10a9417527f07fc5962cc3808 (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/Makefile | 9 | ||||
-rw-r--r-- | drivers/firmware/efi/libstub/Makefile | 3 | ||||
-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 | ||
5 | CPPFLAGS_vmlinux.lds := -DTEXT_OFFSET=$(TEXT_OFFSET) | 5 | CPPFLAGS_vmlinux.lds := -DTEXT_OFFSET=$(TEXT_OFFSET) |
6 | AFLAGS_head.o := -DTEXT_OFFSET=$(TEXT_OFFSET) | 6 | AFLAGS_head.o := -DTEXT_OFFSET=$(TEXT_OFFSET) |
7 | CFLAGS_efi-stub.o := -DTEXT_OFFSET=$(TEXT_OFFSET) | ||
8 | CFLAGS_armv8_deprecated.o := -I$(src) | 7 | CFLAGS_armv8_deprecated.o := -I$(src) |
9 | 8 | ||
10 | KASAN_SANITIZE_efi-stub.o := n | ||
11 | |||
12 | CFLAGS_REMOVE_ftrace.o = -pg | 9 | CFLAGS_REMOVE_ftrace.o = -pg |
13 | CFLAGS_REMOVE_insn.o = -pg | 10 | CFLAGS_REMOVE_insn.o = -pg |
14 | CFLAGS_REMOVE_return_address.o = -pg | 11 | CFLAGS_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 | ||
25 | stub-obj := efi-stub.o efi-entry.o | 22 | extra-$(CONFIG_EFI) := efi-entry.o |
26 | extra-y := $(stub-obj) | ||
27 | stub-obj := $(patsubst %.o,%.stub.o,$(stub-obj)) | ||
28 | 23 | ||
29 | OBJCOPYFLAGS := --prefix-symbols=__efistub_ | 24 | OBJCOPYFLAGS := --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 | |||
42 | arm64-obj-$(CONFIG_CPU_IDLE) += cpuidle.o | 37 | arm64-obj-$(CONFIG_CPU_IDLE) += cpuidle.o |
43 | arm64-obj-$(CONFIG_JUMP_LABEL) += jump_label.o | 38 | arm64-obj-$(CONFIG_JUMP_LABEL) += jump_label.o |
44 | arm64-obj-$(CONFIG_KGDB) += kgdb.o | 39 | arm64-obj-$(CONFIG_KGDB) += kgdb.o |
45 | arm64-obj-$(CONFIG_EFI) += efi.o $(stub-obj) | 40 | arm64-obj-$(CONFIG_EFI) += efi.o efi-entry.stub.o |
46 | arm64-obj-$(CONFIG_PCI) += pci.o | 41 | arm64-obj-$(CONFIG_PCI) += pci.o |
47 | arm64-obj-$(CONFIG_ARMV8_DEPRECATED) += armv8_deprecated.o | 42 | arm64-obj-$(CONFIG_ARMV8_DEPRECATED) += armv8_deprecated.o |
48 | arm64-obj-$(CONFIG_ACPI) += acpi.o | 43 | arm64-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 | |||
34 | lib-$(CONFIG_EFI_ARMSTUB) += arm-stub.o fdt.o string.o \ | 34 | lib-$(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 | ||
37 | lib-$(CONFIG_ARM64) += arm64-stub.o | ||
38 | CFLAGS_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 | |||