Re: [PATCH 1/1] arm: separate .data and .text sections of EFI binaries

2024-02-24 Thread Heinrich Schuchardt
On 2/24/24 10:16, Heinrich Schuchardt wrote: On 2/16/24 11:35, Ilias Apalodimas wrote: Hi Heinrich, [...] } +   .rela.dyn : { *(.rela.dyn) } +   .rela.plt : { *(.rela.plt) } +   .rela.got : { *(.rela.got) } +   .rela.data : { *(.rela.data) *(.rela.data*) } Why are w

Re: [PATCH 1/1] arm: separate .data and .text sections of EFI binaries

2024-02-24 Thread Heinrich Schuchardt
On 2/16/24 11:35, Ilias Apalodimas wrote: Hi Heinrich, [...] } + .rela.dyn : { *(.rela.dyn) } + .rela.plt : { *(.rela.plt) } + .rela.got : { *(.rela.got) } + .rela.data : { *(.rela.data) *(.rela.data*) } Why are we switching from Rel to Rela? This was wrong

Re: [PATCH 1/1] arm: separate .data and .text sections of EFI binaries

2024-02-16 Thread Ilias Apalodimas
Hi Heinrich, [...] > } > + .rela.dyn : { *(.rela.dyn) } > + .rela.plt : { *(.rela.plt) } > + .rela.got : { *(.rela.got) } > + .rela.data : { *(.rela.data) *(.rela.data*) } Why are we switching from Rel to Rela? > _etext = .; > _text_size = . - _te

[PATCH 1/1] arm: separate .data and .text sections of EFI binaries

2024-02-14 Thread Heinrich Schuchardt
EFI binaries should not contain sections that are both writable and executable. Separate the RX .text section from the RW .data section. Signed-off-by: Heinrich Schuchardt --- arch/arm/lib/crt0_arm_efi.S | 35 +++ arch/arm/lib/elf_arm_efi.lds | 33 +++