linux-serial.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Arnd Bergmann <arnd@arndb.de>
To: Peter Hurley <peter@hurleysoftware.com>
Cc: Masahiro Yamada <yamada.masahiro@socionext.com>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	Stefan Agner <stefan@agner.ch>,
	linux-serial@vger.kernel.org,
	linux-arm-kernel <linux-arm-kernel@lists.infradead.org>
Subject: Re: console vs earlycon ?
Date: Wed, 21 Oct 2015 21:00:24 +0200	[thread overview]
Message-ID: <3890353.0icCv0Zeuq@wuerfel> (raw)
In-Reply-To: <5627AFFF.6060900@hurleysoftware.com>

On Wednesday 21 October 2015 11:32:15 Peter Hurley wrote:
> On 10/21/2015 10:13 AM, Arnd Bergmann wrote:
> > On Wednesday 21 October 2015 09:53:47 Peter Hurley wrote:
> > 
> > Ah, I had no idea we were moving in this direction.
> 
> I would not say this was a conscious design decision, but rather an
> outcome of getting-something-working-without-breaking-existing-usage.
> 
> My main focus with earlycon/console has been to try to reduce and
> generalize the existing code.

Ok.

> >> I'm assuming the issue with trying to get console_init() working
> >> is because the dummy color console causes the earlycon to be disabled?
> > 
> > I don't think so.
> > 
> > My line of thinking was more about usability: earlycon requires that
> > you edit the kernel command line at the moment, while console_init()
> > doesn't require any user interaction and just uses the stdout-path.
> > 
> > I guess we could enable earlycon using a Kconfig symbol if we want
> > to, or make it a per-architecture decision whether it's enabled even
> > in the absence of the command line flag.
> 
> Ah, I see. You want to start the stdout-path console at console_init()
> time.

Yes, I see this as a tradeoff: we want the console to be as early as
possible in order to report boot-time errors to the user, but not so early
to require hacks that can cause problems themselves. earlycon has to
do some scary stuff and is relatively recent, while the hacks
necessary for console_init() are much older and better tested.

	Arnd

  reply	other threads:[~2015-10-21 19:00 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-10-21  8:21 console vs earlycon ? Masahiro Yamada
2015-10-21  8:57 ` Arnd Bergmann
2015-10-21  9:09   ` Masahiro Yamada
2015-10-21  9:19     ` Arnd Bergmann
2015-10-21  9:30       ` Masahiro Yamada
2015-10-21 10:36         ` Arnd Bergmann
2015-10-21 13:53           ` Peter Hurley
2015-10-21 14:13             ` Arnd Bergmann
2015-10-21 15:32               ` Peter Hurley
2015-10-21 19:00                 ` Arnd Bergmann [this message]
2015-10-21 19:24                   ` Peter Hurley
2015-10-21 22:54                     ` Arnd Bergmann
2015-10-21 13:20 ` Peter Hurley
2015-10-22  4:47   ` Masahiro Yamada

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=3890353.0icCv0Zeuq@wuerfel \
    --to=arnd@arndb.de \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-serial@vger.kernel.org \
    --cc=peter@hurleysoftware.com \
    --cc=stefan@agner.ch \
    --cc=yamada.masahiro@socionext.com \
    /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;
as well as URLs for NNTP newsgroup(s).