From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-it0-f71.google.com (mail-it0-f71.google.com [209.85.214.71]) by kanga.kvack.org (Postfix) with ESMTP id DA8246B0033 for ; Mon, 27 Nov 2017 16:01:47 -0500 (EST) Received: by mail-it0-f71.google.com with SMTP id t184so12605879itf.2 for ; Mon, 27 Nov 2017 13:01:47 -0800 (PST) Received: from merlin.infradead.org (merlin.infradead.org. [2001:8b0:10b:1231::1]) by mx.google.com with ESMTPS id 199si14262194ita.140.2017.11.27.13.01.45 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Mon, 27 Nov 2017 13:01:45 -0800 (PST) Date: Mon, 27 Nov 2017 22:01:32 +0100 From: Peter Zijlstra Subject: Re: [patch 1/4] x86/paravirt: Dont patch flush_tlb_single Message-ID: <20171127210132.mkjohyfwgmxnmh4x@hirez.programming.kicks-ass.net> References: <20171127203416.236563829@linutronix.de> <20171127204257.497387357@linutronix.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20171127204257.497387357@linutronix.de> Sender: owner-linux-mm@kvack.org List-ID: To: Thomas Gleixner Cc: LKML , Dave Hansen , Andy Lutomirski , Ingo Molnar , Borislav Petkov , Brian Gerst , Denys Vlasenko , "H. Peter Anvin" , Josh Poimboeuf , Linus Torvalds , Rik van Riel , daniel.gruss@iaik.tugraz.at, hughd@google.com, keescook@google.com, linux-mm@kvack.org, michael.schwarz@iaik.tugraz.at, moritz.lipp@iaik.tugraz.at, richard.fellner@student.tugraz.at, Juergen Gross , Boris Ostrovsky On Mon, Nov 27, 2017 at 09:34:17PM +0100, Thomas Gleixner wrote: > native_flush_tlb_single() is not just INLVPG anymore. With > X86_FEATURE_INVPCID_SINGLE and KAISER enabled it flushes also the shadow > mapping. But even with KAISER disabled flushing the particular ASID is the > right thing to do. > > Remove the paravirt patching for it. > > Fixes: 1fde25dc8ef4 ("x86/mm/kaiser: Use PCID feature to make user and kernel switches faster") > Signed-off-by: Thomas Gleixner Argh!! Yes quite. ACK > --- > arch/x86/kernel/paravirt_patch_64.c | 2 -- > 1 file changed, 2 deletions(-) > > --- a/arch/x86/kernel/paravirt_patch_64.c > +++ b/arch/x86/kernel/paravirt_patch_64.c > @@ -10,7 +10,6 @@ DEF_NATIVE(pv_irq_ops, save_fl, "pushfq; > DEF_NATIVE(pv_mmu_ops, read_cr2, "movq %cr2, %rax"); > DEF_NATIVE(pv_mmu_ops, read_cr3, "movq %cr3, %rax"); > DEF_NATIVE(pv_mmu_ops, write_cr3, "movq %rdi, %cr3"); > -DEF_NATIVE(pv_mmu_ops, flush_tlb_single, "invlpg (%rdi)"); > DEF_NATIVE(pv_cpu_ops, wbinvd, "wbinvd"); > > DEF_NATIVE(pv_cpu_ops, usergs_sysret64, "swapgs; sysretq"); > @@ -60,7 +59,6 @@ unsigned native_patch(u8 type, u16 clobb > PATCH_SITE(pv_mmu_ops, read_cr2); > PATCH_SITE(pv_mmu_ops, read_cr3); > PATCH_SITE(pv_mmu_ops, write_cr3); > - PATCH_SITE(pv_mmu_ops, flush_tlb_single); > PATCH_SITE(pv_cpu_ops, wbinvd); > #if defined(CONFIG_PARAVIRT_SPINLOCKS) > case PARAVIRT_PATCH(pv_lock_ops.queued_spin_unlock): > > -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org