Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758232Ab1EZSod (ORCPT ); Thu, 26 May 2011 14:44:33 -0400 Received: from mx1.redhat.com ([209.132.183.28]:24020 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1758117Ab1EZSoc (ORCPT ); Thu, 26 May 2011 14:44:32 -0400 Date: Thu, 26 May 2011 14:43:50 -0400 From: Vivek Goyal To: "K.Prasad" Cc: Linux Kernel Mailing List , Andi Kleen , "Luck, Tony" , kexec@lists.infradead.org, "Eric W. Biederman" , anderson@redhat.com Subject: Re: [RFC Patch 4/6] PANIC_MCE: Introduce a new panic flag for fatal MCE, capture related information Message-ID: <20110526184350.GE29496@redhat.com> References: <20110526170722.GB23266@in.ibm.com> <20110526171521.GD17988@in.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20110526171521.GD17988@in.ibm.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1552 Lines: 41 On Thu, May 26, 2011 at 10:45:21PM +0530, K.Prasad wrote: [..] > Index: linux-2.6.slim_kdump/arch/x86/kernel/cpu/mcheck/mce.c > =================================================================== > --- linux-2.6.slim_kdump.orig/arch/x86/kernel/cpu/mcheck/mce.c > +++ linux-2.6.slim_kdump/arch/x86/kernel/cpu/mcheck/mce.c > @@ -258,8 +258,7 @@ static void wait_for_panic(void) > local_irq_enable(); > while (timeout-- > 0) > udelay(1); > - xpanic(PANIC_NO_KEXEC|PANIC_NO_BACKTRACE, 0, > - "Panicing machine check CPU died"); > + xpanic(PANIC_MCE, 0, NULL, 0, "Panicing machine check CPU died"); > } > > static void mce_panic(char *msg, struct mce *final, char *exp) > @@ -315,8 +314,8 @@ static void mce_panic(char *msg, struct > if (exp) > pr_emerg(HW_ERR "Machine check: %s\n", exp); > if (!fake_panic) { > - xpanic(PANIC_NO_KEXEC|PANIC_NO_BACKTRACE, mce_panic_timeout, > - msg); > + xpanic(PANIC_MCE, mce_panic_timeout, final, > + sizeof(struct mce), msg); > } else > pr_emerg(HW_ERR "Fake kernel panic: %s\n", msg); > } In previous patches you introduce PANIC_NO_KEXEC and PANIC_NO_BACKTRACE. Now in this patch you got rid of those. Are there any other users left of PANIC_NO_BACKTRACE and PANIC_NO_EXEC? If not, then why to introduce these to begin with. Thanks Vivek -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/