aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLinus Torvalds <torvalds@linux-foundation.org>2018-08-13 20:01:03 -0400
committerLinus Torvalds <torvalds@linux-foundation.org>2018-08-13 20:01:03 -0400
commit4d5ac4b8ca1ecbdb600097a26071abc5b47500bf (patch)
tree4ccef33588c8a944140b291646fd741ef493cb62
parent203b4fc903b644223a27ad3f25f3a0f3a3911d1d (diff)
parent5800dc5c19f34e6e03b5adab1282535cb102fafd (diff)
Merge branch 'x86-urgent-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip
Pull misc x86 fixes from Thomas Gleixner: "Two fixes for x86: - Provide a declaration for native_save_fl() which unbreaks the wreckage caused by making it 'extern inline'. - Fix the failing paravirt patching which is supposed to replace indirect with direct calls. The wreckage is caused by an incorrect clobber test" * 'x86-urgent-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip: x86/paravirt: Fix spectre-v2 mitigations for paravirt guests x86/irqflags: Provide a declaration for native_save_fl
-rw-r--r--arch/x86/include/asm/irqflags.h2
-rw-r--r--arch/x86/kernel/paravirt.c14
2 files changed, 12 insertions, 4 deletions
diff --git a/arch/x86/include/asm/irqflags.h b/arch/x86/include/asm/irqflags.h
index c4fc17220df9..c14f2a74b2be 100644
--- a/arch/x86/include/asm/irqflags.h
+++ b/arch/x86/include/asm/irqflags.h
@@ -13,6 +13,8 @@
13 * Interrupt control: 13 * Interrupt control:
14 */ 14 */
15 15
16/* Declaration required for gcc < 4.9 to prevent -Werror=missing-prototypes */
17extern inline unsigned long native_save_fl(void);
16extern inline unsigned long native_save_fl(void) 18extern inline unsigned long native_save_fl(void)
17{ 19{
18 unsigned long flags; 20 unsigned long flags;
diff --git a/arch/x86/kernel/paravirt.c b/arch/x86/kernel/paravirt.c
index 99dc79e76bdc..930c88341e4e 100644
--- a/arch/x86/kernel/paravirt.c
+++ b/arch/x86/kernel/paravirt.c
@@ -88,10 +88,12 @@ unsigned paravirt_patch_call(void *insnbuf,
88 struct branch *b = insnbuf; 88 struct branch *b = insnbuf;
89 unsigned long delta = (unsigned long)target - (addr+5); 89 unsigned long delta = (unsigned long)target - (addr+5);
90 90
91 if (tgt_clobbers & ~site_clobbers) 91 if (len < 5) {
92 return len; /* target would clobber too much for this site */ 92#ifdef CONFIG_RETPOLINE
93 if (len < 5) 93 WARN_ONCE("Failing to patch indirect CALL in %ps\n", (void *)addr);
94#endif
94 return len; /* call too long for patch site */ 95 return len; /* call too long for patch site */
96 }
95 97
96 b->opcode = 0xe8; /* call */ 98 b->opcode = 0xe8; /* call */
97 b->delta = delta; 99 b->delta = delta;
@@ -106,8 +108,12 @@ unsigned paravirt_patch_jmp(void *insnbuf, const void *target,
106 struct branch *b = insnbuf; 108 struct branch *b = insnbuf;
107 unsigned long delta = (unsigned long)target - (addr+5); 109 unsigned long delta = (unsigned long)target - (addr+5);
108 110
109 if (len < 5) 111 if (len < 5) {
112#ifdef CONFIG_RETPOLINE
113 WARN_ONCE("Failing to patch indirect JMP in %ps\n", (void *)addr);
114#endif
110 return len; /* call too long for patch site */ 115 return len; /* call too long for patch site */
116 }
111 117
112 b->opcode = 0xe9; /* jmp */ 118 b->opcode = 0xe9; /* jmp */
113 b->delta = delta; 119 b->delta = delta;