From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1030840AbbJ3K1B (ORCPT ); Fri, 30 Oct 2015 06:27:01 -0400 Received: from mail.skyhub.de ([78.46.96.112]:48454 "EHLO mail.skyhub.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1030730AbbJ3K07 (ORCPT ); Fri, 30 Oct 2015 06:26:59 -0400 Date: Fri, 30 Oct 2015 11:26:47 +0100 From: Borislav Petkov To: Aravind Gopalakrishnan Cc: tglx@linutronix.de, mingo@redhat.com, hpa@zytor.com, tony.luck@intel.com, x86@kernel.org, ashok.raj@intel.com, linux-kernel@vger.kernel.org, linux-edac@vger.kernel.org, peterz@infradead.org, luto@kernel.org, dvlasenk@redhat.com, ross.zwisler@linux.intel.com, dirk.j.brandewie@intel.com Subject: Re: [PATCH V2 1/2] x86/mcheck: Add Scalable MCA cpuid bit Message-ID: <20151030102646.GF20952@pd.tnic> References: <1446059010-28010-1-git-send-email-Aravind.Gopalakrishnan@amd.com> <1446059010-28010-2-git-send-email-Aravind.Gopalakrishnan@amd.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <1446059010-28010-2-git-send-email-Aravind.Gopalakrishnan@amd.com> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Oct 28, 2015 at 02:03:29PM -0500, Aravind Gopalakrishnan wrote: > Scalable MCA (SMCA) is a new feature in AMD Fam17h > processors which indicates presence of MCA extensions. > > MCA extensions expands existing register space for the > MCE banks and also introduces a new MSR range to > accommodate new banks. Future additions to AMD MCE code > will first need to detect if SMCA is enabled before > enabling the new features. > > Adding code to detect if it SMCA is enabled in this patch > and store that info in mce_vendor_flags structure. > > Signed-off-by: Aravind Gopalakrishnan > --- > arch/x86/include/asm/mce.h | 13 ++++++++++++- > arch/x86/kernel/cpu/mcheck/mce.c | 2 ++ > 2 files changed, 14 insertions(+), 1 deletion(-) > > diff --git a/arch/x86/include/asm/mce.h b/arch/x86/include/asm/mce.h > index 2dbc0bf..63307b5 100644 > --- a/arch/x86/include/asm/mce.h > +++ b/arch/x86/include/asm/mce.h > @@ -135,7 +135,18 @@ struct mce_vendor_flags { > * in HW and deferred error interrupts. > */ > succor : 1, > - __reserved_0 : 62; > + > + /* > + * Scalable MCA: This bit indicates support for MCAX > + * (MCA EXtensions) which expands the register space I changed that to SMCA. Let's refer to this feature with only one abbreviation please. It seems hw vendors won't settle on names easily so we should avoid that confusion in the kernel, at least. Anyway, applied, thanks. -- Regards/Gruss, Boris. ECO tip #101: Trim your mails when you reply.