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
| ||
|
Date: Thu, 2 Apr 2015 09:27:48 +1100 From: Julian Calaby <julian.calaby@...il.com> To: Joel Stanley <joel@....id.au> Cc: "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>, linuxppc-dev@...ts.ozlabs.org, sparclinux <sparclinux@...r.kernel.org>, Fabian Frederick <fabf@...net.be>, Andrew Morton <akpm@...ux-foundation.org>, benh@...nel.crashing.org, mpe@...erman.id.au, rusty@...tcorp.com.au, jk@...abs.org, "David S. Miller" <davem@...emloft.net> Subject: Re: [PATCH v2 2/3] kernel/reboot.c: Add orderly_reboot for graceful reboot Hi Joel, On Wed, Apr 1, 2015 at 6:04 PM, Joel Stanley <joel@....id.au> wrote: > The kernel has orderly_poweroff which allows the kernel to initiate a > graceful shutdown of userspace, by running /sbin/poweroff. This adds > orderly_reboot that will cause userspace to shut itself down by calling > /sbin/reboot. > > This will be used for shutdown initiated by a system controller on > platforms that do not use ACPI. > > orderly_reboot() should be used when the system wants to allow userspace > to gracefully shut itself down. For cases where the system may > imminently catch on fire, the existing emergency_restart() provides an > immediate reboot without involving userspace. > > Signed-off-by: Joel Stanley <joel@....id.au> > --- > V2: > - remove unused return value from orderly_* > - use correct command in orderly_poweroff > - the reboot command will not have a proc knob to control it, so > it can be made const and static > > include/linux/reboot.h | 3 ++- > kernel/reboot.c | 53 +++++++++++++++++++++++++++++++++++++++++++++----- > 2 files changed, 50 insertions(+), 6 deletions(-) > > diff --git a/include/linux/reboot.h b/include/linux/reboot.h > index 48bf152..dc5e3bd 100644 > --- a/include/linux/reboot.h > +++ b/include/linux/reboot.h > @@ -67,7 +67,8 @@ void ctrl_alt_del(void); > #define POWEROFF_CMD_PATH_LEN 256 > extern char poweroff_cmd[POWEROFF_CMD_PATH_LEN]; > > -extern int orderly_poweroff(bool force); > +extern void orderly_poweroff(bool force); Should making orderly_poweroff() void be in a separate patch? Thanks, -- Julian Calaby Email: julian.calaby@...il.com Profile: http://www.google.com/profiles/julian.calaby/ -- 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/
Powered by blists - more mailing lists