x86/efi: Drop alignment flags from PE section headers
authorArd Biesheuvel <ardb@kernel.org>
Tue, 12 Sep 2023 09:00:54 +0000 (09:00 +0000)
committerIngo Molnar <mingo@kernel.org>
Fri, 15 Sep 2023 09:18:41 +0000 (11:18 +0200)
The section header flags for alignment are documented in the PE/COFF
spec as being applicable to PE object files only, not to PE executables
such as the Linux bzImage, so let's drop them from the PE header.

Signed-off-by: Ard Biesheuvel <ardb@kernel.org>
Signed-off-by: Ingo Molnar <mingo@kernel.org>
Link: https://lore.kernel.org/r/20230912090051.4014114-20-ardb@google.com
arch/x86/boot/header.S

index b04ca8e2b213c6e647ad855140682545442321c7..8c8148d751c6d22bb20fe51b1339391463afb2c6 100644 (file)
@@ -209,8 +209,7 @@ section_table:
        .word   0                               # NumberOfLineNumbers
        .long   IMAGE_SCN_CNT_CODE              | \
                IMAGE_SCN_MEM_READ              | \
-               IMAGE_SCN_MEM_EXECUTE           | \
-               IMAGE_SCN_ALIGN_16BYTES         # Characteristics
+               IMAGE_SCN_MEM_EXECUTE           # Characteristics
 
        #
        # The EFI application loader requires a relocation section
@@ -230,8 +229,7 @@ section_table:
        .word   0                               # NumberOfLineNumbers
        .long   IMAGE_SCN_CNT_INITIALIZED_DATA  | \
                IMAGE_SCN_MEM_READ              | \
-               IMAGE_SCN_MEM_DISCARDABLE       | \
-               IMAGE_SCN_ALIGN_1BYTES          # Characteristics
+               IMAGE_SCN_MEM_DISCARDABLE       # Characteristics
 
 #ifdef CONFIG_EFI_MIXED
        #
@@ -249,8 +247,7 @@ section_table:
        .word   0                               # NumberOfLineNumbers
        .long   IMAGE_SCN_CNT_INITIALIZED_DATA  | \
                IMAGE_SCN_MEM_READ              | \
-               IMAGE_SCN_MEM_DISCARDABLE       | \
-               IMAGE_SCN_ALIGN_1BYTES          # Characteristics
+               IMAGE_SCN_MEM_DISCARDABLE       # Characteristics
 #endif
 
        #
@@ -271,8 +268,7 @@ section_table:
        .word   0                               # NumberOfLineNumbers
        .long   IMAGE_SCN_CNT_CODE              | \
                IMAGE_SCN_MEM_READ              | \
-               IMAGE_SCN_MEM_EXECUTE           | \
-               IMAGE_SCN_ALIGN_16BYTES         # Characteristics
+               IMAGE_SCN_MEM_EXECUTE           # Characteristics
 
        .set    section_count, (. - section_table) / 40
 #endif /* CONFIG_EFI_STUB */