Re: 2.5.26 broken on headless boxes

From: Petr Vandrovec (vandrove@vc.cvut.cz)
Date: Thu Jul 18 2002 - 17:16:19 EST


On 18 Jul 02 at 22:45, Matthew Wilcox wrote:
> On Thu, Jul 18, 2002 at 11:42:18PM +0200, Petr Vandrovec wrote:
> > CONFIG_VGA_CONSOLE/CONFIG_DUMMY_CONSOLE determines whether your VT can
> > be created at all - maybe _CONSOLE suffix is misleading - without
> > having at least one displaying device virtual terminals cannot be build.
> > I always thought that CONFIG_DUMMY_CONSOLE cannot be unset, but
> > apparently it can...
> >
> > And BTW, when such configuration worked for you last time? It does not
> > look to me like that it should ever work.
>
> erm, 2.5.25 worked, and i didn't change the .config between 2.5.25 and
> 2.5.26 (just ran make oldconfig).

Got it. It was introduced in console.c:1.13 change from jsimmons. Before
this change we did not registered tty driver:

con_init says:
  const char* display_desc = NULL;
  if (conswitchp) display_desc = conswitchp->con_startup();
  if (!display_desc) {
     fg_console = 0;
     return;
  }
  ...
  if (tty_register_driver(&console_driver)) ...

so we did not registered VT subsystem and panic did not happened:
instead of that you got 'cannot open initial console' or something
like that...

But after change tty_register_driver is invoked (through vty_init)
unconditionally from tty_io.c, where it depends only on CONFIG_VT.

So quick untested fix is

--- a/drivers/char/console.c Tue Jul 16 01:22:31 2002
+++ b/drivers/char/console.c Fri Jul 19 00:12:01 2002
@@ -2487,6 +2487,9 @@
 
 int __init vty_init(void)
 {
+ if (!conswitchp) {
+ return 0;
+ }
         memset(&console_driver, 0, sizeof(struct tty_driver));
         console_driver.magic = TTY_DRIVER_MAGIC;
         console_driver.name = "vc/%d";

But I'll leave final decision at James, maybe he want to support
VT without underlying console, and testing almost same condition
on two places looks suspicious to me. Either we need blank timer
and console, or do not. But registering one half in vty_init,
and second half in con_init?
                                            Best regards,
                                                Petr Vandrovec
                                                  
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Tue Jul 23 2002 - 22:00:28 EST