Skip to content

Commit ab32570

Browse files
Peter Zijlstraingomolnar
authored andcommitted
jump_label, x86: Allow short NOPs
Now that objtool is able to rewrite jump_label instructions, have the compiler emit a JMP, such that it can decide on the optimal encoding, and set jump_entry::key bit1 to indicate that objtool should rewrite the instruction to a matching NOP. For x86_64-allyesconfig this gives: jl\ NOP JMP short: 22997 124 long: 30874 90 IOW, we save (22997+124) * 3 bytes of kernel text in hotpaths. Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org> Signed-off-by: Ingo Molnar <mingo@kernel.org> Link: https://lore.kernel.org/r/20210506194158.216763632@infradead.org
1 parent e2d9494 commit ab32570

1 file changed

Lines changed: 18 additions & 0 deletions

File tree

arch/x86/include/asm/jump_label.h

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,22 @@
2020
_ASM_PTR "%c0 + %c1 - .\n\t" \
2121
".popsection \n\t"
2222

23+
#ifdef CONFIG_STACK_VALIDATION
24+
25+
static __always_inline bool arch_static_branch(struct static_key *key, bool branch)
26+
{
27+
asm_volatile_goto("1:"
28+
"jmp %l[l_yes] # objtool NOPs this \n\t"
29+
JUMP_TABLE_ENTRY
30+
: : "i" (key), "i" (2 | branch) : : l_yes);
31+
32+
return false;
33+
l_yes:
34+
return true;
35+
}
36+
37+
#else
38+
2339
static __always_inline bool arch_static_branch(struct static_key * const key, const bool branch)
2440
{
2541
asm_volatile_goto("1:"
@@ -32,6 +48,8 @@ static __always_inline bool arch_static_branch(struct static_key * const key, co
3248
return true;
3349
}
3450

51+
#endif /* STACK_VALIDATION */
52+
3553
static __always_inline bool arch_static_branch_jump(struct static_key * const key, const bool branch)
3654
{
3755
asm_volatile_goto("1:"

0 commit comments

Comments
 (0)