printk: Add per_cpu printk func to allow printk to be diverted
authorSteven Rostedt (Red Hat) <rostedt@goodmis.org>
Thu, 19 Jun 2014 21:33:31 +0000 (17:33 -0400)
committerSteven Rostedt <rostedt@goodmis.org>
Thu, 20 Nov 2014 03:01:21 +0000 (22:01 -0500)
Being able to divert printk to call another function besides the normal
logging is useful for such things like NMI handling. If some functions
are to be called from NMI that does printk() it is possible to lock up
the box if the nmi handler triggers when another printk is happening.

One example of this use is to perform a stack trace on all CPUs via NMI.
But if the NMI is to do the printk() it can cause the system to lock up.
By allowing the printk to be diverted to another function that can safely
record the printk output and then print it when it in a safe context
then NMIs will be safe to call these functions like show_regs().

Link: http://lkml.kernel.org/p/20140619213952.209176403@goodmis.org
Tested-by: Jiri Kosina <jkosina@suse.cz>
Acked-by: Jiri Kosina <jkosina@suse.cz>
Acked-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
Reviewed-by: Petr Mladek <pmladek@suse.cz>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
include/linux/percpu.h
include/linux/printk.h
kernel/printk/printk.c

index a3aa63e47637c9dc9bfa77a1166aadf67df3b158..ba2e85a0ff5baf42793a4e7f3a9a5e24c73ff7dd 100644 (file)
@@ -134,4 +134,7 @@ extern phys_addr_t per_cpu_ptr_to_phys(void *addr);
        (typeof(type) __percpu *)__alloc_percpu(sizeof(type),           \
                                                __alignof__(type))
 
+/* To avoid include hell, as printk can not declare this, we declare it here */
+DECLARE_PER_CPU(printk_func_t, printk_func);
+
 #endif /* __LINUX_PERCPU_H */
index d78125f73ac4f9e3838c2449921bb69301c02261..3bbd979d32fbeecd09e7bb6cb082e25a4977fd15 100644 (file)
@@ -162,6 +162,8 @@ extern int kptr_restrict;
 
 extern void wake_up_klogd(void);
 
+typedef int(*printk_func_t)(const char *fmt, va_list args);
+
 void log_buf_kexec_setup(void);
 void __init setup_log_buf(int early);
 void dump_stack_set_arch_desc(const char *fmt, ...);
index ced2b84b1cb7b4f0c11e0c3d9a09bc83b1b435f0..f7b723f98cb90f098217d534e526ae8548b3c6a2 100644 (file)
@@ -1807,6 +1807,30 @@ asmlinkage int printk_emit(int facility, int level,
 }
 EXPORT_SYMBOL(printk_emit);
 
+int vprintk_default(const char *fmt, va_list args)
+{
+       int r;
+
+#ifdef CONFIG_KGDB_KDB
+       if (unlikely(kdb_trap_printk)) {
+               r = vkdb_printf(fmt, args);
+               return r;
+       }
+#endif
+       r = vprintk_emit(0, -1, NULL, 0, fmt, args);
+
+       return r;
+}
+EXPORT_SYMBOL_GPL(vprintk_default);
+
+/*
+ * This allows printk to be diverted to another function per cpu.
+ * This is useful for calling printk functions from within NMI
+ * without worrying about race conditions that can lock up the
+ * box.
+ */
+DEFINE_PER_CPU(printk_func_t, printk_func) = vprintk_default;
+
 /**
  * printk - print a kernel message
  * @fmt: format string
@@ -1830,19 +1854,15 @@ EXPORT_SYMBOL(printk_emit);
  */
 asmlinkage __visible int printk(const char *fmt, ...)
 {
+       printk_func_t vprintk_func;
        va_list args;
        int r;
 
-#ifdef CONFIG_KGDB_KDB
-       if (unlikely(kdb_trap_printk)) {
-               va_start(args, fmt);
-               r = vkdb_printf(fmt, args);
-               va_end(args);
-               return r;
-       }
-#endif
        va_start(args, fmt);
-       r = vprintk_emit(0, -1, NULL, 0, fmt, args);
+       preempt_disable();
+       vprintk_func = this_cpu_read(printk_func);
+       r = vprintk_func(fmt, args);
+       preempt_enable();
        va_end(args);
 
        return r;