From: Ingo Molnar <mingo@elte.hu>
To: Robin Getz <rgetz@blackfin.uclinux.org>
Cc: Andrew Morton <akpm@linux-foundation.org>,
linux-kernel@vger.kernel.org,
Linus Torvalds <torvalds@linux-foundation.org>
Subject: Re: Boot Consoles question...
Date: Fri, 10 Jul 2009 12:28:57 +0200 [thread overview]
Message-ID: <20090710102857.GF5499@elte.hu> (raw)
In-Reply-To: <200907071949.41311.rgetz@blackfin.uclinux.org>
* Robin Getz <rgetz@blackfin.uclinux.org> wrote:
> On Sat 4 Jul 2009 12:07, Robin Getz pondered:
> > On Sat 4 Jul 2009 06:29, Ingo Molnar pondered:
> > > Could be changed i guess ... but is it really an issue?
> >
> > It is just a change from "normal" (when the kernel has no boot console).
> >
> > > One artifact
> > > could be manual scroll-back - it would perhaps be nice indeed to
> > > allow the scrollback to the top of the bootlog.
> >
> > Exactly.
> >
> > One of my thoughts (was since CON_PRINTBUFFER isn't used after
> > register_console()) - was for the CON_BOOT's CON_PRINTBUFFER flag to control
> > the clearing of the CON_PRINTBUFFER for the real console or not...
> >
> > All early_printk consoles that I looked at have their CON_PRINTBUFFER set.
> >
> > Which means that something like should do the trick -- allow people who want
> > to override things to do so, and still have the today's setup work as is...
>
> I guess no one liked that idea?
No, this means no-one objected :)
> How about at least making sure that the real console gets a
> message that something is on the bootconsole? Right now the switch
> message:
>
> console handover:boot [early_shadow0] -> real [ttyBF0]
>
> only is printed on the bootconsole, not on the real console - so
> someone looking at the real console may not know there is anything
> on the boot console. They just think that things are missing...
Mind sending a full (changelogged, titled, etc.) patch for the other
bit as well? It kind of overlaps this one but both make sense,
especially if people end up objecting against the more intrusive one
and it gets dropped/reverted ;-)
Ingo
next prev parent reply other threads:[~2009-07-10 10:29 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-07-04 4:17 Boot Consoles question Robin Getz
2009-07-04 10:29 ` Ingo Molnar
2009-07-04 16:07 ` Robin Getz
2009-07-07 23:49 ` Robin Getz
2009-07-10 10:28 ` Ingo Molnar [this message]
2009-07-10 14:28 ` Robin Getz
2009-07-10 16:08 ` Ingo Molnar
2009-07-09 17:08 ` [PATCH] kernel.printk.c - ensure that "console enabled" messages are printed on the console Robin Getz
2009-07-10 10:25 ` Ingo Molnar
2009-07-10 14:13 ` [PATCH] kernel.printk.c - ensure that "console enabled"messages " Robin Getz
2009-07-10 14:16 ` Ingo Molnar
2009-07-10 14:26 ` Mike Frysinger
2009-07-18 0:48 ` [PATCH] kernel.printk.c - ensure that "console enabled" messages " Andrew Morton
2009-07-20 5:29 ` Stephen Rothwell
2009-07-10 10:43 ` [tip:core/printk] printk: Ensure " tip-bot for Robin Getz
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=20090710102857.GF5499@elte.hu \
--to=mingo@elte.hu \
--cc=akpm@linux-foundation.org \
--cc=linux-kernel@vger.kernel.org \
--cc=rgetz@blackfin.uclinux.org \
--cc=torvalds@linux-foundation.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox