From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755638Ab1EZHkx (ORCPT ); Thu, 26 May 2011 03:40:53 -0400 Received: from zone0.gcu-squad.org ([212.85.147.21]:25833 "EHLO services.gcu-squad.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751808Ab1EZHkw (ORCPT ); Thu, 26 May 2011 03:40:52 -0400 Date: Thu, 26 May 2011 09:39:25 +0200 From: Jean Delvare To: Andi Kleen Cc: x86@kernel.org, linux-kernel@vger.kernel.org, Andi Kleen Subject: Re: [PATCH 2/3] x86, intel: Use cpu_update for Atom errata check Message-ID: <20110526093925.1b9d8bba@endymion.delvare> In-Reply-To: <1306351948-23382-2-git-send-email-andi@firstfloor.org> References: <1306351948-23382-1-git-send-email-andi@firstfloor.org> <1306351948-23382-2-git-send-email-andi@firstfloor.org> X-Mailer: Claws Mail 3.7.5 (GTK+ 2.20.1; x86_64-unknown-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, 25 May 2011 12:32:27 -0700, Andi Kleen wrote: > From: Andi Kleen > > Now that the cpu update level is available the Atom PSE errata > check can use it directly without reading the MSR again. > > Signed-off-by: Andi Kleen Acked-by: Jean Delvare > --- > arch/x86/kernel/cpu/intel.c | 15 ++++----------- > 1 files changed, 4 insertions(+), 11 deletions(-) > > diff --git a/arch/x86/kernel/cpu/intel.c b/arch/x86/kernel/cpu/intel.c > index ba5ba17..701efa4 100644 > --- a/arch/x86/kernel/cpu/intel.c > +++ b/arch/x86/kernel/cpu/intel.c > @@ -55,17 +55,10 @@ static void __cpuinit early_init_intel(struct cpuinfo_x86 *c) > * need the microcode to have already been loaded... so if it is > * not, recommend a BIOS update and disable large pages. > */ > - if (c->x86 == 6 && c->x86_model == 0x1c && c->x86_mask <= 2) { > - u32 ucode, junk; > - > - wrmsr(MSR_IA32_UCODE_REV, 0, 0); > - sync_core(); > - rdmsr(MSR_IA32_UCODE_REV, junk, ucode); > - > - if (ucode < 0x20e) { > - printk(KERN_WARNING "Atom PSE erratum detected, BIOS microcode update recommended\n"); > - clear_cpu_cap(c, X86_FEATURE_PSE); > - } > + if (c->x86 == 6 && c->x86_model == 0x1c && c->x86_mask <= 2 && > + c->cpu_update < 0x20e) { > + printk(KERN_WARNING "Atom PSE erratum detected, BIOS microcode update recommended\n"); > + clear_cpu_cap(c, X86_FEATURE_PSE); > } > > #ifdef CONFIG_X86_64 -- Jean Delvare