From: Oleg Drokin Date: Wed, 26 Jul 2017 06:03:12 +0000 (-0400) Subject: Module parameter to enable spl_panic() to panic the kernel X-Git-Tag: spl-0.7.12~51 X-Git-Url: https://git.proxmox.com/?p=mirror_spl.git;a=commitdiff_plain;h=410f7ab5943684ef74be0e58ae17d3a0278ad0be Module parameter to enable spl_panic() to panic the kernel In unattended operations it's often more useful to have node panic and reboot when it encounters problems as opposed to sit there indefinitely waiting for somebody to discover it. This implements an spl_panic_crash module parameter, set it to nonzero to cause spl_panic() to call panic(). Reviewed-by: Brian Behlendorf Reviewed-by: Giuseppe Di Natale Signed-off-by: Oleg Drokin Closes #634 --- diff --git a/module/spl/spl-err.c b/module/spl/spl-err.c index cf9f096..2f66b6e 100644 --- a/module/spl/spl-err.c +++ b/module/spl/spl-err.c @@ -28,6 +28,17 @@ #include #include +/* + * It is often useful to actually have the panic crash the node so you + * can then get notified of the event, get the crashdump for later + * analysis and other such goodies. + * But we would still default to the current default of not to do that. + */ +unsigned int spl_panic_halt; +module_param(spl_panic_halt, uint, 0644); +MODULE_PARM_DESC(spl_panic_halt, + "Cause kernel panic on assertion failures"); + /* * Limit the number of stack traces dumped to not more than 5 every * 60 seconds to prevent denial-of-service attacks from debug code. @@ -62,6 +73,9 @@ spl_panic(const char *file, const char *func, int line, const char *fmt, ...) { printk(KERN_EMERG "%s", msg); printk(KERN_EMERG "PANIC at %s:%d:%s()\n", newfile, line, func); + if (spl_panic_halt) + panic("%s", msg); + spl_dumpstack(); /* Halt the thread to facilitate further debugging */