From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756101AbdECULo (ORCPT ); Wed, 3 May 2017 16:11:44 -0400 Received: from mail.skyhub.de ([5.9.137.197]:55752 "EHLO mail.skyhub.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753414AbdECULi (ORCPT ); Wed, 3 May 2017 16:11:38 -0400 Date: Wed, 3 May 2017 22:11:34 +0200 From: Borislav Petkov To: Radim =?utf-8?B?S3LEjW3DocWZ?= Cc: linux-kernel@vger.kernel.org, kvm@vger.kernel.org, Paolo Bonzini , Alexander Graf , "Michael S. Tsirkin" , "Gabriel L. Somlo" Subject: Re: [PATCH 1/4] KVM: svm: prevent MWAIT in guest with erratum 400 Message-ID: <20170503201134.fabwojfrqkhime7c@pd.tnic> References: <20170503193733.13409-1-rkrcmar@redhat.com> <20170503193733.13409-2-rkrcmar@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20170503193733.13409-2-rkrcmar@redhat.com> User-Agent: NeoMutt/20170113 (1.7.2) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, May 03, 2017 at 09:37:30PM +0200, Radim Krčmář wrote: > The host might miss APIC timer interrupts if the guest enters a specific > C-state. Quoting the erratum: > > 400 APIC Timer Interrupt Does Not Occur in Processor C-States > > Description > > An APIC timer interrupt that becomes pending in low-power states C1E > or C3 will not cause the processor to enter the C0 state even if the > interrupt is enabled by Timer Local Vector Table Entry[Mask], > APIC320[16]). APIC timer functionality is otherwise unaffected. > > Potential Effect on System > > System hang may occur provided that the operating system has not > configured another interrupt source. APIC timer interrupts may be > delayed or, when the APIC timer is configured in rollover mode > (APIC320[17]), the APIC timer may roll over multiple times in the > low-power state with only one interrupt presented after the processor > resumes. The standard use of the APIC timer does not make this effect > significant. > > Signed-off-by: Radim Krčmář > --- > arch/x86/kvm/x86.h | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/arch/x86/kvm/x86.h b/arch/x86/kvm/x86.h > index 612067074905..3ed7dd8737ab 100644 > --- a/arch/x86/kvm/x86.h > +++ b/arch/x86/kvm/x86.h > @@ -223,8 +223,7 @@ static inline bool kvm_mwait_in_guest(void) > > switch (boot_cpu_data.x86_vendor) { > case X86_VENDOR_AMD: > - /* All AMD CPUs have a working MWAIT implementation */ > - return true; > + return !boot_cpu_has_bug(X86_BUG_AMD_E400); Well, this looks wrong: it is X86_BUG_AMD_APIC_C1E, which actually denotes that we must enable the E400 workaround because the platform actually goes into C1E. X86_BUG_AMD_E400 gets set only on the affected f/m/s range but if the BIOS doesn't put the CPU in C1E, we don't hit the erratum and all is peachy. Also, what do APIC timer interrupts even have to do with MWAIT-ing in the guest, especially if we enable the workaround and switch to HPET on the host? Maybe I'm missing something here... -- Regards/Gruss, Boris. Good mailing practices for 400: avoid top-posting and trim the reply.