Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S966521Ab3DQTN5 (ORCPT ); Wed, 17 Apr 2013 15:13:57 -0400 Received: from relay1.sgi.com ([192.48.179.29]:46490 "EHLO relay.sgi.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757200Ab3DQTN4 (ORCPT ); Wed, 17 Apr 2013 15:13:56 -0400 Date: Wed, 17 Apr 2013 14:13:54 -0500 From: Robin Holt To: Ingo Molnar , "H. Peter Anvin" , "Srivatsa S. Bhat" , Russ Anderson Cc: Linux Kernel Mailing List , the arch/x86 maintainers Subject: Re: [PATCH -v5 4/5] checkpatch.pl the new kernel/reboot.c file. Message-ID: <20130417191354.GH3658@sgi.com> References: <1366224198-49485-1-git-send-email-holt@sgi.com> <1366224198-49485-5-git-send-email-holt@sgi.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1366224198-49485-5-git-send-email-holt@sgi.com> User-Agent: Mutt/1.5.20 (2009-06-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4264 Lines: 122 On Wed, Apr 17, 2013 at 01:43:17PM -0500, Robin Holt wrote: > I did allow the remaining 81 character line behind. It did not seem > like it was worth changing. Otherwise, it now passes checkpatch.pl. > > Signed-off-by: Robin Holt > To: Ingo Molnar > To: Russ Anderson > Cc: Shawn Guo > Cc: Oleg Nesterov > Cc: Andrew Morton > Cc: "H. Peter Anvin" > Cc: Lai Jiangshan > Cc: Linus Torvalds > Cc: Linux Kernel Mailing List > Cc: Michel Lespinasse > Cc: Oleg Nesterov > Cc: "Paul E. McKenney" > Cc: Paul Mackerras > Cc: Peter Zijlstra > Cc: Robin Holt > Cc: "rusty@rustcorp.com.au" > Cc: Tejun Heo > Cc: the arch/x86 maintainers > Cc: Thomas Gleixner > Cc: Joe Perches > > --- > > Changes since -v3: > Integrated feedback from Joe Perches. > --- > kernel/reboot.c | 26 +++++++++++++------------- > 1 file changed, 13 insertions(+), 13 deletions(-) > > diff --git a/kernel/reboot.c b/kernel/reboot.c > index ff92601..c4eb2a4 100644 > --- a/kernel/reboot.c > +++ b/kernel/reboot.c > @@ -4,6 +4,8 @@ > * Copyright (C) 2013 Linus Torvalds > */ > > +#define pr_fmt(fmt) "reboot: " fmt > + > #include > #include > #include > @@ -99,9 +101,9 @@ void kernel_restart(char *cmd) > migrate_to_reboot_cpu(); > syscore_shutdown(); > if (!cmd) > - printk(KERN_EMERG "Restarting system.\n"); > + pr_emerg("Restarting system\n"); > else > - printk(KERN_EMERG "Restarting system with command '%s'.\n", cmd); > + pr_emerg("Restarting system with command '%s'\n", cmd); > kmsg_dump(KMSG_DUMP_RESTART); > machine_restart(cmd); > } > @@ -110,7 +112,7 @@ EXPORT_SYMBOL_GPL(kernel_restart); > static void kernel_shutdown_prepare(enum system_states state) > { > blocking_notifier_call_chain(&reboot_notifier_list, > - (state == SYSTEM_HALT)?SYS_HALT:SYS_POWER_OFF, NULL); > + (state == SYSTEM_HALT) ? SYS_HALT : SYS_POWER_OFF, NULL); > system_state = state; > usermodehelper_disable(); > device_shutdown(); > @@ -125,11 +127,10 @@ void kernel_halt(void) > kernel_shutdown_prepare(SYSTEM_HALT); > migrate_to_reboot_cpu(); > syscore_shutdown(); > - printk(KERN_EMERG "System halted.\n"); > + pr_emerg("System halted\n"); > kmsg_dump(KMSG_DUMP_HALT); > machine_halt(); > } > - > EXPORT_SYMBOL_GPL(kernel_halt); > > /** > @@ -144,7 +145,7 @@ void kernel_power_off(void) > pm_power_off_prepare(); > migrate_to_reboot_cpu(); > syscore_shutdown(); > - printk(KERN_EMERG "Power down.\n"); > + pr_emerg("Power down\n"); > kmsg_dump(KMSG_DUMP_POWEROFF); > machine_power_off(); > } > @@ -173,10 +174,10 @@ SYSCALL_DEFINE4(reboot, int, magic1, int, magic2, unsigned int, cmd, > > /* For safety, we require "magic" arguments. */ > if (magic1 != LINUX_REBOOT_MAGIC1 || > - (magic2 != LINUX_REBOOT_MAGIC2 && > - magic2 != LINUX_REBOOT_MAGIC2A && > + (magic2 != LINUX_REBOOT_MAGIC2 && > + magic2 != LINUX_REBOOT_MAGIC2A && > magic2 != LINUX_REBOOT_MAGIC2B && > - magic2 != LINUX_REBOOT_MAGIC2C)) > + magic2 != LINUX_REBOOT_MAGIC2C)) > return -EINVAL; > > /* > @@ -285,14 +286,13 @@ static int __orderly_poweroff(bool force) > ret = call_usermodehelper(argv[0], argv, envp, UMH_WAIT_EXEC); > argv_free(argv); > } else { > - printk(KERN_WARNING "%s failed to allocate memory for \"%s\"\n", > - __func__, poweroff_cmd); > + pr_warn("%s failed to allocate memory for \"%s\"\n", > + __func__, poweroff_cmd); Argh. I had intended to remove this. I will fix up my workarea and be ready to not miss it on the next pass. Robin -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/