Re: [PATCH] arm: Align section alignment with manual relocation offset code

[Alexander Graf]

Hi Julien,

Thanks a lot for the very detailed report!

On 29.04.19 19:35, Julien ROBIN wrote:
>
> Seems that this patch is a step forward for arm-efi (32 bits), and
> doesn't affect others EFI targets (arm64-efi, i386-efi and x86_64-efi
> are still working fine, including the aarch64-laptop). So it's
> definitely better than nothing. However, for arm-efi (32 bits), as
> opposed to the previously suggested revert, this patch fails with some
> QEMU firmwares :
>
>   * --target=arm-linux-gnueabihf  --with-platform=efi
>       o fine with u-boot
>       o fine with https://efi.akeo.ie/QEMU_EFI/
>       o *not running fine* with "AAVMF32_code.fd" available into
>         Debian "qemu-efi-arm" package (buster and sid) : continuous
>         rain of Exception messages
>       o *not running fine* with recent versions of :
>         
> http://snapshots.linaro.org/components/kernel/leg-virt-tianocore-edk2-upstream/latest/QEMU-ARM/
>         Welcome to Grub! Followed by 2 exception messages
>

This is because of some breakage with Thumb trampolines which seem to
get written into NX space now. Trying to isolate it down further with
Leif ...


The reason I didn't see the breakage was that both my local as well as
the Travis builds were using ARM mode, rather than Thumb. The reason it
works with U-Boot and the old OVMF version is that it they don't
implement NX.


Thanks a lot again,


Alex