From patchwork Thu Nov 10 00:07:02 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Kyle Moffett X-Patchwork-Id: 124748 Return-Path: X-Original-To: patchwork-incoming@ozlabs.org Delivered-To: patchwork-incoming@ozlabs.org Received: from ozlabs.org (localhost [IPv6:::1]) by ozlabs.org (Postfix) with ESMTP id CEA96101095 for ; Thu, 10 Nov 2011 11:09:17 +1100 (EST) Received: from border.exmeritus.com (border.exmeritus.com [IPv6:2002:46a7:f11a:ffff::]) by ozlabs.org (Postfix) with ESMTP id C33F5100FF5 for ; Thu, 10 Nov 2011 11:08:29 +1100 (EST) Received: from ysera.exmeritus.com (firewall2.exmeritus.com [10.13.38.2]) by border.exmeritus.com (Postfix) with ESMTP id 029CBAC081; Wed, 9 Nov 2011 19:08:27 -0500 (EST) From: Kyle Moffett To: linuxppc-dev@lists.ozlabs.org Subject: [RFC PATCH 04/17] powerpc: Allow multiple machine-check handlers Date: Wed, 9 Nov 2011 19:07:02 -0500 Message-Id: <1320883635-17194-5-git-send-email-Kyle.D.Moffett@boeing.com> X-Mailer: git-send-email 1.7.2.5 In-Reply-To: <4E42AB6F.1050900@freescale.com> References: <4E42AB6F.1050900@freescale.com> Cc: Baruch Siach , Timur Tabi , linux-kernel@vger.kernel.org, Paul Gortmaker , Paul Mackerras , Anton Blanchard , Kyle Moffett , Scott Wood X-BeenThere: linuxppc-dev@lists.ozlabs.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Errors-To: linuxppc-dev-bounces+patchwork-incoming=ozlabs.org@lists.ozlabs.org Sender: linuxppc-dev-bounces+patchwork-incoming=ozlabs.org@lists.ozlabs.org Certain processor types are co-supportable, and their machine-check handlers will be referenced if the entries in cputable.c are actually generated, so allow more than one machine-check handler to be built in. This fixes a bug where configuring FreeScale E5500 support (P5020DS) into the kernel would break machine-check handling on PPC64 A2 systems by using a bogus machine_check_generic() handler that does nothing. Signed-off-by: Kyle Moffett --- arch/powerpc/kernel/traps.c | 18 ++++++++---------- 1 files changed, 8 insertions(+), 10 deletions(-) diff --git a/arch/powerpc/kernel/traps.c b/arch/powerpc/kernel/traps.c index 4e59082..e3113341 100644 --- a/arch/powerpc/kernel/traps.c +++ b/arch/powerpc/kernel/traps.c @@ -418,7 +418,8 @@ int machine_check_47x(struct pt_regs *regs) return 0; } -#elif defined(CONFIG_E500) +#endif +#if defined(CONFIG_FSL_E500MC) || defined(CONFIG_FSL_E5500) int machine_check_e500mc(struct pt_regs *regs) { unsigned long mcsr = mfspr(SPRN_MCSR); @@ -517,7 +518,8 @@ silent_out: mtspr(SPRN_MCSR, mcsr); return mfspr(SPRN_MCSR) == 0 && recoverable; } - +#endif +#ifdef CONFIG_FSL_E500_V1_V2 int machine_check_e500(struct pt_regs *regs) { unsigned long reason = get_mc_reason(regs); @@ -557,12 +559,8 @@ int machine_check_e500(struct pt_regs *regs) return 0; } - -int machine_check_generic(struct pt_regs *regs) -{ - return 0; -} -#elif defined(CONFIG_E200) +#endif +#ifdef CONFIG_E200 int machine_check_e200(struct pt_regs *regs) { unsigned long reason = get_mc_reason(regs); @@ -587,7 +585,8 @@ int machine_check_e200(struct pt_regs *regs) return 0; } -#else +#endif + int machine_check_generic(struct pt_regs *regs) { unsigned long reason = get_mc_reason(regs); @@ -623,7 +622,6 @@ int machine_check_generic(struct pt_regs *regs) } return 0; } -#endif /* everything else */ void machine_check_exception(struct pt_regs *regs) {