Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S269286AbUJFRmN (ORCPT ); Wed, 6 Oct 2004 13:42:13 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S269310AbUJFRmN (ORCPT ); Wed, 6 Oct 2004 13:42:13 -0400 Received: from mail.kroah.org ([69.55.234.183]:28338 "EHLO perch.kroah.org") by vger.kernel.org with ESMTP id S269286AbUJFRmL (ORCPT ); Wed, 6 Oct 2004 13:42:11 -0400 Date: Wed, 6 Oct 2004 10:38:23 -0700 From: Greg KH To: J?rn Engel Cc: Andrew Morton , linux-kernel@vger.kernel.org Subject: Re: [PATCH] Console: fall back to /dev/null when no console is availlable Message-ID: <20041006173823.GA26740@kroah.com> References: <20041005185214.GA3691@wohnheim.fh-wedel.de> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20041005185214.GA3691@wohnheim.fh-wedel.de> User-Agent: Mutt/1.5.6i Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1177 Lines: 29 On Tue, Oct 05, 2004 at 08:52:14PM +0200, J?rn Engel wrote: > --- linux-2.6.8cow/init/main.c~console 2004-10-05 20:46:40.000000000 +0200 > +++ linux-2.6.8cow/init/main.c 2004-10-05 20:46:08.000000000 +0200 > @@ -695,8 +695,11 @@ > system_state = SYSTEM_RUNNING; > numa_default_policy(); > > - if (sys_open((const char __user *) "/dev/console", O_RDWR, 0) < 0) > + if (sys_open((const char __user *) "/dev/console", O_RDWR, 0) < 0) { > printk("Warning: unable to open an initial console.\n"); > + if (open("/dev/null", O_RDWR, 0) == 0) > + printk(" Falling back to /dev/null.\n"); > + } Your printk() calls need the proper KERN_* level. And what happens if you can't open /dev/null? (hint, udev enabled boxes usually do not have a /dev/null this early in the boot process). Does this mean we should add a /dev/null to the initramfs image, like the /dev/console node we currently have there? thanks, greg k-h - 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/