x86/ibt: Convert IBT selftest to asm
authorJosh Poimboeuf <jpoimboe@kernel.org>
Sat, 8 Apr 2023 00:16:41 +0000 (17:16 -0700)
committerPeter Zijlstra <peterz@infradead.org>
Thu, 17 Aug 2023 15:07:09 +0000 (17:07 +0200)
commitc6cfcbd8ca43766851a8c952e3b570727147020f
treef67a704f7078b00d4f6124cee2c455cbaefaf8d8
parentc6b53dcec07c842af75123d9b29684bdbd36a407
x86/ibt: Convert IBT selftest to asm

The following warning is reported when frame pointers and kernel IBT are
enabled:

  vmlinux.o: warning: objtool: ibt_selftest+0x11: sibling call from callable instruction with modified stack frame

The problem is that objtool interprets the indirect branch in
ibt_selftest() as a sibling call, and GCC inserts a (partial) frame
pointer prologue before it:

  0000 000000000003f550 <ibt_selftest>:
  0000    3f550: f3 0f 1e fa           endbr64
  0004    3f554: e8 00 00 00 00        call   3f559 <ibt_selftest+0x9> 3f555: R_X86_64_PLT32 __fentry__-0x4
  0009    3f559: 55                    push   %rbp
  000a    3f55a: 48 8d 05 02 00 00 00  lea    0x2(%rip),%rax        # 3f563 <ibt_selftest_ip>
  0011    3f561: ff e0                 jmp    *%rax

Note the inline asm is missing ASM_CALL_CONSTRAINT, so the 'push %rbp'
happens before the indirect branch and the 'mov %rsp, %rbp' happens
afterwards.

Simplify the generated code and make it easier to understand for both
tools and humans by moving the selftest to proper asm.

Signed-off-by: Josh Poimboeuf <jpoimboe@kernel.org>
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Link: https://lkml.kernel.org/r/99a7e16b97bda97bf0a04aa141d6241cd8a839a2.1680912949.git.jpoimboe@kernel.org
arch/x86/include/asm/traps.h
arch/x86/kernel/Makefile
arch/x86/kernel/cet.c
arch/x86/kernel/cpu/common.c
arch/x86/kernel/ibt_selftest.S [new file with mode: 0644]