From mboxrd@z Thu Jan 1 00:00:00 1970 From: Radim =?utf-8?B?S3LEjW3DocWZ?= Subject: Re: [PATCH] x86: svm: make wbinvd faster Date: Tue, 10 Mar 2015 12:01:31 +0100 Message-ID: <20150310110131.GA12854@potion.brq.redhat.com> References: <20150228001917.15247.41063.stgit@joelvmguard2.amd.com> <20150309232846.GB12125@amt.cnet> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Joel Schopp , Gleb Natapov , Paolo Bonzini , kvm@vger.kernel.org, David Kaplan , Joerg Roedel , linux-kernel@vger.kernel.org, Borislav Petkov To: Marcelo Tosatti Return-path: Content-Disposition: inline In-Reply-To: <20150309232846.GB12125@amt.cnet> Sender: linux-kernel-owner@vger.kernel.org List-Id: kvm.vger.kernel.org 2015-03-09 20:28-0300, Marcelo Tosatti: > On Fri, Feb 27, 2015 at 06:19:18PM -0600, Joel Schopp wrote: > > From: David Kaplan > > No need to re-decode WBINVD since we know what it is from the intercept. > > > > Signed-off-by: David Kaplan > > [extracted from larger unlrelated patch, forward ported, tested] > > Signed-off-by: Joel Schopp > > Can't you disable the intercept if need_emulate_wbinvd(vcpu) == false? I don't think we want to: it should be faster to intercept and ignore than to invalidate all caches. The exit doesn't affect other physical cores and costs just about 10(?) L3 cache misses.