xen/arm64: Place a speculation barrier following an ret instruction
authorJulien Grall <jgrall@amazon.com>
Tue, 16 Jun 2020 15:33:12 +0000 (16:33 +0100)
committerJulien Grall <jgrall@amazon.com>
Tue, 20 Apr 2021 12:17:34 +0000 (13:17 +0100)
commitbcab2ac84931ab58e2f2cd1242ed551d8aaff021
treeb8c5c56d0e0d1bc453b49d96a6b8e735ce01b1e6
parent730d0f6082e66eefae64f35bc62e51fc54d02d55
xen/arm64: Place a speculation barrier following an ret instruction

Some CPUs can speculate past a RET instruction and potentially perform
speculative accesses to memory before processing the return.

There is no known gadget available after the RET instruction today.
However some of the registers (such as in check_pending_guest_serror())
may contain a value provided by the guest.

In order to harden the code, it would be better to add a speculation
barrier after each RET instruction. The performance impact is meant to
be negligeable as the speculation barrier is not meant to be
architecturally executed.

Rather than manually inserting a speculation barrier, use a macro
which overrides the mnemonic RET and replace with RET + SB. We need to
use the opcode for RET to prevent any macro recursion.

This patch is only covering the assembly code. C code would need to be
covered separately using the compiler support.

Note that the definition of the macros sb needs to be moved earlier in
asm-arm/macros.h so it can be used by the new macro.

This is part of the work to mitigate straight-line speculation.

Signed-off-by: Julien Grall <jgrall@amazon.com>
Reviewed-by: Bertrand Marquis <bertrand.marquis@arm.com>
Acked-by: Stefano Stabellini <sstabellini@kernel.org>
xen/include/asm-arm/arm64/macros.h
xen/include/asm-arm/macros.h