From mboxrd@z Thu Jan 1 00:00:00 1970 From: dwalker@fifo99.com Subject: Re: [PATCH 1/3] panic: Disable crash_kexec_post_notifiers if kdump is not available Date: Mon, 13 Jul 2015 20:26:11 +0000 Message-ID: <20150713202611.GA16525@fifo99.com> References: <20150710113331.4368.10495.stgit@softrs> <20150710113331.4368.63745.stgit@softrs> <87wpy82kqf.fsf@x220.int.ebiederm.org> Mime-Version: 1.0 Return-path: Content-Disposition: inline In-Reply-To: <87wpy82kqf.fsf@x220.int.ebiederm.org> Sender: linux-kernel-owner@vger.kernel.org List-ID: Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: "Eric W. Biederman" Cc: Hidehiro Kawai , Andrew Morton , Vivek Goyal , linux-mips@linux-mips.org, Baoquan He , linux-sh@vger.kernel.org, linux-s390@vger.kernel.org, kexec@lists.infradead.org, linux-kernel@vger.kernel.org, Ingo Molnar , HATAYAMA Daisuke , Masami Hiramatsu , linuxppc-dev@lists.ozlabs.org, linux-metag@vger.kernel.org, linux-arm-kernel@lists.infradead.org On Fri, Jul 10, 2015 at 08:41:28AM -0500, Eric W. Biederman wrote: > Hidehiro Kawai writes: > > > You can call panic notifiers and kmsg dumpers before kdump by > > specifying "crash_kexec_post_notifiers" as a boot parameter. > > However, it doesn't make sense if kdump is not available. In that > > case, disable "crash_kexec_post_notifiers" boot parameter so that > > you can't change the value of the parameter. > > Nacked-by: "Eric W. Biederman" I think it would make sense if he just replaced "kdump" with "kexec". Daniel