diff options
| author | Sam Edwards <cfsworks@gmail.com> | 2025-03-15 15:18:04 -0700 |
|---|---|---|
| committer | Tom Rini <trini@konsulko.com> | 2025-04-02 14:33:50 -0600 |
| commit | d5734b183c3d578fff1c1e81e46a1d04342edffe (patch) | |
| tree | f8d6919a55da15d0f362dc6fa3713a5420d5981a | |
| parent | 16448c443c8cacba7dacb3e919c0b414f70b8a7c (diff) | |
arm: Replace 'adrl' in EFI crt0
LLVM's IAS does not (and cannot easily) support the 'adrl'
pseudoinstruction, and ARM developers generally do not consider it
portable across assembler implementations either.
Instead, expand it into the two subtract instructions it would emit
anyway. An explanation of the math follows:
The .+8 and .+4 refer to the same memory location; this is because the
.+4 expression occurs in a subsequent instruction, 4 bytes after the
first. This memory location is the value of the PC register when it is
read by the first sub instruction. Thus, both inner parenthesized
expressions evaluate to the same result: PC's offset relative to
image_base. The subtract instructions then remove one byte each
(low, then high) of the total offset, thereby getting the absolute
address of image_base loaded in r0.
Signed-off-by: Sam Edwards <CFSworks@gmail.com>
Reviewed-by: Ilias Apalodimas <ilias.apalodimas@linaro.org>
| -rw-r--r-- | arch/arm/lib/crt0_arm_efi.S | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/arch/arm/lib/crt0_arm_efi.S b/arch/arm/lib/crt0_arm_efi.S index 91b0fe12c51..235b3a0c48f 100644 --- a/arch/arm/lib/crt0_arm_efi.S +++ b/arch/arm/lib/crt0_arm_efi.S @@ -149,7 +149,8 @@ _start: adr r1, .L_DYNAMIC ldr r0, [r1] add r1, r0, r1 - adrl r0, image_base + sub r0, pc, #((.+8-image_base) & 0xff) + sub r0, r0, #((.+4-image_base) & 0xff00) bl _relocate teq r0, #0 bne 0f |