diff options
author | Gerd Hoffmann <kraxel@suse.de> | 2007-05-08 03:26:49 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-05-08 14:15:04 -0400 |
commit | 69331af79cf29e26d1231152a172a1a10c2df511 (patch) | |
tree | 0c6f805fc78c1969b8c46f02070cb9dc39f3f944 /arch/alpha | |
parent | 6ae9200f2cab7b328e505fc9a7021db64e0590cf (diff) |
Fixes and cleanups for earlyprintk aka boot console
The console subsystem already has an idea of a boot console, using the
CON_BOOT flag. The implementation has some flaws though. The major
problem is that presence of a boot console makes register_console() ignore
any other console devices (unless explicitly specified on the kernel
command line).
This patch fixes the console selection code to *not* consider a boot
console a full-featured one, so the first non-boot console registering will
become the default console instead. This way the unregister call for the
boot console in the register_console() function actually triggers and the
handover from the boot console to the real console device works smoothly.
Added a printk for the handover, so you know which console device the
output goes to when the boot console stops printing messages.
The disable_early_printk() call is obsolete with that patch, explicitly
disabling the early console isn't needed any more as it works automagically
with that patch.
I've walked through the tree, dropped all disable_early_printk() instances
found below arch/ and tagged the consoles with CON_BOOT if needed. The
code is tested on x86, sh (thanks to Paul) and mips (thanks to Ralf).
Changes to last version: Rediffed against -rc3, adapted to mips cleanups by
Ralf, fixed "udbg-immortal" cmd line arg on powerpc.
Signed-off-by: Gerd Hoffmann <kraxel@exsuse.de>
Acked-by: Paul Mundt <lethal@linux-sh.org>
Acked-by: Ralf Baechle <ralf@linux-mips.org>
Cc: Andi Kleen <ak@suse.de>
Cc: Alan Cox <alan@lxorguk.ukuu.org.uk>
Cc: Richard Henderson <rth@twiddle.net>
Cc: Ivan Kokshaysky <ink@jurassic.park.msu.ru>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Cc: Jeremy Fitzhardinge <jeremy@goop.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'arch/alpha')
-rw-r--r-- | arch/alpha/kernel/setup.c | 9 | ||||
-rw-r--r-- | arch/alpha/kernel/srmcons.c | 2 |
2 files changed, 1 insertions, 10 deletions
diff --git a/arch/alpha/kernel/setup.c b/arch/alpha/kernel/setup.c index d352c2b05f1a..915f26345c45 100644 --- a/arch/alpha/kernel/setup.c +++ b/arch/alpha/kernel/setup.c | |||
@@ -744,15 +744,6 @@ setup_arch(char **cmdline_p) | |||
744 | paging_init(); | 744 | paging_init(); |
745 | } | 745 | } |
746 | 746 | ||
747 | void __init | ||
748 | disable_early_printk(void) | ||
749 | { | ||
750 | if (alpha_using_srm && srmcons_output) { | ||
751 | unregister_srm_console(); | ||
752 | srmcons_output = 0; | ||
753 | } | ||
754 | } | ||
755 | |||
756 | static char sys_unknown[] = "Unknown"; | 747 | static char sys_unknown[] = "Unknown"; |
757 | static char systype_names[][16] = { | 748 | static char systype_names[][16] = { |
758 | "0", | 749 | "0", |
diff --git a/arch/alpha/kernel/srmcons.c b/arch/alpha/kernel/srmcons.c index 85a821aaceb4..930cedc8be24 100644 --- a/arch/alpha/kernel/srmcons.c +++ b/arch/alpha/kernel/srmcons.c | |||
@@ -300,7 +300,7 @@ static struct console srmcons = { | |||
300 | .write = srm_console_write, | 300 | .write = srm_console_write, |
301 | .device = srm_console_device, | 301 | .device = srm_console_device, |
302 | .setup = srm_console_setup, | 302 | .setup = srm_console_setup, |
303 | .flags = CON_PRINTBUFFER, | 303 | .flags = CON_PRINTBUFFER | CON_BOOT, |
304 | .index = -1, | 304 | .index = -1, |
305 | }; | 305 | }; |
306 | 306 | ||