All of lore.kernel.org
 help / color / mirror / Atom feed
From: Blaisorblade <blaisorblade@yahoo.it>
To: user-mode-linux-devel@lists.sourceforge.net
Cc: Jeff Dike <jdike@addtoit.com>
Subject: Re: [uml-devel] [patch 094/241] uml: fix error output during early boot
Date: Sat, 10 Feb 2007 13:31:20 +0100	[thread overview]
Message-ID: <200702101331.20261.blaisorblade@yahoo.it> (raw)
In-Reply-To: <200702100944.l1A9iSEp008813@shell0.pdx.osdl.net>

On Saturday 10 February 2007 10:44, akpm@linux-foundation.org wrote:
> From: Jeff Dike <jdike@addtoit.com>
>
> The startup code panics a lot if anything goes wrong early on.  This is
> wrong for several reasons, like the kernel isn't running, so you can't
> really be calling into it yet, but the harm comes from useful error
> messages being trapped in the printk ring where no one will ever see them.
>
> This patch changes these panics to perror and printf in wrappers which also
> exit.  Normal, informational, prints are also wrapped so that
> fflush(stdout) is called after each one.  This is so the output appears in
> the correct sequence in the event of an error.

libc's printf has an enormous stack usage, so that my_printf is used for 
direct output instead of printf; if this is ok because it's bootup code, then 
*fatal* should be marked as "bootup code only"; otherwise we may reuse 
vsprintf() and write() or fputs(); this should match what happens in 
my_printf, to avoid any unlikely problem with libc's buffering.
-- 
Inform me of my mistakes, so I can add them to my list!
Paolo Giarrusso, aka Blaisorblade
http://www.user-mode-linux.org/~blaisorblade
Chiacchiera con i tuoi amici in tempo reale! 
 http://it.yahoo.com/mail_it/foot/*http://it.messenger.yahoo.com 


-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier.
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
User-mode-linux-devel mailing list
User-mode-linux-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel

       reply	other threads:[~2007-02-10 12:31 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <200702100944.l1A9iSEp008813@shell0.pdx.osdl.net>
2007-02-10 12:31 ` Blaisorblade [this message]
2007-02-10 22:21   ` [uml-devel] [patch 094/241] uml: fix error output during early boot Jeff Dike
2007-02-12 18:59     ` Blaisorblade

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=200702101331.20261.blaisorblade@yahoo.it \
    --to=blaisorblade@yahoo.it \
    --cc=jdike@addtoit.com \
    --cc=user-mode-linux-devel@lists.sourceforge.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.