dri-devel.lists.freedesktop.org archive mirror
 help / color / mirror / Atom feed
From: Hans de Goede <hdegoede@redhat.com>
To: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
Cc: Hans de Goede <hdegoede@redhat.com>,
	linux-fbdev@vger.kernel.org, dri-devel@lists.freedesktop.org
Subject: [PATCH 0/1] console/fbcon: Add support for deferred console takeover
Date: Mon, 28 May 2018 15:30:41 +0200	[thread overview]
Message-ID: <20180528133042.29675-1-hdegoede@redhat.com> (raw)

Hi All,

Allthough this patch has turned out pleasantly small / simpel, I guess
it still might be somewhat controversial. So let me explain why this
patch is necessary.

Various (desktop oriented) Linux distributions have spend a lot of time
to not show way too technial boot messages to end users during bootup.
What we would really like for the boot experience is something like
MacOS X / Windows 10 do. The (EFI) firmware boots up a logo and we
leave that in place until the login-manager (e.g. gdm) starts and then
the login-manager takes over the framebuffer including the current logo
contents and fades that into the login screen.

I've already written and upstreamed a patch for shim to not switch the
EFI framebuffer to text-mode, unless it has some error message to display
this is upstream now:
https://github.com/rhboot/shim/commit/1ff4a36a23ac5c17144275ccb3e1e1061750a137

I've written a similar patch for grub2 to not switch to text-mode when
using timeout_style=hidden for the menu and no key to show the menu is
pressed. This has been posted upstream and I need to do a v2 addressing
some small remarks.

This brings us to the next piece of software taking over control of
the framebuffer, the kernel.

I already have a patch in -next to fix the i915 driver not taking over
the BIOS used framebuffer (and thus loosing the vendor logo) on newer
hardware:
https://cgit.freedesktop.org/drm-intel/commit?id=011f22eb545a35f972036bb6a245c95c2e7e15a0

But this is only useful if we don't get fbcon binding to any fbdev
devices showing up and replacing there contents with a black screen
right away.

That is where this patch comes in it defers fbcon binding to any registered
fbdevs, until some output, any output at all, is send to the console (tty0),
this way error messages will still get shown, but a well behaved userspace
which does not spam the console with messages will retain the vendor logo
from the EFI firmware all the way to the login screen.

Regards,

Hans


_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

             reply	other threads:[~2018-05-28 13:30 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-05-28 13:30 Hans de Goede [this message]
2018-05-28 13:30 ` [PATCH] console/fbcon: Add support for deferred console takeover Hans de Goede
2018-05-28 14:00 ` [PATCH 0/1] " Jani Nikula

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=20180528133042.29675-1-hdegoede@redhat.com \
    --to=hdegoede@redhat.com \
    --cc=b.zolnierkie@samsung.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=linux-fbdev@vger.kernel.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;
as well as URLs for NNTP newsgroup(s).