diff options
author | Marvin Häuser <mhaeuser@posteo.de> | 2023-04-20 15:24:15 +0000 |
---|---|---|
committer | mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> | 2023-04-20 16:20:35 +0000 |
commit | 8f4ec0cc433a33967cdbbb945acd37b6ae1d3fce (patch) | |
tree | d3dc34352f27a31bf77db94eb723c6d1d4019299 /UefiPayloadPkg | |
parent | f433fa59d22480f3ae60ea29e98a6b33227cbd7a (diff) | |
download | edk2-8f4ec0cc433a33967cdbbb945acd37b6ae1d3fce.tar.gz |
ArmPkg/ArmMmuLib: Fix ArmReplaceLiveTranslationEntry() alignment
As the ASM_FUNC() macro performs a section switch, the preceding
.balign directive applies the alignment constraint to the current
location in the previous section. As the linker may not merge the
sections in-order, ArmReplaceLiveTranslationEntry() may be left
unaligned.
Replace the explicit invocation of .balign with the ASM_FUNC_ALIGN()
macro, which guarantees the alignment constraint is applied correctly.
To make sure related issues are reliably caught in the future, align the
end of the function before checking the total occupied size. This
ensures crossing a 0x200 boundary will cause a compilation error.
Signed-off-by: Marvin Häuser <mhaeuser@posteo.de>
Reviewed-by: Leif Lindholm <quic_llindhol@quicinc.com>
Reviewed-by: Ard Biesheuvel <ardb@kernel.org>
Diffstat (limited to 'UefiPayloadPkg')
0 files changed, 0 insertions, 0 deletions