diff options
| author | Linus Torvalds <torvalds@linux-foundation.org> | 2018-03-31 13:26:48 -0400 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2018-03-31 13:26:48 -0400 |
| commit | 93e04d4ad7c567f87388dd0338df7d8878ab65e9 (patch) | |
| tree | eefa261c2f597ed6a39a5785f76840357a071a12 | |
| parent | b5dbc28762fd3fd40ba76303be0c7f707826f982 (diff) | |
| parent | bd6271039ee6f0c9b468148fc2d73e0584af6b4f (diff) | |
Merge branch 'x86-pti-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip
Pull x86 PTI fixes from Ingo Molnar:
"Two fixes: a relatively simple objtool fix that makes Clang built
kernels work with ORC debug info, plus an alternatives macro fix"
* 'x86-pti-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip:
x86/alternatives: Fixup alternative_call_2
objtool: Add Clang support
| -rw-r--r-- | arch/x86/include/asm/alternative.h | 4 | ||||
| -rw-r--r-- | tools/objtool/check.c | 11 |
2 files changed, 12 insertions, 3 deletions
diff --git a/arch/x86/include/asm/alternative.h b/arch/x86/include/asm/alternative.h index cf5961ca8677..4cd6a3b71824 100644 --- a/arch/x86/include/asm/alternative.h +++ b/arch/x86/include/asm/alternative.h | |||
| @@ -218,13 +218,11 @@ static inline int alternatives_text_reserved(void *start, void *end) | |||
| 218 | */ | 218 | */ |
| 219 | #define alternative_call_2(oldfunc, newfunc1, feature1, newfunc2, feature2, \ | 219 | #define alternative_call_2(oldfunc, newfunc1, feature1, newfunc2, feature2, \ |
| 220 | output, input...) \ | 220 | output, input...) \ |
| 221 | { \ | ||
| 222 | asm volatile (ALTERNATIVE_2("call %P[old]", "call %P[new1]", feature1,\ | 221 | asm volatile (ALTERNATIVE_2("call %P[old]", "call %P[new1]", feature1,\ |
| 223 | "call %P[new2]", feature2) \ | 222 | "call %P[new2]", feature2) \ |
| 224 | : output, ASM_CALL_CONSTRAINT \ | 223 | : output, ASM_CALL_CONSTRAINT \ |
| 225 | : [old] "i" (oldfunc), [new1] "i" (newfunc1), \ | 224 | : [old] "i" (oldfunc), [new1] "i" (newfunc1), \ |
| 226 | [new2] "i" (newfunc2), ## input); \ | 225 | [new2] "i" (newfunc2), ## input) |
| 227 | } | ||
| 228 | 226 | ||
| 229 | /* | 227 | /* |
| 230 | * use this macro(s) if you need more than one output parameter | 228 | * use this macro(s) if you need more than one output parameter |
diff --git a/tools/objtool/check.c b/tools/objtool/check.c index 92b6a2c21631..5409f6f6c48d 100644 --- a/tools/objtool/check.c +++ b/tools/objtool/check.c | |||
| @@ -1386,6 +1386,17 @@ static int update_insn_state(struct instruction *insn, struct insn_state *state) | |||
| 1386 | state->vals[op->dest.reg].offset = -state->stack_size; | 1386 | state->vals[op->dest.reg].offset = -state->stack_size; |
| 1387 | } | 1387 | } |
| 1388 | 1388 | ||
| 1389 | else if (op->src.reg == CFI_BP && op->dest.reg == CFI_SP && | ||
| 1390 | cfa->base == CFI_BP) { | ||
| 1391 | |||
| 1392 | /* | ||
| 1393 | * mov %rbp, %rsp | ||
| 1394 | * | ||
| 1395 | * Restore the original stack pointer (Clang). | ||
| 1396 | */ | ||
| 1397 | state->stack_size = -state->regs[CFI_BP].offset; | ||
| 1398 | } | ||
| 1399 | |||
| 1389 | else if (op->dest.reg == cfa->base) { | 1400 | else if (op->dest.reg == cfa->base) { |
| 1390 | 1401 | ||
| 1391 | /* mov %reg, %rsp */ | 1402 | /* mov %reg, %rsp */ |
