diff options
author | Will Deacon <[email protected]> | 2013-11-05 10:16:52 +0000 |
---|---|---|
committer | Catalin Marinas <[email protected]> | 2013-11-05 10:23:13 +0000 |
commit | 122e2fa0d310d262cb85cf0b003032e5d2bc2ae7 (patch) | |
tree | d6d40641150f867af562c1dfb84c3d17f05fdd72 | |
parent | dab7ea360967f26337be1f26f56b12771da8c731 (diff) |
arm64: module: ensure instruction is little-endian before manipulation
Relocations that require an instruction immediate to be re-encoded must
ensure that the instruction pattern is represented in a little-endian
format for the manipulation code to work correctly.
This patch converts the loaded instruction into native-endianess prior
to encoding and then converts back to little-endian byteorder before
updating memory.
Signed-off-by: Will Deacon <[email protected]>
Tested-by: Matthew Leach <[email protected]>
Signed-off-by: Catalin Marinas <[email protected]>
-rw-r--r-- | arch/arm64/kernel/module.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/arch/arm64/kernel/module.c b/arch/arm64/kernel/module.c index ca0e3d55da99..2c28a6cf93e6 100644 --- a/arch/arm64/kernel/module.c +++ b/arch/arm64/kernel/module.c @@ -111,6 +111,9 @@ static u32 encode_insn_immediate(enum aarch64_imm_type type, u32 insn, u64 imm) u32 immlo, immhi, lomask, himask, mask; int shift; + /* The instruction stream is always little endian. */ + insn = le32_to_cpu(insn); + switch (type) { case INSN_IMM_MOVNZ: /* @@ -179,7 +182,7 @@ static u32 encode_insn_immediate(enum aarch64_imm_type type, u32 insn, u64 imm) insn &= ~(mask << shift); insn |= (imm & mask) << shift; - return insn; + return cpu_to_le32(insn); } static int reloc_insn_movw(enum aarch64_reloc_op op, void *place, u64 val, |