From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753308AbdJLILb (ORCPT ); Thu, 12 Oct 2017 04:11:31 -0400 Received: from mail-wm0-f66.google.com ([74.125.82.66]:38064 "EHLO mail-wm0-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751479AbdJLILa (ORCPT ); Thu, 12 Oct 2017 04:11:30 -0400 X-Google-Smtp-Source: AOwi7QCHZzbTfmeQDW/AbGc+r2GpwwmzCB6e4SAS0X0oSPYU1j5RHajgoKKE0e98420JsRlOBql2yw== Date: Thu, 12 Oct 2017 10:11:26 +0200 From: Ingo Molnar To: Andi Kleen Cc: x86@kernel.org, linux-kernel@vger.kernel.org, Andi Kleen , Thomas Gleixner Subject: Re: [PATCH v9 4/5] x86/xsave: Make XSAVE check the base CPUID features before enabling Message-ID: <20171012081126.g4shmny3qumwzf4m@gmail.com> References: <20171007000328.12843-1-andi@firstfloor.org> <20171007000328.12843-5-andi@firstfloor.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20171007000328.12843-5-andi@firstfloor.org> User-Agent: NeoMutt/20170113 (1.7.2) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org * Andi Kleen wrote: > From: Andi Kleen > > Before enabling XSAVE, not only check the XSAVE specific CPUID bits, > but also the base CPUID features of the respective XSAVE feature. > This allows to disable individual XSAVE states using the existing > clearcpuid= option, which can be useful for performance testing > and debugging, and also in general avoids inconsistencies. > > v2: > Add curly brackets (Thomas Gleixner) > Signed-off-by: Andi Kleen > --- > arch/x86/kernel/fpu/xstate.c | 23 +++++++++++++++++++++++ > 1 file changed, 23 insertions(+) > > diff --git a/arch/x86/kernel/fpu/xstate.c b/arch/x86/kernel/fpu/xstate.c > index f1d5476c9022..924bd895b5ee 100644 > --- a/arch/x86/kernel/fpu/xstate.c > +++ b/arch/x86/kernel/fpu/xstate.c > @@ -15,6 +15,7 @@ > #include > > #include > +#include > > /* > * Although we spell it out in here, the Processor Trace > @@ -36,6 +37,19 @@ static const char *xfeature_names[] = > "unknown xstate feature" , > }; > > +static short xsave_cpuid_features[] = { > + X86_FEATURE_FPU, > + X86_FEATURE_XMM, > + X86_FEATURE_AVX, > + X86_FEATURE_MPX, > + X86_FEATURE_MPX, > + X86_FEATURE_AVX512F, > + X86_FEATURE_AVX512F, > + X86_FEATURE_AVX512F, > + X86_FEATURE_INTEL_PT, > + X86_FEATURE_PKU, > +}; > + > /* > * Mask of xstate features supported by the CPU and the kernel: > */ > @@ -726,6 +740,7 @@ void __init fpu__init_system_xstate(void) > unsigned int eax, ebx, ecx, edx; > static int on_boot_cpu __initdata = 1; > int err; > + int i; > > WARN_ON_FPU(!on_boot_cpu); > on_boot_cpu = 0; > @@ -759,6 +774,14 @@ void __init fpu__init_system_xstate(void) > goto out_disable; > } > > + /* > + * Clear XSAVE features that are disabled in the normal CPUID. > + */ > + for (i = 0; i < ARRAY_SIZE(xsave_cpuid_features); i++) { > + if (!boot_cpu_has(xsave_cpuid_features[i])) > + xfeatures_mask &= ~BIT(i); > + } > + > xfeatures_mask &= fpu__get_supported_xfeatures_mask(); > > /* Enable xstate instructions to be able to continue with initialization: */ This patch has similar problems to the ones I reported against 2/5. Thanks, Ingo