[PATCH V6 0/8] Add ARM EFI stub
Roy Franz
roy.franz at linaro.org
Fri Jan 10 11:30:09 EST 2014
This patch series adds EFI stub support for the ARM architecture. The
stub for ARM is implemented in a similar manner to x86 in that it is a
shim layer between EFI and the normal zImage/bzImage boot process, and
that an image with the stub configured is bootable as both a zImage and
EFI application. This patchset adds common code that is shared with
the ARM64 EFI stub.
This patch depends on Leif Lindholm's ARM v7 runtime services patchset, which
in turn depends on Mark Salter's early_io_remap() patchset. This EFI stub
patchset can be tested without those by simply modifying the Kconfig for
EFI_STUB to not depend on EFI. The kernel will boot using the stub,
however no EFI support will be available in the kernel.
I have addressed all the feedback I have recieved to date, and I am hoping
that this patchset is acceptable for merging into 3.14. It has been
functionally stable for some time, and provides useful functionality.
Further consolidation of the ARM/ARM64 stubs planned as part of the
ongoing development of the stubs and supporting tools for both
architectures (getting the FDT from a UEFI configuration table, using UEFI
memory map instead of FDT for memory description.)
Changes since v5:
* Move more code dealing with the Device tree to common code shared by
ARM and ARM64.
* move formatting of informative and error messaging to common code to
harmonize formatting between ARM/ARM64 stubs.
Changes since v4:
* Common and x86 EFI stub changes that this patch depends on were merged
in 3.13.
* Updated FDT bindings to match agreed upon bindings in Leif's runtime
services patchset. This includes adding the kernel_banner version string,
and changing how the EFI memory map is passed. The EFI memory map is now
allocated as EFI_RUNTIME_SERVICES_DATA since it may be passed to the kernel.
The FDT bindings for arm/arm64 are identical.
* Added space in PE/COFF header for signature to allow signing of the image
for EFI secure boot.
* Moved shared FDT function to drivers/firmware/efi/fdt.c. This will be shared
with arm64. (Mark Salter has a patchset for the arm64 stub.)
Changes since v3:
* Common and x86 EFI stub changes have been broken out and submitted separately.
This patch series depends on the "[PATCH V5 00/18] ARM EFI stub common code"
series as merged by Matt Fleming. (Patch 10 of that series omitted)
* Re-added -fno-stack-protector. Needed for compilation with
CONFIG_CC_STACKPROTECTOR option. Cannot easily add support for stack
protection since there are no functions in stub that never return, which
is a requirement for use. (x86 stub also disables stack protector.)
Note that even in the absense of -fno-stack-protector, stack protection
was never present in the decompressor, as the required functions are not
implemented.
* Fixed long summaries in commit messages.
* Further cleanups of assembly in head.S
Changes since v2:
* Now depends on "arm: Add [U]EFI runtime services support" patches by leif.lindholm at linaro.org.
The EFI memory map is the only memory map passed to the kernel, so EFI support
is now required.
* remove "-fno-stack-protector" from decompressor Makefile. The current code doesn't
trigger the stack protection, so the decompressor now compiles with it still
on. Note that there has never been any stack protection in the decompressor
since the stack usage doesn't trigger the heuristic in GCC, so right now
the "-fno-stack-protector" is a noop.
* Changed EFI command line handling to not have a fixed limit.
* Change FDT memory allocation to retry with a larger allocation if
first educated guess is inadequate.
* Correctly set 'SizeOfCode' in PE/COFF header.
* Reviewed ".setup" section that is in x86 PE/COFF header. This is used for x86
to account for code that is not in the .text section. We don't need this
for ARM, as all of our code is in the .text section, or in the PE/COFF header
itself.
* Moved EFI_STUB_ERROR #define to header file to share between stub C and ASM.
* Variety of cleanups and fixes in head.S.
* Changed update_fdt_and_exit_boot() to just update the device tree, and
renamed appropriately. Memory allocations moved out of this function as
well, which enables the retries if the initial FDT size is too small.
Note that in order to do the retried allocations, the original FDT and
command line memory regions are left allocated. This is OK since the kernel
has the memory map and will free these allocations along with the initrd
and new fdt allocations.
* Added prefix to all prints, reduced number of prints, and reviewed all
messages.
* Change mixed usage of dtb/fdt to all be fdt or "device tree" in efi-stub.c
* remove unnecessary zimage_size variable from relocate_kernel()
* correct return types on EFI functions - should be efi_status_t, not int.
Roy Franz (8):
efi-stub.txt updates for ARM
Add shared printk wrapper for consistent prefixing
Add helper function to get and convert EFI command line
Add shared FDT related functions for ARM/ARM64
Add strstr to compressed string.c for ARM.
Add EFI stub for ARM
Disable stack protection for decompressor/stub
Add config EFI_STUB for ARM to Kconfig
Documentation/efi-stub.txt | 27 +++-
arch/arm/Kconfig | 11 ++
arch/arm/boot/compressed/Makefile | 17 ++-
arch/arm/boot/compressed/efi-header.S | 117 +++++++++++++++++
arch/arm/boot/compressed/efi-stub.c | 214 ++++++++++++++++++++++++++++++
arch/arm/boot/compressed/efi-stub.h | 5 +
arch/arm/boot/compressed/head.S | 83 +++++++++++-
arch/arm/boot/compressed/string.c | 21 +++
drivers/firmware/efi/efi-stub-helper.c | 55 ++++++--
drivers/firmware/efi/fdt.c | 225 ++++++++++++++++++++++++++++++++
10 files changed, 748 insertions(+), 27 deletions(-)
create mode 100644 arch/arm/boot/compressed/efi-header.S
create mode 100644 arch/arm/boot/compressed/efi-stub.c
create mode 100644 arch/arm/boot/compressed/efi-stub.h
create mode 100644 drivers/firmware/efi/fdt.c
--
1.7.10.4
More information about the linux-arm-kernel
mailing list