Re: [PATCH] printk: Use symbolic defines for console loglevels

[Randy Dunlap]

On 05/19/2014 11:27 AM, Borislav Petkov wrote:
> From: Borislav Petkov <bp@xxxxxxx>
> 
> ... instead of naked numbers.
> 
> Stuff in sysrq.c used to set it to 8 which is supposed to mean above
> default level so set it to DEBUG instead as we're terminating/killing
> all tasks and we want to be verbose there.
> 
> There should be no functionality change resulting from this patch.
> 

Acked-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx>

Thanks.

> Cc: Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx>
> Cc: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
> Cc: Randy Dunlap <rdunlap@xxxxxxxxxxxxx>
> Cc: x86@xxxxxxxxxx
> Cc: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
> Cc: Jiri Slaby <jslaby@xxxxxxx>
> Cc: Jason Wessel <jason.wessel@xxxxxxxxxxxxx>
> Cc: kgdb-bugreport@xxxxxxxxxxxxxxxxxxxxx
> Cc: Hedi Berriche <hedi@xxxxxxx>
> Cc: Mike Travis <travis@xxxxxxx>
> Cc: Dimitri Sivanich <sivanich@xxxxxxx>
> Cc: Steven Rostedt <rostedt@xxxxxxxxxxx>
> Cc: Kees Cook <keescook@xxxxxxxxxxxx>
> Cc: Peter Zijlstra <peterz@xxxxxxxxxxxxx>
> Signed-off-by: Borislav Petkov <bp@xxxxxxx>
> ---
>  arch/x86/kernel/head64.c      |  2 +-
>  arch/x86/platform/uv/uv_nmi.c |  2 +-
>  drivers/nubus/nubus.c         | 18 +++++++++---------
>  drivers/tty/sysrq.c           |  8 ++++----
>  include/linux/printk.h        | 15 +++++++++++++--
>  init/main.c                   |  4 ++--
>  kernel/debug/kdb/kdb_bt.c     |  2 +-
>  kernel/debug/kdb/kdb_io.c     |  2 +-
>  kernel/debug/kdb/kdb_main.c   |  2 +-
>  kernel/printk/printk.c        | 13 +++----------
>  10 files changed, 36 insertions(+), 32 deletions(-)


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