From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mx1.redhat.com ([209.132.183.28]) by canuck.infradead.org with esmtp (Exim 4.76 #1 (Red Hat Linux)) id 1QnzIc-0001pB-8h for kexec@lists.infradead.org; Mon, 01 Aug 2011 20:41:18 +0000 Date: Mon, 1 Aug 2011 16:41:14 -0400 From: Vivek Goyal Subject: Re: [patch v2 04/10] kdump: Trigger kdump via panic notifier chain on s390 Message-ID: <20110801204114.GJ3805@redhat.com> References: <20110727125504.491183728@linux.vnet.ibm.com> <20110727125808.028895659@linux.vnet.ibm.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20110727125808.028895659@linux.vnet.ibm.com> List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: kexec-bounces@lists.infradead.org Errors-To: kexec-bounces+dwmw2=twosheds.infradead.org@lists.infradead.org To: Michael Holzheu Cc: oomichi@mxs.nes.nec.co.jp, linux-s390@vger.kernel.org, mahesh@linux.vnet.ibm.com, heiko.carstens@de.ibm.com, linux-kernel@vger.kernel.org, hbabu@us.ibm.com, horms@verge.net.au, ebiederm@xmission.com, schwidefsky@de.ibm.com, kexec@lists.infradead.org On Wed, Jul 27, 2011 at 02:55:08PM +0200, Michael Holzheu wrote: > From: Michael Holzheu > > On s390 we have the possibility to configure actions that are executed in > case of a kernel panic. E.g. it is possible to automatically trigger an s390 > stand-alone dump. The actions are called via a panic notifier. We also want > to trigger kdump via the notifier call chain. Therefore this patch disables > for s390 the direct kdump invocation in the panic() function. Doesn't this reduce the reliability of the operation as you are assuming that panic notifier list is fine and not corrupted. There might be other generic notifiers registerd on panic notifier list too. So in your case, are there multiple subsystem registering for panic notifiers? If not, why not call crash_kexec() directly. Are there any other actions you want to take on panic then calling crash_kexec()? Thanks Vivek > > Signed-off-by: Michael Holzheu > --- > kernel/panic.c | 3 +++ > 1 file changed, 3 insertions(+) > > --- a/kernel/panic.c > +++ b/kernel/panic.c > @@ -84,9 +84,12 @@ NORET_TYPE void panic(const char * fmt, > /* > * If we have crashed and we have a crash kernel loaded let it handle > * everything else. > + * For s390 kdump is triggered via the panic notifier call chain. > * Do we want to call this before we try to display a message? > */ > +#if !defined(CONFIG_S390) > crash_kexec(NULL); > +#endif > > kmsg_dump(KMSG_DUMP_PANIC); > _______________________________________________ kexec mailing list kexec@lists.infradead.org http://lists.infradead.org/mailman/listinfo/kexec