arm64: cpufeature: correctly handle MRS to XZR
authorMark Rutland <mark.rutland@arm.com>
Thu, 9 Feb 2017 15:19:20 +0000 (15:19 +0000)
committerWill Deacon <will.deacon@arm.com>
Wed, 15 Feb 2017 12:20:29 +0000 (12:20 +0000)
In emulate_mrs() we may erroneously write back to the user SP rather
than XZR if we trap an MRS instruction where Xt == 31.

Use the new pt_regs_write_reg() helper to handle this correctly.

Signed-off-by: Mark Rutland <mark.rutland@arm.com>
Fixes: 77c97b4ee21290f5 ("arm64: cpufeature: Expose CPUID registers by emulation")
Cc: Andre Przywara <andre.przywara@arm.com>
Cc: Catalin Marinas <catalin.marinas@arm.com>
Cc: Marc Zyngier <marc.zyngier@arm.com>
Cc: Suzuki K Poulose <suzuki.poulose@arm.com>
Cc: Will Deacon <will.deacon@arm.com>
Signed-off-by: Will Deacon <will.deacon@arm.com>
arch/arm64/kernel/cpufeature.c

index 1ee5357d0c6aee8c87a6a77455a1f47859995ab6..abda8e861865779d96087b3de57d60305d26e582 100644 (file)
@@ -1214,7 +1214,7 @@ static int emulate_mrs(struct pt_regs *regs, u32 insn)
        rc = emulate_sys_reg(sys_reg, &val);
        if (!rc) {
                dst = aarch64_insn_decode_register(AARCH64_INSN_REGTYPE_RT, insn);
-               regs->user_regs.regs[dst] = val;
+               pt_regs_write_reg(regs, dst, val);
                regs->pc += 4;
        }