aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAravind Gopalakrishnan <Aravind.Gopalakrishnan@amd.com>2015-03-31 11:04:41 -0400
committerIngo Molnar <mingo@kernel.org>2015-04-03 09:20:38 -0400
commitcee8f5a6c8c917613dd021552909d071b1dab592 (patch)
tree87b67514e0b4470ac3282b037c1499dd7201a53c
parentf5c8a104116a56503b6e824e7782b2e805b29abb (diff)
x86/mce/severity: Fix warning about indented braces
Dan reported compiler warnings about missing curly braces in mce_severity_amd(). Reindent the catch-all "return MCE_AR_SEVERITY" correctly to single tab. While at it, chain ctx == IN_KERNEL check with mcgstatus check to make it cleaner, as suggested by Boris. No functional changes are introduced by this patch. Reported-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Aravind Gopalakrishnan <Aravind.Gopalakrishnan@amd.com> Signed-off-by: Borislav Petkov <bp@suse.de> Cc: H. Peter Anvin <hpa@zytor.com> Cc: Thomas Gleixner <tglx@linutronix.de> Cc: Tony Luck <tony.luck@intel.com> Cc: linux-edac <linux-edac@vger.kernel.org> Link: http://lkml.kernel.org/r/1427814281-18192-1-git-send-email-Aravind.Gopalakrishnan@amd.com Signed-off-by: Ingo Molnar <mingo@kernel.org>
-rw-r--r--arch/x86/kernel/cpu/mcheck/mce-severity.c9
1 files changed, 4 insertions, 5 deletions
diff --git a/arch/x86/kernel/cpu/mcheck/mce-severity.c b/arch/x86/kernel/cpu/mcheck/mce-severity.c
index 155c9261d3ef..9c682c222071 100644
--- a/arch/x86/kernel/cpu/mcheck/mce-severity.c
+++ b/arch/x86/kernel/cpu/mcheck/mce-severity.c
@@ -208,12 +208,11 @@ static int mce_severity_amd(struct mce *m, int tolerant, char **msg, bool is_exc
208 */ 208 */
209 if (mce_flags.overflow_recov) { 209 if (mce_flags.overflow_recov) {
210 /* software can try to contain */ 210 /* software can try to contain */
211 if (!(m->mcgstatus & MCG_STATUS_RIPV)) 211 if (!(m->mcgstatus & MCG_STATUS_RIPV) && (ctx == IN_KERNEL))
212 if (ctx == IN_KERNEL) 212 return MCE_PANIC_SEVERITY;
213 return MCE_PANIC_SEVERITY;
214 213
215 /* kill current process */ 214 /* kill current process */
216 return MCE_AR_SEVERITY; 215 return MCE_AR_SEVERITY;
217 } else { 216 } else {
218 /* at least one error was not logged */ 217 /* at least one error was not logged */
219 if (m->status & MCI_STATUS_OVER) 218 if (m->status & MCI_STATUS_OVER)