printk: remove used-once early_vprintk
authorJoe Perches <joe@perches.com>
Wed, 10 Dec 2014 23:45:53 +0000 (15:45 -0800)
committerLinus Torvalds <torvalds@linux-foundation.org>
Thu, 11 Dec 2014 01:41:10 +0000 (17:41 -0800)
Eliminate the unlikely possibility of message interleaving for
early_printk/early_vprintk use.

early_vprintk can be done via the %pV extension so remove this
unnecessary function and change early_printk to have the equivalent
vprintk code.

All uses of early_printk already end with a newline so also remove the
unnecessary newline from the early_printk function.

Signed-off-by: Joe Perches <joe@perches.com>
Acked-by: Chris Metcalf <cmetcalf@tilera.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
arch/tile/kernel/early_printk.c
include/linux/printk.h
kernel/printk/printk.c

index b608e00e7f6d91f6be53a59ecc8e0268795d3dcb..aefb2c086726d37ceff456b57576691a68d7f5e7 100644 (file)
@@ -43,13 +43,20 @@ static struct console early_hv_console = {
 
 void early_panic(const char *fmt, ...)
 {
-       va_list ap;
+       struct va_format vaf;
+       va_list args;
+
        arch_local_irq_disable_all();
-       va_start(ap, fmt);
-       early_printk("Kernel panic - not syncing: ");
-       early_vprintk(fmt, ap);
-       early_printk("\n");
-       va_end(ap);
+
+       va_start(args, fmt);
+
+       vaf.fmt = fmt;
+       vaf.va = &args;
+
+       early_printk("Kernel panic - not syncing: %pV", &vaf);
+
+       va_end(args);
+
        dump_stack();
        hv_halt();
 }
index d78125f73ac4f9e3838c2449921bb69301c02261..3dd489f2dedc264eb57dc6e613669307d4a861e9 100644 (file)
@@ -118,7 +118,6 @@ int no_printk(const char *fmt, ...)
 #ifdef CONFIG_EARLY_PRINTK
 extern asmlinkage __printf(1, 2)
 void early_printk(const char *fmt, ...);
-void early_vprintk(const char *fmt, va_list ap);
 #else
 static inline __printf(1, 2) __cold
 void early_printk(const char *s, ...) { }
index ced2b84b1cb7b4f0c11e0c3d9a09bc83b1b435f0..4815c98ae175762eb1f9229b702107e9a0947f7c 100644 (file)
@@ -1881,23 +1881,20 @@ static size_t cont_print_text(char *text, size_t size) { return 0; }
 #ifdef CONFIG_EARLY_PRINTK
 struct console *early_console;
 
-void early_vprintk(const char *fmt, va_list ap)
-{
-       if (early_console) {
-               char buf[512];
-               int n = vscnprintf(buf, sizeof(buf), fmt, ap);
-
-               early_console->write(early_console, buf, n);
-       }
-}
-
 asmlinkage __visible void early_printk(const char *fmt, ...)
 {
        va_list ap;
+       char buf[512];
+       int n;
+
+       if (!early_console)
+               return;
 
        va_start(ap, fmt);
-       early_vprintk(fmt, ap);
+       n = vscnprintf(buf, sizeof(buf), fmt, ap);
        va_end(ap);
+
+       early_console->write(early_console, buf, n);
 }
 #endif