diff options
author | Borislav Petkov <bp@suse.de> | 2016-02-10 15:51:16 +0100 |
---|---|---|
committer | Ingo Molnar <mingo@kernel.org> | 2016-02-14 11:19:06 +0100 |
commit | e2c7698cd61f11d4077fdb28148b2d31b82ac848 (patch) | |
tree | f6a4a3c3b8f7151930b5778f24cd65a4dbee9ec4 /mm/memory.c | |
parent | x86/dmi: Switch dmi_remap() from ioremap() [uncached] to ioremap_cache() (diff) | |
download | linux-e2c7698cd61f11d4077fdb28148b2d31b82ac848.tar.xz linux-e2c7698cd61f11d4077fdb28148b2d31b82ac848.zip |
x86/mm: Fix INVPCID asm constraint
So we want to specify the dependency on both @pcid and @addr so that the
compiler doesn't reorder accesses to them *before* the TLB flush. But
for that to work, we need to express this properly in the inline asm and
deref the whole desc array, not the pointer to it. See clwb() for an
example.
This fixes the build error on 32-bit:
arch/x86/include/asm/tlbflush.h: In function ‘__invpcid’:
arch/x86/include/asm/tlbflush.h:26:18: error: memory input 0 is not directly addressable
which gcc4.7 caught but 5.x didn't. Which is strange. :-\
Signed-off-by: Borislav Petkov <bp@suse.de>
Cc: Andrew Morton <akpm@linux-foundation.org>
Cc: Andrey Ryabinin <aryabinin@virtuozzo.com>
Cc: Andy Lutomirski <luto@amacapital.net>
Cc: Borislav Petkov <bp@alien8.de>
Cc: Brian Gerst <brgerst@gmail.com>
Cc: Dave Hansen <dave.hansen@linux.intel.com>
Cc: Denys Vlasenko <dvlasenk@redhat.com>
Cc: H. Peter Anvin <hpa@zytor.com>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Luis R. Rodriguez <mcgrof@suse.com>
Cc: Michael Matz <matz@suse.de>
Cc: Oleg Nesterov <oleg@redhat.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: Toshi Kani <toshi.kani@hp.com>
Cc: linux-mm@kvack.org
Signed-off-by: Ingo Molnar <mingo@kernel.org>
Diffstat (limited to 'mm/memory.c')
0 files changed, 0 insertions, 0 deletions