@@ -85,7 +85,7 @@ bs_die:
# Offset to the PE header.
#
.long LINUX_PE_MAGIC
- .long pe_header
+ .long pe_header - bootsect_start
#endif /* CONFIG_EFI_STUB */
.section ".bsdata", "a"
@@ -96,6 +96,8 @@ bugger_off_msg:
.byte 0
#ifdef CONFIG_EFI_STUB
+ .section ".peheader", "a"
+ .align 8
pe_header:
.long PE_MAGIC
@@ -161,7 +163,7 @@ extra_header_fields:
#
.long 0 # SizeOfImage
- .long 0x200 # SizeOfHeaders
+ .long 0x800 # SizeOfHeaders
.long 0 # CheckSum
.word IMAGE_SUBSYSTEM_EFI_APPLICATION # Subsystem (EFI application)
#ifdef CONFIG_EFI_DXE_MEM_ATTRIBUTES
@@ -192,26 +194,6 @@ extra_header_fields:
# Section table
section_table:
- #
- # The offset & size fields are filled in by build.c.
- #
- .ascii ".setup"
- .byte 0
- .byte 0
- .long 0
- .long 0x0 # startup_{32,64}
- .long 0 # Size of initialized data
- # on disk
- .long 0x0 # startup_{32,64}
- .long 0 # PointerToRelocations
- .long 0 # PointerToLineNumbers
- .word 0 # NumberOfRelocations
- .word 0 # NumberOfLineNumbers
- .long IMAGE_SCN_CNT_CODE | \
- IMAGE_SCN_MEM_READ | \
- IMAGE_SCN_MEM_EXECUTE | \
- IMAGE_SCN_ALIGN_16BYTES # Characteristics
-
#
# The EFI application loader requires a relocation section
# because EFI applications must be relocatable. The .reloc
@@ -16,6 +16,7 @@ SECTIONS
. = 495;
.header : { *(.header) }
.entrytext : { *(.entrytext) }
+ .peheader : { *(.peheader) }
.inittext : { *(.inittext) }
.initdata : { *(.initdata) }
__end_init = .;
@@ -296,16 +296,13 @@ static void update_pecoff_section_header(char *section_name, uint32_t offset, ui
update_pecoff_section_header_fields(section_name, offset, size, size, offset);
}
-static void update_pecoff_setup_and_reloc(unsigned int size)
+static void update_pecoff_reloc(unsigned int size)
{
- uint32_t setup_offset = SECTOR_SIZE;
uint32_t reloc_offset = size - PECOFF_RELOC_RESERVE - PECOFF_COMPAT_RESERVE;
#ifdef CONFIG_EFI_MIXED
uint32_t compat_offset = reloc_offset + PECOFF_RELOC_RESERVE;
#endif
- uint32_t setup_size = reloc_offset - setup_offset;
- update_pecoff_section_header(".setup", setup_offset, setup_size);
update_pecoff_section_header(".reloc", reloc_offset, PECOFF_RELOC_RESERVE);
/*
@@ -353,7 +350,7 @@ static unsigned int update_pecoff_sections(unsigned int text_start, unsigned int
* Size of code: Subtract the size of the first sector (512 bytes)
* which includes the header.
*/
- put_unaligned_le32(file_sz - SECTOR_SIZE + bss_sz, &hdr->text_size);
+ put_unaligned_le32(text_sz + bss_sz, &hdr->text_size);
/* Size of image */
put_unaligned_le32(init_sz, &hdr->image_size);
@@ -407,7 +404,7 @@ static void efi_stub_entry_update(void)
#else
-static inline void update_pecoff_setup_and_reloc(unsigned int size) {}
+static inline void update_pecoff_reloc(unsigned int size) {}
static inline void update_pecoff_text(unsigned int text_start,
unsigned int file_sz,
unsigned int init_sz) {}
@@ -542,7 +539,7 @@ int main(int argc, char **argv)
#ifdef CONFIG_EFI_STUB
/* PE specification require 512-byte minimum section file alignment */
kern_size = round_up(kern_file_size + 4, SECTOR_SIZE);
- update_pecoff_setup_and_reloc(setup_size);
+ update_pecoff_reloc(setup_size);
#else
/* Number of 16-byte paragraphs, including space for a 4-byte CRC */
kern_size = round_up(kern_file_size + 4, PARAGRAPH_SIZE);
We are currently limited in the number of PE/COFF sections we can describe in the PE header, due to lack of space. This is caused by the presence of the setup header at offset 0x1f1, leaving only the space before it for PE metadata. However, now that we no longer copy the setup_header from this part of the image for use by the EFI stub, we no longer have to describe it as part of the loadable image. This means we can put the PE header *after* the setup header, and use as much space as we like. It also means we don't have to describe this part of the image in PE/COFF, and simply treat it as part of the header. This means we can drop the ".setup" section as well. Signed-off-by: Ard Biesheuvel <ardb@kernel.org> --- arch/x86/boot/header.S | 26 +++----------------- arch/x86/boot/setup.ld | 1 + arch/x86/boot/tools/build.c | 11 +++------ 3 files changed, 9 insertions(+), 29 deletions(-)