linux-kernel - Re: [PATCH] kernel/panic: Add "late_kdump" option for kdump in unstable condition

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Thu, 17 Apr 2014 09:43:38 -0400
From:	Vivek Goyal <vgoyal@...hat.com>
To:	Masami Hiramatsu <masami.hiramatsu.pt@...achi.com>
Cc:	linux-kernel@...r.kernel.org,
	Satoru MORIYA <satoru.moriya.br@...achi.com>,
	Yoshihiro YUNOMAE <yoshihiro.yunomae.ez@...achi.com>,
	Takenori Nagano <t-nagano@...jp.nec.com>,
	Eric Biederman <ebiederm@...ssion.com>,
	Motohiro Kosaki <Motohiro.Kosaki@...fujitsu.com>,
	Andrew Morton <akpm@...ux-foundation.org>
Subject: Re: [PATCH] kernel/panic: Add "late_kdump" option for kdump in
 unstable condition

On Thu, Apr 17, 2014 at 10:59:14AM +0900, Masami Hiramatsu wrote:

[..]
> >> @@ -112,9 +113,14 @@ void panic(const char *fmt, ...)
> >>  	/*
> >>  	 * If we have crashed and we have a crash kernel loaded let it handle
> >>  	 * everything else.
> >> -	 * Do we want to call this before we try to display a message?
> >> +	 * If we want to call this after we try to display a message, pass
> >> +	 * the "late_kdump" option to the kernel.
> >>  	 */
> >> -	crash_kexec(NULL);
> >> +	if (!late_kdump)
> >> +		crash_kexec(NULL);
> >> +	else
> >> +		pr_emerg("Warning: late_kdump option is set. Please DO NOT "
> >> +			"report bugs about kdump failure with this option.\n");
> > 
> > I think above message about DO NOT report bugs seems unnecessary. 
> 
> OK, so I just notify the option is set as below.
> "Warning: crash_kexec_post_notifiers is set.\n"

I would say for the time being don't put any extra message. User will
think what this warning is supposed to mean. We can introduce one if
we run into many issues.

For the time being we can just look at kernel logs and look at command
line options and see whether crash_kexec_post_notifiers was set or not.

Thanks
Vivek
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/