linux-kernel - Re: [PATCH] x86: svm: make wbinvd faster

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20150310223713.GA17773@amt.cnet>
Date:	Tue, 10 Mar 2015 19:37:13 -0300
From:	Marcelo Tosatti <mtosatti@...hat.com>
To:	Radim Krčmář <rkrcmar@...hat.com>
Cc:	Joel Schopp <joel.schopp@....com>, Gleb Natapov <gleb@...nel.org>,
	Paolo Bonzini <pbonzini@...hat.com>, kvm@...r.kernel.org,
	David Kaplan <David.Kaplan@....com>,
	Joerg Roedel <joro@...tes.org>, linux-kernel@...r.kernel.org,
	Borislav Petkov <bp@...en8.de>
Subject: Re: [PATCH] x86: svm: make wbinvd faster

On Tue, Mar 10, 2015 at 12:01:31PM +0100, Radim Krčmář wrote:
> 2015-03-09 20:28-0300, Marcelo Tosatti:
> > On Fri, Feb 27, 2015 at 06:19:18PM -0600, Joel Schopp wrote:
> > > From: David Kaplan <David.Kaplan@....com>
> > > No need to re-decode WBINVD since we know what it is from the intercept.
> > > 
> > > Signed-off-by: David Kaplan <David.Kaplan@....com>
> > > [extracted from larger unlrelated patch, forward ported, tested]
> > > Signed-off-by: Joel Schopp <joel.schopp@....com>
> > 
> > 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.

Yes, right.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/