From mboxrd@z Thu Jan 1 00:00:00 1970 From: Eduardo Habkost Subject: Re: [RFC,05/10] x86/speculation: Add basic IBRS support infrastructure Date: Mon, 29 Jan 2018 23:30:14 -0200 Message-ID: <20180130013014.GE21702@localhost.localdomain> References: <7EB9643C-D2DD-477A-90DE-05DC653D2D4B@vmware.com> <20180129224124.GU26209@tassilo.jf.intel.com> <20180130011059.GB21702@localhost.localdomain> <30A89AF5-51E6-4D7F-B669-63C2B3191302@vmware.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Jim Mattson , Andi Kleen , Arjan van de Ven , KarimAllah Ahmed , "Wilson, Matt" , "linux-kernel@vger.kernel.org" , Andrea Arcangeli , Andy Lutomirski , Ashok Raj , Asit Mallick , Borislav Petkov , Dan Williams , Dave Hansen , Greg Kroah-Hartman , "H . Peter Anvin" , Ingo Molnar , Janakarajan Natarajan , Joerg Roedel , Jun Nakajima , Laura Abbott Return-path: Received: from mx1.redhat.com ([209.132.183.28]:38652 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751562AbeA3BaS (ORCPT ); Mon, 29 Jan 2018 20:30:18 -0500 Content-Disposition: inline In-Reply-To: <30A89AF5-51E6-4D7F-B669-63C2B3191302@vmware.com> Sender: kvm-owner@vger.kernel.org List-ID: On Tue, Jan 30, 2018 at 01:20:52AM +0000, David Dunn wrote: > Eduardo, > > This is why it would be good to have a CPUID bit that says: > "apply SkyLake RSB stuffing." That's preferable to "trust FMS" > for VMware. Agreed it would be more useful than "trust FMS". However, I believe a "no need to apply Skylake RSB stuffing" bit (which I called "we promise we won't migrate to Skylake" previously) would allow guests to enable safer behavior by default under older hypervisors that don't support this bit. > > If Intel defines such a feature flag, sets it on SkyLake, and > Linux uses it... that would be very helpful for VMware. > > I won't speak for GCE and AWS. But hopefully they can indicate > whether it would help them as well. I agree that having a standard flag on the CPUID space to specify that would be very helpful. > > If Intel cannot define/implement such a flag on SkyLake, then > maybe the engineers on this email could define a flag in the > hypervisor specific CPUID space. Linux would need to query > that flag if it sees CPUID[1].ECX[31] set. That's not as nice > since it makes detection on bare metal and virtualization > platforms different, but it better than keying off FMS. Agreed. -- Eduardo