diff options
| author | Ard Biesheuvel <ard.biesheuvel@linaro.org> | 2019-04-07 21:06:16 +0200 |
|---|---|---|
| committer | Will Deacon <will.deacon@arm.com> | 2019-04-08 17:58:13 +0200 |
| commit | 5a3ae7b314a2259b1188b22b392f5eba01e443ee (patch) | |
| tree | 43d55b1ab816545f622a3c6330fad1898b78426c /arch/arm64/include/asm/module.h | |
| parent | Linux 5.1-rc4 (diff) | |
| download | linux-5a3ae7b314a2259b1188b22b392f5eba01e443ee.tar.xz linux-5a3ae7b314a2259b1188b22b392f5eba01e443ee.zip | |
arm64/ftrace: fix inadvertent BUG() in trampoline check
The ftrace trampoline code (which deals with modules loaded out of
BL range of the core kernel) uses plt_entries_equal() to check whether
the per-module trampoline equals a zero buffer, to decide whether the
trampoline has already been initialized.
This triggers a BUG() in the opcode manipulation code, since we end
up checking the ADRP offset of a 0x0 opcode, which is not an ADRP
instruction.
So instead, add a helper to check whether a PLT is initialized, and
call that from the frace code.
Cc: <stable@vger.kernel.org> # v5.0
Fixes: bdb85cd1d206 ("arm64/module: switch to ADRP/ADD sequences for PLT entries")
Acked-by: Mark Rutland <mark.rutland@arm.com>
Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Signed-off-by: Will Deacon <will.deacon@arm.com>
Diffstat (limited to 'arch/arm64/include/asm/module.h')
| -rw-r--r-- | arch/arm64/include/asm/module.h | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/arch/arm64/include/asm/module.h b/arch/arm64/include/asm/module.h index 905e1bb0e7bd..cd9f4e9d04d3 100644 --- a/arch/arm64/include/asm/module.h +++ b/arch/arm64/include/asm/module.h @@ -73,4 +73,9 @@ static inline bool is_forbidden_offset_for_adrp(void *place) struct plt_entry get_plt_entry(u64 dst, void *pc); bool plt_entries_equal(const struct plt_entry *a, const struct plt_entry *b); +static inline bool plt_entry_is_initialized(const struct plt_entry *e) +{ + return e->adrp || e->add || e->br; +} + #endif /* __ASM_MODULE_H */ |