* [PATCH v2 0/3] ArmVirtPkg: use PE/COFF metadata for self relocation
@ 2020-06-11 12:52 Ard Biesheuvel
2020-06-11 12:52 ` [PATCH v2 1/3] ArmVirtPkg: add FDF rule for self-relocating PrePi Ard Biesheuvel
` (3 more replies)
0 siblings, 4 replies; 6+ messages in thread
From: Ard Biesheuvel @ 2020-06-11 12:52 UTC (permalink / raw)
To: devel
Cc: Ard Biesheuvel, Laszlo Ersek, Leif Lindholm, Ilias Apalodimas,
Julien Grall, Jiewen Yao, Sami Mujawar
As suggested by Jiewen in response to Ilias RFC [0], it is better to use
the PE/COFF metadata for self-relocating executables than to rely on ELF
metadata, given how the latter is only available when using ELF based
toolchains. Also, we have had some maintenance issues with this code in
the past, as PIE linking of non-position independent objects is not a well
tested code path in toolchains in general.
So implement this for the self-relocating PrePi in ArmVirtPkg first.
First, we need to ensure that the module in question is emitted with its
PE/COFF relocation metadata preserved, by creating a special FDF rule.
We also need to provide a way for the code to refer to the start of the
image directly, by adding it to the linker script.
Then, it is simply a matter of swapping out the two assembly routines,
and adding the C code that serves the same purpose but based on PE/COFF
base relocations.
Note that PE/COFF relocations are considerably more compact than ELF RELA
relocations, so this does not impact the memory footprint of the resulting
image adversely.
[0] https://edk2.groups.io/g/devel/message/60835
Changes since v1:
- Drop change to linker script, and instead, use the existing FV parsing code
(which is already incorporated into PrePi to load other modules), to find
the start address of the image before relocation. This way, we can support
TE images as well as PE32 images naturally, and not rely on GCC/binutils
specific artifacts that make porting to a native PE/COFF toolchain more
difficult
- Switch to TE format in the SELF_RELOC FDF rule - this is not terribly
likely to matter in practice, but since PrePi is the only module that
is incorporated in uncompressed form, and given that we used TE format
before these changes, it is a more appropriate default.
- Add acks from Jiewen, Laszlo and Sami. Note that I have dropped the
Tested-bys - apologies for wasting anyone's time, but they could not
be carried over due to the changes.
Cc: Laszlo Ersek <lersek@redhat.com>
Cc: Leif Lindholm <leif@nuviainc.com>
Cc: Ilias Apalodimas <ilias.apalodimas@linaro.org>
Cc: Julien Grall <julien@xen.org>
Cc: Jiewen Yao <jiewen.yao@intel.com>
Cc: Sami Mujawar <Sami.Mujawar@arm.com>
Ard Biesheuvel (3):
ArmVirtPkg: add FDF rule for self-relocating PrePi
ArmVirtPkg/PrePi: use standard PeCoff routines for self-relocation
ArmVirtPkg: remove unused files
ArmVirtPkg/ArmVirtQemuKernel.dsc | 10 ++--
ArmVirtPkg/ArmVirtXen.dsc | 10 ++--
ArmVirtPkg/ArmVirtQemuKernel.fdf | 2 +-
ArmVirtPkg/ArmVirtXen.fdf | 2 +-
ArmVirtPkg/PrePi/ArmVirtPrePiUniCoreRelocatable.inf | 4 +-
ArmVirtPkg/Include/Platform/Hidden.h | 22 ---------
ArmVirtPkg/PrePi/PrePi.c | 35 ++++++++++++++
ArmVirtPkg/ArmVirtRules.fdf.inc | 5 ++
ArmVirtPkg/PrePi/AArch64/ModuleEntryPoint.S | 49 +++++---------------
ArmVirtPkg/PrePi/Arm/ModuleEntryPoint.S | 47 +++++--------------
ArmVirtPkg/PrePi/Scripts/PrePi-PIE.lds | 41 ----------------
11 files changed, 75 insertions(+), 152 deletions(-)
delete mode 100644 ArmVirtPkg/Include/Platform/Hidden.h
delete mode 100644 ArmVirtPkg/PrePi/Scripts/PrePi-PIE.lds
--
2.26.2
^ permalink raw reply [flat|nested] 6+ messages in thread* [PATCH v2 1/3] ArmVirtPkg: add FDF rule for self-relocating PrePi 2020-06-11 12:52 [PATCH v2 0/3] ArmVirtPkg: use PE/COFF metadata for self relocation Ard Biesheuvel @ 2020-06-11 12:52 ` Ard Biesheuvel 2020-06-11 12:52 ` [PATCH v2 2/3] ArmVirtPkg/PrePi: use standard PeCoff routines for self-relocation Ard Biesheuvel ` (2 subsequent siblings) 3 siblings, 0 replies; 6+ messages in thread From: Ard Biesheuvel @ 2020-06-11 12:52 UTC (permalink / raw) To: devel Cc: Ard Biesheuvel, Laszlo Ersek, Leif Lindholm, Ilias Apalodimas, Julien Grall, Jiewen Yao, Sami Mujawar, Jiewen Yao In preparation for making the self-relocating PrePi use the ordinary BasePeCoffLib routines for relocating the image in place in memory at start, add a special FDF rule that builds SEC modules as PE32 images with the relocation metadata preserved. Signed-off-by: Ard Biesheuvel <ard.biesheuvel@arm.com> Acked-by: Jiewen Yao <Jiewen.yao@intel.com> Reviewed-by: Laszlo Ersek <lersek@redhat.com> Acked-by: Sami Mujawar <Sami.Mujawar@arm.com> --- ArmVirtPkg/ArmVirtQemuKernel.fdf | 2 +- ArmVirtPkg/ArmVirtXen.fdf | 2 +- ArmVirtPkg/ArmVirtRules.fdf.inc | 5 +++++ 3 files changed, 7 insertions(+), 2 deletions(-) diff --git a/ArmVirtPkg/ArmVirtQemuKernel.fdf b/ArmVirtPkg/ArmVirtQemuKernel.fdf index 72fc8dd698f8..55e33aba0d55 100644 --- a/ArmVirtPkg/ArmVirtQemuKernel.fdf +++ b/ArmVirtPkg/ArmVirtQemuKernel.fdf @@ -136,7 +136,7 @@ [FV.FVMAIN_COMPACT] READ_LOCK_CAP = TRUE READ_LOCK_STATUS = TRUE - INF ArmVirtPkg/PrePi/ArmVirtPrePiUniCoreRelocatable.inf + INF RuleOverride = SELF_RELOC ArmVirtPkg/PrePi/ArmVirtPrePiUniCoreRelocatable.inf FILE FV_IMAGE = 9E21FD93-9C72-4c15-8C4B-E77F1DB2D792 { SECTION GUIDED EE4E5898-3914-4259-9D6E-DC7BD79403CF PROCESSING_REQUIRED = TRUE { diff --git a/ArmVirtPkg/ArmVirtXen.fdf b/ArmVirtPkg/ArmVirtXen.fdf index 6a97bceeacbc..f708878f4965 100644 --- a/ArmVirtPkg/ArmVirtXen.fdf +++ b/ArmVirtPkg/ArmVirtXen.fdf @@ -233,7 +233,7 @@ [FV.FVMAIN_COMPACT] READ_LOCK_CAP = TRUE READ_LOCK_STATUS = TRUE - INF ArmVirtPkg/PrePi/ArmVirtPrePiUniCoreRelocatable.inf + INF RuleOverride = SELF_RELOC ArmVirtPkg/PrePi/ArmVirtPrePiUniCoreRelocatable.inf FILE FV_IMAGE = 9E21FD93-9C72-4c15-8C4B-E77F1DB2D792 { SECTION GUIDED EE4E5898-3914-4259-9D6E-DC7BD79403CF PROCESSING_REQUIRED = TRUE { diff --git a/ArmVirtPkg/ArmVirtRules.fdf.inc b/ArmVirtPkg/ArmVirtRules.fdf.inc index 63de26abe056..b8ec040d2330 100644 --- a/ArmVirtPkg/ArmVirtRules.fdf.inc +++ b/ArmVirtPkg/ArmVirtRules.fdf.inc @@ -39,6 +39,11 @@ [Rule.Common.SEC] TE TE Align = Auto $(INF_OUTPUT)/$(MODULE_NAME).efi } +[Rule.Common.SEC.SELF_RELOC] + FILE SEC = $(NAMED_GUID) { + TE TE Align = Auto $(INF_OUTPUT)/$(MODULE_NAME).efi + } + [Rule.Common.PEI_CORE] FILE PEI_CORE = $(NAMED_GUID) FIXED { TE TE Align = Auto $(INF_OUTPUT)/$(MODULE_NAME).efi -- 2.26.2 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH v2 2/3] ArmVirtPkg/PrePi: use standard PeCoff routines for self-relocation 2020-06-11 12:52 [PATCH v2 0/3] ArmVirtPkg: use PE/COFF metadata for self relocation Ard Biesheuvel 2020-06-11 12:52 ` [PATCH v2 1/3] ArmVirtPkg: add FDF rule for self-relocating PrePi Ard Biesheuvel @ 2020-06-11 12:52 ` Ard Biesheuvel 2020-06-11 12:52 ` [PATCH v2 3/3] ArmVirtPkg: remove unused files Ard Biesheuvel 2020-06-11 17:50 ` [PATCH v2 0/3] ArmVirtPkg: use PE/COFF metadata for self relocation Laszlo Ersek 3 siblings, 0 replies; 6+ messages in thread From: Ard Biesheuvel @ 2020-06-11 12:52 UTC (permalink / raw) To: devel Cc: Ard Biesheuvel, Laszlo Ersek, Leif Lindholm, Ilias Apalodimas, Julien Grall, Jiewen Yao, Sami Mujawar, Jiewen Yao Instead of having a GCC specific routine to perform self-relocation based on ELF metadata, use the PE/COFF metadata and the existing PeCoff library routines. This reduces the amount of bespoke assembler code that is a burden to maintain, and is not portable across the set of toolchains we support. This does require some special care, as we have no control over how the C code references global symbols, so we need to emit these references from the calling assembler code. Otherwise, they may be emitted as absolute references, in which case they need to be fixed up themselves, leading to a circular dependency. Signed-off-by: Ard Biesheuvel <ard.biesheuvel@arm.com> Acked-by: Jiewen Yao <Jiewen.yao@intel.com> Acked-by: Laszlo Ersek <lersek@redhat.com> Acked-by: Sami Mujawar <Sami.Mujawar@arm.com> --- ArmVirtPkg/ArmVirtQemuKernel.dsc | 10 ++-- ArmVirtPkg/ArmVirtXen.dsc | 10 ++-- ArmVirtPkg/PrePi/ArmVirtPrePiUniCoreRelocatable.inf | 4 +- ArmVirtPkg/PrePi/PrePi.c | 35 ++++++++++++++ ArmVirtPkg/PrePi/AArch64/ModuleEntryPoint.S | 49 +++++--------------- ArmVirtPkg/PrePi/Arm/ModuleEntryPoint.S | 47 +++++-------------- 6 files changed, 68 insertions(+), 87 deletions(-) diff --git a/ArmVirtPkg/ArmVirtQemuKernel.dsc b/ArmVirtPkg/ArmVirtQemuKernel.dsc index 2a6fd6bc06be..9449a01d6e40 100644 --- a/ArmVirtPkg/ArmVirtQemuKernel.dsc +++ b/ArmVirtPkg/ArmVirtQemuKernel.dsc @@ -83,14 +83,12 @@ [LibraryClasses.common.DXE_DRIVER] [LibraryClasses.common.UEFI_DRIVER] UefiScsiLib|MdePkg/Library/UefiScsiLib/UefiScsiLib.inf -[BuildOptions.common.EDKII.SEC, BuildOptions.common.EDKII.BASE] +[BuildOptions] # - # CLANG38 with LTO support enabled uses the GNU GOLD linker, which insists - # on emitting GOT based symbol references when running in shared mode, unless - # we override visibility to 'hidden' in all modules that make up the PrePi - # build. + # We need to avoid jump tables in SEC modules, so that the PE/COFF + # self-relocation code itself is guaranteed to be position independent. # - GCC:*_CLANG38_*_CC_FLAGS = -include $(WORKSPACE)/ArmVirtPkg/Include/Platform/Hidden.h + GCC:*_*_*_CC_FLAGS = -fno-jump-tables ################################################################################ # diff --git a/ArmVirtPkg/ArmVirtXen.dsc b/ArmVirtPkg/ArmVirtXen.dsc index 8a489b253684..278f5d3828ce 100644 --- a/ArmVirtPkg/ArmVirtXen.dsc +++ b/ArmVirtPkg/ArmVirtXen.dsc @@ -52,14 +52,12 @@ [LibraryClasses] [LibraryClasses.common.UEFI_DRIVER] UefiScsiLib|MdePkg/Library/UefiScsiLib/UefiScsiLib.inf -[BuildOptions.common.EDKII.SEC, BuildOptions.common.EDKII.BASE] +[BuildOptions] # - # CLANG38 with LTO support enabled uses the GNU GOLD linker, which insists - # on emitting GOT based symbol references when running in shared mode, unless - # we override visibility to 'hidden' in all modules that make up the PrePi - # build. + # We need to avoid jump tables in SEC modules, so that the PE/COFF + # self-relocation code itself is guaranteed to be position independent. # - GCC:*_CLANG38_*_CC_FLAGS = -include $(WORKSPACE)/ArmVirtPkg/Include/Platform/Hidden.h + GCC:*_*_*_CC_FLAGS = -fno-jump-tables ################################################################################ # diff --git a/ArmVirtPkg/PrePi/ArmVirtPrePiUniCoreRelocatable.inf b/ArmVirtPkg/PrePi/ArmVirtPrePiUniCoreRelocatable.inf index 9e58e56fce09..7edf5018089d 100755 --- a/ArmVirtPkg/PrePi/ArmVirtPrePiUniCoreRelocatable.inf +++ b/ArmVirtPkg/PrePi/ArmVirtPrePiUniCoreRelocatable.inf @@ -46,6 +46,7 @@ [LibraryClasses] SerialPortLib ExtractGuidedSectionLib LzmaDecompressLib + PeCoffLib PrePiLib MemoryAllocationLib HobLib @@ -95,6 +96,3 @@ [Pcd] gArmVirtTokenSpaceGuid.PcdDeviceTreeInitialBaseAddress gArmTokenSpaceGuid.PcdFdBaseAddress gArmTokenSpaceGuid.PcdFvBaseAddress - -[BuildOptions] - GCC:*_*_*_DLINK_FLAGS = -Wl,-Bsymbolic,-pie,-T,$(MODULE_DIR)/Scripts/PrePi-PIE.lds diff --git a/ArmVirtPkg/PrePi/PrePi.c b/ArmVirtPkg/PrePi/PrePi.c index 72e35028c5bb..4f0c3f98bad6 100755 --- a/ArmVirtPkg/PrePi/PrePi.c +++ b/ArmVirtPkg/PrePi/PrePi.c @@ -9,6 +9,7 @@ #include <PiPei.h> #include <Pi/PiBootMode.h> +#include <Library/PeCoffLib.h> #include <Library/PrePiLib.h> #include <Library/PrintLib.h> #include <Library/PrePiHobListPointerLib.h> @@ -128,3 +129,37 @@ CEntryPoint ( // DXE Core should always load and never return ASSERT (FALSE); } + +VOID +RelocatePeCoffImage ( + IN EFI_PEI_FV_HANDLE FwVolHeader, + IN PE_COFF_LOADER_READ_FILE ImageRead + ) +{ + EFI_PEI_FILE_HANDLE FileHandle; + VOID *SectionData; + PE_COFF_LOADER_IMAGE_CONTEXT ImageContext; + EFI_STATUS Status; + + FileHandle = NULL; + Status = FfsFindNextFile (EFI_FV_FILETYPE_SECURITY_CORE, FwVolHeader, + &FileHandle); + ASSERT_EFI_ERROR (Status); + + Status = FfsFindSectionData (EFI_SECTION_PE32, FileHandle, &SectionData); + if (EFI_ERROR (Status)) { + Status = FfsFindSectionData (EFI_SECTION_TE, FileHandle, &SectionData); + } + ASSERT_EFI_ERROR (Status); + + ZeroMem (&ImageContext, sizeof ImageContext); + + ImageContext.Handle = (EFI_HANDLE)SectionData; + ImageContext.ImageRead = ImageRead; + PeCoffLoaderGetImageInfo (&ImageContext); + + if (ImageContext.ImageAddress != (UINTN)SectionData) { + ImageContext.ImageAddress = (UINTN)SectionData; + PeCoffLoaderRelocateImage (&ImageContext); + } +} diff --git a/ArmVirtPkg/PrePi/AArch64/ModuleEntryPoint.S b/ArmVirtPkg/PrePi/AArch64/ModuleEntryPoint.S index 34d6abecb0ac..01623b6b3591 100644 --- a/ArmVirtPkg/PrePi/AArch64/ModuleEntryPoint.S +++ b/ArmVirtPkg/PrePi/AArch64/ModuleEntryPoint.S @@ -9,40 +9,6 @@ #include <AsmMacroIoLibV8.h> ASM_FUNC(_ModuleEntryPoint) - // - // We are built as a ET_DYN PIE executable, so we need to process all - // relative relocations regardless of whether or not we are executing from - // the same offset we were linked at. This is only possible if we are - // running from RAM. - // - adr x8, __reloc_base - adr x9, __reloc_start - adr x10, __reloc_end - -.Lreloc_loop: - cmp x9, x10 - bhs .Lreloc_done - - // - // AArch64 uses the ELF64 RELA format, which means each entry in the - // relocation table consists of - // - // UINT64 offset : the relative offset of the value that needs to - // be relocated - // UINT64 info : relocation type and symbol index (the latter is - // not used for R_AARCH64_RELATIVE relocations) - // UINT64 addend : value to be added to the value being relocated - // - ldp x11, x12, [x9], #24 // read offset into x11 and info into x12 - cmp x12, #0x403 // check info == R_AARCH64_RELATIVE? - bne .Lreloc_loop // not a relative relocation? then skip - - ldr x12, [x9, #-8] // read addend into x12 - add x12, x12, x8 // add reloc base to addend to get relocated value - str x12, [x11, x8] // write relocated value at offset - b .Lreloc_loop -.Lreloc_done: - bl ASM_PFX(DiscoverDramFromDt) // Get ID of this CPU in Multicore system @@ -170,15 +136,24 @@ ASM_PFX(DiscoverDramFromDt): str x1, [x8] str x7, [x9] + // + // The runtime address may be different from the link time address so fix + // up the PE/COFF relocations. Since we are calling a C function, use the + // window at the beginning of the FD image as a temp stack. + // + mov x0, x7 + adr x1, PeCoffLoaderImageReadFromMemory + mov sp, x7 + bl RelocatePeCoffImage + // // Discover the memory size and offset from the DTB, and record in the // respective PCDs. This will also return false if a corrupt DTB is - // encountered. Since we are calling a C function, use the window at the - // beginning of the FD image as a temp stack. + // encountered. // + mov x0, x28 adr x1, PcdGet64 (PcdSystemMemoryBase) adr x2, PcdGet64 (PcdSystemMemorySize) - mov sp, x7 bl FindMemnode cbz x0, .Lout diff --git a/ArmVirtPkg/PrePi/Arm/ModuleEntryPoint.S b/ArmVirtPkg/PrePi/Arm/ModuleEntryPoint.S index 72d756fdb571..f0536c65eb52 100644 --- a/ArmVirtPkg/PrePi/Arm/ModuleEntryPoint.S +++ b/ArmVirtPkg/PrePi/Arm/ModuleEntryPoint.S @@ -9,38 +9,6 @@ #include <AsmMacroIoLib.h> ASM_FUNC(_ModuleEntryPoint) - // - // We are built as a ET_DYN PIE executable, so we need to process all - // relative relocations if we are executing from a different offset than we - // were linked at. This is only possible if we are running from RAM. - // - ADRL (r4, __reloc_base) - ADRL (r5, __reloc_start) - ADRL (r6, __reloc_end) - -.Lreloc_loop: - cmp r5, r6 - bhs .Lreloc_done - - // - // AArch32 uses the ELF32 REL format, which means each entry in the - // relocation table consists of - // - // UINT32 offset : the relative offset of the value that needs to - // be relocated - // UINT32 info : relocation type and symbol index (the latter is - // not used for R_ARM_RELATIVE relocations) - // - ldrd r8, r9, [r5], #8 // read offset into r8 and info into r9 - cmp r9, #23 // check info == R_ARM_RELATIVE? - bne .Lreloc_loop // not a relative relocation? then skip - - ldr r9, [r8, r4] // read addend into r9 - add r9, r9, r1 // add image base to addend to get relocated value - str r9, [r8, r4] // write relocated value at offset - b .Lreloc_loop -.Lreloc_done: - // Do early platform specific actions bl ASM_PFX(ArmPlatformPeiBootAction) @@ -172,15 +140,24 @@ ASM_PFX(ArmPlatformPeiBootAction): str r1, [r8] str r5, [r7] + // + // The runtime address may be different from the link time address so fix + // up the PE/COFF relocations. Since we are calling a C function, use the + // window at the beginning of the FD image as a temp stack. + // + mov r0, r5 + ADRL (r1, PeCoffLoaderImageReadFromMemory) + mov sp, r5 + bl RelocatePeCoffImage + // // Discover the memory size and offset from the DTB, and record in the // respective PCDs. This will also return false if a corrupt DTB is - // encountered. Since we are calling a C function, use the window at the - // beginning of the FD image as a temp stack. + // encountered. // + mov r0, r10 ADRL (r1, PcdGet64 (PcdSystemMemoryBase)) ADRL (r2, PcdGet64 (PcdSystemMemorySize)) - mov sp, r5 bl FindMemnode teq r0, #0 beq .Lout -- 2.26.2 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH v2 3/3] ArmVirtPkg: remove unused files 2020-06-11 12:52 [PATCH v2 0/3] ArmVirtPkg: use PE/COFF metadata for self relocation Ard Biesheuvel 2020-06-11 12:52 ` [PATCH v2 1/3] ArmVirtPkg: add FDF rule for self-relocating PrePi Ard Biesheuvel 2020-06-11 12:52 ` [PATCH v2 2/3] ArmVirtPkg/PrePi: use standard PeCoff routines for self-relocation Ard Biesheuvel @ 2020-06-11 12:52 ` Ard Biesheuvel 2020-06-11 17:50 ` [PATCH v2 0/3] ArmVirtPkg: use PE/COFF metadata for self relocation Laszlo Ersek 3 siblings, 0 replies; 6+ messages in thread From: Ard Biesheuvel @ 2020-06-11 12:52 UTC (permalink / raw) To: devel Cc: Ard Biesheuvel, Laszlo Ersek, Leif Lindholm, Ilias Apalodimas, Julien Grall, Jiewen Yao, Sami Mujawar, Jiewen Yao We no longer use ELF PIE executables to implement the self-relocating PrePi so drop the custom linker script and visibility override header file. Signed-off-by: Ard Biesheuvel <ard.biesheuvel@arm.com> Acked-by: Jiewen Yao <Jiewen.yao@intel.com> Reviewed-by: Laszlo Ersek <lersek@redhat.com> Acked-by: Sami Mujawar <Sami.Mujawar@arm.com> --- ArmVirtPkg/Include/Platform/Hidden.h | 22 ----------- ArmVirtPkg/PrePi/Scripts/PrePi-PIE.lds | 41 -------------------- 2 files changed, 63 deletions(-) diff --git a/ArmVirtPkg/Include/Platform/Hidden.h b/ArmVirtPkg/Include/Platform/Hidden.h deleted file mode 100644 index 7a7bdb42b8bd..000000000000 --- a/ArmVirtPkg/Include/Platform/Hidden.h +++ /dev/null @@ -1,22 +0,0 @@ -/** @file - - Copyright (c) 2018, Linaro Limited. All rights reserved. - - SPDX-License-Identifier: BSD-2-Clause-Patent - -**/ - -#ifndef __PLATFORM_HIDDEN_H -#define __PLATFORM_HIDDEN_H - -// -// Setting the GCC -fvisibility=hidden command line option is not quite the same -// as setting the pragma below: the former only affects definitions, whereas the -// pragma affects extern declarations as well. So if we want to ensure that no -// GOT indirected symbol references are emitted, we need to use the pragma, or -// GOT based cross object references could be emitted, e.g., in libraries, and -// these cannot be relaxed to ordinary symbol references at link time. -// -#pragma GCC visibility push (hidden) - -#endif diff --git a/ArmVirtPkg/PrePi/Scripts/PrePi-PIE.lds b/ArmVirtPkg/PrePi/Scripts/PrePi-PIE.lds deleted file mode 100644 index c9a15ca3493a..000000000000 --- a/ArmVirtPkg/PrePi/Scripts/PrePi-PIE.lds +++ /dev/null @@ -1,41 +0,0 @@ -/** @file - - Copyright (c) 2015, Linaro Ltd. All rights reserved.<BR> - - SPDX-License-Identifier: BSD-2-Clause-Patent - -**/ - -SECTIONS -{ - PROVIDE(__reloc_base = .); - - . = PECOFF_HEADER_SIZE; - .text : ALIGN(CONSTANT(COMMONPAGESIZE)) { - *(.text .text*) - *(.got .got*) - *(.rodata .rodata*) - *(.data .data*) - *(.bss .bss*) - - . = ALIGN(0x20); - PROVIDE(__reloc_start = .); - *(.rel .rel.*) - *(.rela .rela.*) - PROVIDE(__reloc_end = .); - } - - .note (INFO) : { *(.note.gnu.build-id) } - - /DISCARD/ : { - *(.note.GNU-stack) - *(.gnu.hash) - *(.gnu_debuglink) - *(.interp) - *(.dynamic) - *(.dynsym) - *(.dynstr) - *(.hash) - *(.comment) - } -} -- 2.26.2 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v2 0/3] ArmVirtPkg: use PE/COFF metadata for self relocation 2020-06-11 12:52 [PATCH v2 0/3] ArmVirtPkg: use PE/COFF metadata for self relocation Ard Biesheuvel ` (2 preceding siblings ...) 2020-06-11 12:52 ` [PATCH v2 3/3] ArmVirtPkg: remove unused files Ard Biesheuvel @ 2020-06-11 17:50 ` Laszlo Ersek 2020-06-12 22:18 ` Ard Biesheuvel 3 siblings, 1 reply; 6+ messages in thread From: Laszlo Ersek @ 2020-06-11 17:50 UTC (permalink / raw) To: Ard Biesheuvel, devel Cc: Leif Lindholm, Ilias Apalodimas, Julien Grall, Jiewen Yao, Sami Mujawar On 06/11/20 14:52, Ard Biesheuvel wrote: > As suggested by Jiewen in response to Ilias RFC [0], it is better to use > the PE/COFF metadata for self-relocating executables than to rely on ELF > metadata, given how the latter is only available when using ELF based > toolchains. Also, we have had some maintenance issues with this code in > the past, as PIE linking of non-position independent objects is not a well > tested code path in toolchains in general. > > So implement this for the self-relocating PrePi in ArmVirtPkg first. > > First, we need to ensure that the module in question is emitted with its > PE/COFF relocation metadata preserved, by creating a special FDF rule. > > We also need to provide a way for the code to refer to the start of the > image directly, by adding it to the linker script. > > Then, it is simply a matter of swapping out the two assembly routines, > and adding the C code that serves the same purpose but based on PE/COFF > base relocations. > > Note that PE/COFF relocations are considerably more compact than ELF RELA > relocations, so this does not impact the memory footprint of the resulting > image adversely. > > [0] https://edk2.groups.io/g/devel/message/60835 > > Changes since v1: > - Drop change to linker script, and instead, use the existing FV parsing code > (which is already incorporated into PrePi to load other modules), to find > the start address of the image before relocation. This way, we can support > TE images as well as PE32 images naturally, and not rely on GCC/binutils > specific artifacts that make porting to a native PE/COFF toolchain more > difficult > - Switch to TE format in the SELF_RELOC FDF rule - this is not terribly > likely to matter in practice, but since PrePi is the only module that > is incorporated in uncompressed form, and given that we used TE format > before these changes, it is a more appropriate default. Right, I noticed that when I compared the new rule in v1 against the pre-existent SEC rule. I'm happy to see my feedback tags carried forward. Thanks Laszlo > - Add acks from Jiewen, Laszlo and Sami. Note that I have dropped the > Tested-bys - apologies for wasting anyone's time, but they could not > be carried over due to the changes. > > Cc: Laszlo Ersek <lersek@redhat.com> > Cc: Leif Lindholm <leif@nuviainc.com> > Cc: Ilias Apalodimas <ilias.apalodimas@linaro.org> > Cc: Julien Grall <julien@xen.org> > Cc: Jiewen Yao <jiewen.yao@intel.com> > Cc: Sami Mujawar <Sami.Mujawar@arm.com> > > Ard Biesheuvel (3): > ArmVirtPkg: add FDF rule for self-relocating PrePi > ArmVirtPkg/PrePi: use standard PeCoff routines for self-relocation > ArmVirtPkg: remove unused files > > ArmVirtPkg/ArmVirtQemuKernel.dsc | 10 ++-- > ArmVirtPkg/ArmVirtXen.dsc | 10 ++-- > ArmVirtPkg/ArmVirtQemuKernel.fdf | 2 +- > ArmVirtPkg/ArmVirtXen.fdf | 2 +- > ArmVirtPkg/PrePi/ArmVirtPrePiUniCoreRelocatable.inf | 4 +- > ArmVirtPkg/Include/Platform/Hidden.h | 22 --------- > ArmVirtPkg/PrePi/PrePi.c | 35 ++++++++++++++ > ArmVirtPkg/ArmVirtRules.fdf.inc | 5 ++ > ArmVirtPkg/PrePi/AArch64/ModuleEntryPoint.S | 49 +++++--------------- > ArmVirtPkg/PrePi/Arm/ModuleEntryPoint.S | 47 +++++-------------- > ArmVirtPkg/PrePi/Scripts/PrePi-PIE.lds | 41 ---------------- > 11 files changed, 75 insertions(+), 152 deletions(-) > delete mode 100644 ArmVirtPkg/Include/Platform/Hidden.h > delete mode 100644 ArmVirtPkg/PrePi/Scripts/PrePi-PIE.lds > ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2 0/3] ArmVirtPkg: use PE/COFF metadata for self relocation 2020-06-11 17:50 ` [PATCH v2 0/3] ArmVirtPkg: use PE/COFF metadata for self relocation Laszlo Ersek @ 2020-06-12 22:18 ` Ard Biesheuvel 0 siblings, 0 replies; 6+ messages in thread From: Ard Biesheuvel @ 2020-06-12 22:18 UTC (permalink / raw) To: Laszlo Ersek, devel Cc: Leif Lindholm, Ilias Apalodimas, Julien Grall, Jiewen Yao, Sami Mujawar On 6/11/20 7:50 PM, Laszlo Ersek wrote: > On 06/11/20 14:52, Ard Biesheuvel wrote: >> As suggested by Jiewen in response to Ilias RFC [0], it is better to use >> the PE/COFF metadata for self-relocating executables than to rely on ELF >> metadata, given how the latter is only available when using ELF based >> toolchains. Also, we have had some maintenance issues with this code in >> the past, as PIE linking of non-position independent objects is not a well >> tested code path in toolchains in general. >> >> So implement this for the self-relocating PrePi in ArmVirtPkg first. >> >> First, we need to ensure that the module in question is emitted with its >> PE/COFF relocation metadata preserved, by creating a special FDF rule. >> >> We also need to provide a way for the code to refer to the start of the >> image directly, by adding it to the linker script. >> >> Then, it is simply a matter of swapping out the two assembly routines, >> and adding the C code that serves the same purpose but based on PE/COFF >> base relocations. >> >> Note that PE/COFF relocations are considerably more compact than ELF RELA >> relocations, so this does not impact the memory footprint of the resulting >> image adversely. >> >> [0] https://edk2.groups.io/g/devel/message/60835 >> >> Changes since v1: >> - Drop change to linker script, and instead, use the existing FV parsing code >> (which is already incorporated into PrePi to load other modules), to find >> the start address of the image before relocation. This way, we can support >> TE images as well as PE32 images naturally, and not rely on GCC/binutils >> specific artifacts that make porting to a native PE/COFF toolchain more >> difficult >> - Switch to TE format in the SELF_RELOC FDF rule - this is not terribly >> likely to matter in practice, but since PrePi is the only module that >> is incorporated in uncompressed form, and given that we used TE format >> before these changes, it is a more appropriate default. > > Right, I noticed that when I compared the new rule in v1 against the > pre-existent SEC rule. I'm happy to see my feedback tags carried forward. > > Thanks > Laszlo > Merged as #692 Thanks all. >> - Add acks from Jiewen, Laszlo and Sami. Note that I have dropped the >> Tested-bys - apologies for wasting anyone's time, but they could not >> be carried over due to the changes. >> >> Cc: Laszlo Ersek <lersek@redhat.com> >> Cc: Leif Lindholm <leif@nuviainc.com> >> Cc: Ilias Apalodimas <ilias.apalodimas@linaro.org> >> Cc: Julien Grall <julien@xen.org> >> Cc: Jiewen Yao <jiewen.yao@intel.com> >> Cc: Sami Mujawar <Sami.Mujawar@arm.com> >> >> Ard Biesheuvel (3): >> ArmVirtPkg: add FDF rule for self-relocating PrePi >> ArmVirtPkg/PrePi: use standard PeCoff routines for self-relocation >> ArmVirtPkg: remove unused files >> >> ArmVirtPkg/ArmVirtQemuKernel.dsc | 10 ++-- >> ArmVirtPkg/ArmVirtXen.dsc | 10 ++-- >> ArmVirtPkg/ArmVirtQemuKernel.fdf | 2 +- >> ArmVirtPkg/ArmVirtXen.fdf | 2 +- >> ArmVirtPkg/PrePi/ArmVirtPrePiUniCoreRelocatable.inf | 4 +- >> ArmVirtPkg/Include/Platform/Hidden.h | 22 --------- >> ArmVirtPkg/PrePi/PrePi.c | 35 ++++++++++++++ >> ArmVirtPkg/ArmVirtRules.fdf.inc | 5 ++ >> ArmVirtPkg/PrePi/AArch64/ModuleEntryPoint.S | 49 +++++--------------- >> ArmVirtPkg/PrePi/Arm/ModuleEntryPoint.S | 47 +++++-------------- >> ArmVirtPkg/PrePi/Scripts/PrePi-PIE.lds | 41 ---------------- >> 11 files changed, 75 insertions(+), 152 deletions(-) >> delete mode 100644 ArmVirtPkg/Include/Platform/Hidden.h >> delete mode 100644 ArmVirtPkg/PrePi/Scripts/PrePi-PIE.lds >> > ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2020-06-12 22:18 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2020-06-11 12:52 [PATCH v2 0/3] ArmVirtPkg: use PE/COFF metadata for self relocation Ard Biesheuvel 2020-06-11 12:52 ` [PATCH v2 1/3] ArmVirtPkg: add FDF rule for self-relocating PrePi Ard Biesheuvel 2020-06-11 12:52 ` [PATCH v2 2/3] ArmVirtPkg/PrePi: use standard PeCoff routines for self-relocation Ard Biesheuvel 2020-06-11 12:52 ` [PATCH v2 3/3] ArmVirtPkg: remove unused files Ard Biesheuvel 2020-06-11 17:50 ` [PATCH v2 0/3] ArmVirtPkg: use PE/COFF metadata for self relocation Laszlo Ersek 2020-06-12 22:18 ` Ard Biesheuvel
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox