LM
/
u-boot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

efi_loader: fix building aarch64 EFI binaries 

While our EFI binaries execute without problems on EDK II they crash on
a Lenovo X13s. Let our binaries look more like what EDK II produces:

* move all writable data to a .data section
* align sections to 4 KiB boundaries (matching EFI page size)
* remove IMAGE_SCN_LNK_NRELOC_OVFL from .reloc section flags

Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
Reviewed-by: Ilias Apalodimas <ilias.apalodimas@linaro.org>
This commit is contained in:
Heinrich Schuchardt 2022-12-31 11:58:54 +01:00
parent 673a92c5d2
commit d7ddeb66a6
2 changed files with 38 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

47
arch/arm/lib/crt0_aarch64_efi.S
View File

@ -25,7 +25,7 @@ pe_header:
.long IMAGE_NT_SIGNATURE /* 'PE' */ .long IMAGE_NT_SIGNATURE /* 'PE' */
coff_header: coff_header:
.short IMAGE_FILE_MACHINE_ARM64 /* AArch64 */ .short IMAGE_FILE_MACHINE_ARM64 /* AArch64 */
.short 2 /* nr_sections */ .short 3 /* nr_sections */
.long 0 /* TimeDateStamp */ .long 0 /* TimeDateStamp */
.long 0 /* PointerToSymbolTable */ .long 0 /* PointerToSymbolTable */
.long 0 /* NumberOfSymbols */ .long 0 /* NumberOfSymbols */
@ -40,7 +40,7 @@ optional_header:
.short IMAGE_NT_OPTIONAL_HDR64_MAGIC /* PE32+ format */ .short IMAGE_NT_OPTIONAL_HDR64_MAGIC /* PE32+ format */
.byte 0x02 /* MajorLinkerVersion */ .byte 0x02 /* MajorLinkerVersion */
.byte 0x14 /* MinorLinkerVersion */ .byte 0x14 /* MinorLinkerVersion */
.long _edata - _start /* SizeOfCode */ .long _etext - _start /* SizeOfCode */
.long 0 /* SizeOfInitializedData */ .long 0 /* SizeOfInitializedData */
.long 0 /* SizeOfUninitializedData */ .long 0 /* SizeOfUninitializedData */
.long _start - ImageBase /* AddressOfEntryPoint */ .long _start - ImageBase /* AddressOfEntryPoint */
@ -48,7 +48,7 @@ optional_header:
extra_header_fields: extra_header_fields:
.quad 0 /* ImageBase */ .quad 0 /* ImageBase */
.long 0x200 /* SectionAlignment */ .long 0x1000 /* SectionAlignment */
.long 0x200 /* FileAlignment */ .long 0x200 /* FileAlignment */
.short 0 /* MajorOperatingSystemVersion */ .short 0 /* MajorOperatingSystemVersion */
.short 0 /* MinorOperatingSystemVersion */ .short 0 /* MinorOperatingSystemVersion */
@ -100,25 +100,46 @@ section_table:
.long 0 /* PointerToLineNumbers */ .long 0 /* PointerToLineNumbers */
.short 0 /* NumberOfRelocations */ .short 0 /* NumberOfRelocations */
.short 0 /* NumberOfLineNumbers */ .short 0 /* NumberOfLineNumbers */
.long 0x42100040 /* Characteristics (section flags) */ /* Characteristics (section flags) */
.long (IMAGE_SCN_MEM_READ | \
IMAGE_SCN_MEM_DISCARDABLE | \
IMAGE_SCN_CNT_INITIALIZED_DATA)
.ascii ".text" .ascii ".text"
.byte 0 .byte 0
.byte 0 .byte 0
.byte 0 /* end of 0 padding of section name */ .byte 0 /* end of 0 padding of section name */
.long _edata - _start /* VirtualSize */ .long _etext - _start /* VirtualSize */
.long _start - ImageBase /* VirtualAddress */ .long _start - ImageBase /* VirtualAddress */
.long _edata - _start /* SizeOfRawData */ .long _etext - _start /* SizeOfRawData */
.long _start - ImageBase /* PointerToRawData */ .long _start - ImageBase /* PointerToRawData */
.long 0 /* PointerToRelocations */
.long 0 /* PointerToLineNumbers */
.short 0 /* NumberOfRelocations */
.short 0 /* NumberOfLineNumbers */
/* Characteristics (section flags) */
.long (IMAGE_SCN_MEM_READ | \
IMAGE_SCN_MEM_EXECUTE | \
IMAGE_SCN_CNT_CODE)
.long 0 /* PointerToRelocations (0 for executables) */ .ascii ".data"
.long 0 /* PointerToLineNumbers (0 for executables) */ .byte 0
.short 0 /* NumberOfRelocations (0 for executables) */ .byte 0
.short 0 /* NumberOfLineNumbers (0 for executables) */ .byte 0 /* end of 0 padding of section name */
.long 0xe0500020 /* Characteristics (section flags) */ .long _data_size /* VirtualSize */
.long _data - ImageBase /* VirtualAddress */
.long _data_size /* SizeOfRawData */
.long _data - ImageBase /* PointerToRawData */
.long 0 /* PointerToRelocations */
.long 0 /* PointerToLineNumbers */
.short 0 /* NumberOfRelocations */
.short 0 /* NumberOfLineNumbers */
/* Characteristics (section flags) */
.long (IMAGE_SCN_MEM_WRITE | \
IMAGE_SCN_MEM_READ | \
IMAGE_SCN_CNT_INITIALIZED_DATA)
.align 9 .align 12
_start: _start:
stp x29, x30, [sp, #-32]! stp x29, x30, [sp, #-32]!
mov x29, sp mov x29, sp

6
arch/arm/lib/elf_aarch64_efi.lds
View File

@ -18,11 +18,13 @@ SECTIONS
*(.gnu.linkonce.t.*) *(.gnu.linkonce.t.*)
*(.srodata) *(.srodata)
*(.rodata*) *(.rodata*)
. = ALIGN(16);
*(.dynamic);
. = ALIGN(512); . = ALIGN(512);
} }
_etext = .; _etext = .;
_text_size = . - _text; _text_size = . - _text;
.dynamic : { *(.dynamic) } . = ALIGN(4096);
.data : { .data : {
_data = .; _data = .;
*(.sdata) *(.sdata)
@ -48,11 +50,11 @@ SECTIONS
_bss_end = .; _bss_end = .;
_edata = .; _edata = .;
} }
_data_size = _edata - _data;
.rela.dyn : { *(.rela.dyn) } .rela.dyn : { *(.rela.dyn) }
.rela.plt : { *(.rela.plt) } .rela.plt : { *(.rela.plt) }
.rela.got : { *(.rela.got) } .rela.got : { *(.rela.got) }
.rela.data : { *(.rela.data) *(.rela.data*) } .rela.data : { *(.rela.data) *(.rela.data*) }
_data_size = . - _etext;
. = ALIGN(4096); . = ALIGN(4096);
.dynsym : { *(.dynsym) } .dynsym : { *(.dynsym) }