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 PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Fri, 13 May 2016 13:58:04 -0700 From: Calvin Owens <calvinowens@...com> To: "Paul E. McKenney" <paulmck@...ux.vnet.ibm.com>, Andrew Morton <akpm@...ux-foundation.org>, David Howells <dhowells@...hat.com>, Pranith Kumar <bobby.prani@...il.com>, David Woodhouse <David.Woodhouse@...el.com>, Johannes Weiner <hannes@...xchg.org>, Ard Biesheuvel <ard.biesheuvel@...aro.org>, Petr Mladek <pmladek@...e.com>, Tejun Heo <tj@...nel.org>, Sergey Senozhatsky <sergey.senozhatsky@...il.com>, Vasily Averin <vvs@...tuozzo.com>, Thierry Reding <treding@...dia.com>, Geliang Tang <geliangtang@....com>, Ivan Delalande <colona@...sta.com> CC: <linux-kernel@...r.kernel.org>, <kernel-team@...com>, <calvinowens@...com> Subject: [RFC][PATCH] printk: Add option to append kernel version to the dict We use netconsole to collect kernel logs from all the servers at Facebook. We use this patch internally so each logline has a record of which kernel version emitted it. At first glance, this might seem lazy: as you would expect, we have a database which records which kernel version a host is currently running. But there are a lot of situations where that database cannot be current: early-ish boot crashes are probably the best example, but even beyond that there are lots of varieties of kernel brokenness that can prevent the database from being updated. Doing it explicitly this way ensures that we always know exactly what version emitted a given message. Doing it in printk() itself rather than extended netconsole ends up being much simpler, and has the advantage that future extended console implementations will be able to benefit from this as well. Signed-off-by: Calvin Owens <calvinowens@...com> --- init/Kconfig | 8 ++++++++ kernel/printk/printk.c | 14 ++++++++++++++ 2 files changed, 22 insertions(+) diff --git a/init/Kconfig b/init/Kconfig index 0dfd09d..f800256 100644 --- a/init/Kconfig +++ b/init/Kconfig @@ -1454,6 +1454,14 @@ config PRINTK very difficult to diagnose system problems, saying N here is strongly discouraged. +config PRINTK_APPEND_UNAME + default n + bool "Append the kernel version to the printk() dict" if EXPERT + help + This option causes the printk() implementation to append the kernel + version for extended console messages. This can be very useful for + monitoring a large deployment of servers with extended netconsole. + config BUG bool "BUG() support" if EXPERT default y diff --git a/kernel/printk/printk.c b/kernel/printk/printk.c index bfbf284..3f8debd 100644 --- a/kernel/printk/printk.c +++ b/kernel/printk/printk.c @@ -608,6 +608,18 @@ static ssize_t msg_print_ext_body(char *buf, size_t size, return p - buf; } +#if defined(CONFIG_PRINTK_APPEND_UNAME) +static ssize_t msg_print_ext_uname(char *buf, size_t size) +{ + return scnprintf(buf, size, " UNAME=%s\n", init_utsname()->release); +} +#else +static ssize_t msg_print_ext_uname(char *buf, size_t size) +{ + return 0; +} +#endif + /* /dev/kmsg - userspace message inject/listen interface */ struct devkmsg_user { u64 seq; @@ -2305,6 +2317,8 @@ skip: sizeof(ext_text) - ext_len, log_dict(msg), msg->dict_len, log_text(msg), msg->text_len); + ext_len += msg_print_ext_uname(ext_text + ext_len, + sizeof(ext_text) - ext_len); } console_idx = log_next(console_idx); console_seq++; -- 2.8.0.rc2
Powered by blists - more mailing lists