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] [day] [month] [year] [list]
Date:	Tue, 15 Jul 2014 09:12:51 -0700
From:	Guenter Roeck <>
To:	Catalin Marinas <>
Cc:	"" <>,
	Wim Van Sebroeck <>,
	Maxime Ripard <>,
	Will Deacon <>,
	Arnd Bergmann <>,
	Heiko Stuebner <>,
	Russell King <>,
	Jonas Jensen <>,
	Randy Dunlap <>,
	Andrew Morton <>,
	Steven Rostedt <>,
	Ingo Molnar <>,
	Dmitry Eremin-Solenikov <>,
	David Woodhouse <>,
	Tomasz Figa <>,
	"" <>,
	"" <>
Subject: Re: [PATCH v4 1/7] kernel: Add support for restart notifier call

On Mon, Jul 14, 2014 at 07:58:59AM -0700, Guenter Roeck wrote:
> On 07/14/2014 07:53 AM, Catalin Marinas wrote:
[ ... ]
> >
> >I'm not fully convinced implying a 'notifier' is the right approach
> >here. By analogy with the reboot notifier, this is something drivers
> >would want to know about and do some work before the actual system
> >restart (e.g. disable watchdogs as in the reboot notifier case). The
> >restart notifier here is meant to perform the actual system restart.
> >Arguably, the actual restart should be handled by priority 0 with some
> >preparation before but we have reboot notifier already, so I don't think
> >it's worth another notifier.
> >
> >While re-using the notifier mechanism behind the scene is fine, I think
> >we should at least rename the functions to something like
> >(un)register_restart_handler().
> >
> Fine with me. Any other comments / suggestions on the name of the function ?
Any further feedback on the function names ? If not I'll resubmit the series
tonight or tomorrow morning with the functions renamed as suggested above.

To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
More majordomo info at
Please read the FAQ at

Powered by blists - more mailing lists