From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jani Nikula Date: Mon, 28 May 2018 13:56:46 +0000 Subject: Re: [PATCH 0/1] console/fbcon: Add support for deferred console takeover Message-Id: <87fu2bj2wv.fsf@intel.com> List-Id: References: <20180528133042.29675-1-hdegoede@redhat.com> In-Reply-To: <20180528133042.29675-1-hdegoede@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Bartlomiej Zolnierkiewicz Cc: Hans de Goede , linux-fbdev@vger.kernel.org, dri-devel@lists.freedesktop.org On Mon, 28 May 2018, Hans de Goede wrote: > 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?id1f22eb545a35f972036bb6a245c95c2e7e15a0 > > 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. FWIW I think the main idea here is sound: just preserve whatever image the firmware decided to display until something noteworthy happens or gdm takes over. (And you could perhaps override the image in grub if you like, and preserve that until gdm instead.) BR, Jani. -- Jani Nikula, Intel Open Source Graphics Center From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jani Nikula Subject: Re: [PATCH 0/1] console/fbcon: Add support for deferred console takeover Date: Mon, 28 May 2018 17:00:32 +0300 Message-ID: <87fu2bj2wv.fsf@intel.com> References: <20180528133042.29675-1-hdegoede@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by gabe.freedesktop.org (Postfix) with ESMTPS id 54C6F6E2A0 for ; Mon, 28 May 2018 13:56:37 +0000 (UTC) In-Reply-To: <20180528133042.29675-1-hdegoede@redhat.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Bartlomiej Zolnierkiewicz Cc: Hans de Goede , linux-fbdev@vger.kernel.org, dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org T24gTW9uLCAyOCBNYXkgMjAxOCwgSGFucyBkZSBHb2VkZSA8aGRlZ29lZGVAcmVkaGF0LmNvbT4g d3JvdGU6Cj4gVmFyaW91cyAoZGVza3RvcCBvcmllbnRlZCkgTGludXggZGlzdHJpYnV0aW9ucyBo YXZlIHNwZW5kIGEgbG90IG9mIHRpbWUKPiB0byBub3Qgc2hvdyB3YXkgdG9vIHRlY2huaWFsIGJv b3QgbWVzc2FnZXMgdG8gZW5kIHVzZXJzIGR1cmluZyBib290dXAuCj4gV2hhdCB3ZSB3b3VsZCBy ZWFsbHkgbGlrZSBmb3IgdGhlIGJvb3QgZXhwZXJpZW5jZSBpcyBzb21ldGhpbmcgbGlrZQo+IE1h Y09TIFggLyBXaW5kb3dzIDEwIGRvLiBUaGUgKEVGSSkgZmlybXdhcmUgYm9vdHMgdXAgYSBsb2dv IGFuZCB3ZQo+IGxlYXZlIHRoYXQgaW4gcGxhY2UgdW50aWwgdGhlIGxvZ2luLW1hbmFnZXIgKGUu Zy4gZ2RtKSBzdGFydHMgYW5kIHRoZW4KPiB0aGUgbG9naW4tbWFuYWdlciB0YWtlcyBvdmVyIHRo ZSBmcmFtZWJ1ZmZlciBpbmNsdWRpbmcgdGhlIGN1cnJlbnQgbG9nbwo+IGNvbnRlbnRzIGFuZCBm YWRlcyB0aGF0IGludG8gdGhlIGxvZ2luIHNjcmVlbi4KPgo+IEkndmUgYWxyZWFkeSB3cml0dGVu IGFuZCB1cHN0cmVhbWVkIGEgcGF0Y2ggZm9yIHNoaW0gdG8gbm90IHN3aXRjaCB0aGUKPiBFRkkg ZnJhbWVidWZmZXIgdG8gdGV4dC1tb2RlLCB1bmxlc3MgaXQgaGFzIHNvbWUgZXJyb3IgbWVzc2Fn ZSB0byBkaXNwbGF5Cj4gdGhpcyBpcyB1cHN0cmVhbSBub3c6Cj4gaHR0cHM6Ly9naXRodWIuY29t L3JoYm9vdC9zaGltL2NvbW1pdC8xZmY0YTM2YTIzYWM1YzE3MTQ0Mjc1Y2NiM2UxZTEwNjE3NTBh MTM3Cj4KPiBJJ3ZlIHdyaXR0ZW4gYSBzaW1pbGFyIHBhdGNoIGZvciBncnViMiB0byBub3Qgc3dp dGNoIHRvIHRleHQtbW9kZSB3aGVuCj4gdXNpbmcgdGltZW91dF9zdHlsZT1oaWRkZW4gZm9yIHRo ZSBtZW51IGFuZCBubyBrZXkgdG8gc2hvdyB0aGUgbWVudSBpcwo+IHByZXNzZWQuIFRoaXMgaGFz IGJlZW4gcG9zdGVkIHVwc3RyZWFtIGFuZCBJIG5lZWQgdG8gZG8gYSB2MiBhZGRyZXNzaW5nCj4g c29tZSBzbWFsbCByZW1hcmtzLgo+Cj4gVGhpcyBicmluZ3MgdXMgdG8gdGhlIG5leHQgcGllY2Ug b2Ygc29mdHdhcmUgdGFraW5nIG92ZXIgY29udHJvbCBvZgo+IHRoZSBmcmFtZWJ1ZmZlciwgdGhl IGtlcm5lbC4KPgo+IEkgYWxyZWFkeSBoYXZlIGEgcGF0Y2ggaW4gLW5leHQgdG8gZml4IHRoZSBp OTE1IGRyaXZlciBub3QgdGFraW5nIG92ZXIKPiB0aGUgQklPUyB1c2VkIGZyYW1lYnVmZmVyIChh bmQgdGh1cyBsb29zaW5nIHRoZSB2ZW5kb3IgbG9nbykgb24gbmV3ZXIKPiBoYXJkd2FyZToKPiBo dHRwczovL2NnaXQuZnJlZWRlc2t0b3Aub3JnL2RybS1pbnRlbC9jb21taXQ/aWQ9MDExZjIyZWI1 NDVhMzVmOTcyMDM2YmI2YTI0NWM5NWMyZTdlMTVhMAo+Cj4gQnV0IHRoaXMgaXMgb25seSB1c2Vm dWwgaWYgd2UgZG9uJ3QgZ2V0IGZiY29uIGJpbmRpbmcgdG8gYW55IGZiZGV2Cj4gZGV2aWNlcyBz aG93aW5nIHVwIGFuZCByZXBsYWNpbmcgdGhlcmUgY29udGVudHMgd2l0aCBhIGJsYWNrIHNjcmVl bgo+IHJpZ2h0IGF3YXkuCj4KPiBUaGF0IGlzIHdoZXJlIHRoaXMgcGF0Y2ggY29tZXMgaW4gaXQg ZGVmZXJzIGZiY29uIGJpbmRpbmcgdG8gYW55IHJlZ2lzdGVyZWQKPiBmYmRldnMsIHVudGlsIHNv bWUgb3V0cHV0LCBhbnkgb3V0cHV0IGF0IGFsbCwgaXMgc2VuZCB0byB0aGUgY29uc29sZSAodHR5 MCksCj4gdGhpcyB3YXkgZXJyb3IgbWVzc2FnZXMgd2lsbCBzdGlsbCBnZXQgc2hvd24sIGJ1dCBh IHdlbGwgYmVoYXZlZCB1c2Vyc3BhY2UKPiB3aGljaCBkb2VzIG5vdCBzcGFtIHRoZSBjb25zb2xl IHdpdGggbWVzc2FnZXMgd2lsbCByZXRhaW4gdGhlIHZlbmRvciBsb2dvCj4gZnJvbSB0aGUgRUZJ IGZpcm13YXJlIGFsbCB0aGUgd2F5IHRvIHRoZSBsb2dpbiBzY3JlZW4uCgpGV0lXIEkgdGhpbmsg dGhlIG1haW4gaWRlYSBoZXJlIGlzIHNvdW5kOiBqdXN0IHByZXNlcnZlIHdoYXRldmVyIGltYWdl CnRoZSBmaXJtd2FyZSBkZWNpZGVkIHRvIGRpc3BsYXkgdW50aWwgc29tZXRoaW5nIG5vdGV3b3J0 aHkgaGFwcGVucyBvcgpnZG0gdGFrZXMgb3Zlci4gKEFuZCB5b3UgY291bGQgcGVyaGFwcyBvdmVy cmlkZSB0aGUgaW1hZ2UgaW4gZ3J1YiBpZiB5b3UKbGlrZSwgYW5kIHByZXNlcnZlIHRoYXQgdW50 aWwgZ2RtIGluc3RlYWQuKQoKQlIsCkphbmkuCgotLSAKSmFuaSBOaWt1bGEsIEludGVsIE9wZW4g U291cmNlIEdyYXBoaWNzIENlbnRlcgpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fXwpkcmktZGV2ZWwgbWFpbGluZyBsaXN0CmRyaS1kZXZlbEBsaXN0cy5mcmVl ZGVza3RvcC5vcmcKaHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5m by9kcmktZGV2ZWwK