From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752896AbdK2WbR (ORCPT ); Wed, 29 Nov 2017 17:31:17 -0500 Received: from mx2.suse.de ([195.135.220.15]:41907 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751748AbdK2WbN (ORCPT ); Wed, 29 Nov 2017 17:31:13 -0500 Date: Wed, 29 Nov 2017 23:31:04 +0100 From: Borislav Petkov To: "H. Peter Anvin" Cc: "Kirill A. Shutemov" , Thomas Gleixner , "Kirill A. Shutemov" , Ingo Molnar , x86@kernel.org, Linus Torvalds , Andy Lutomirski , Cyrill Gorcunov , Andi Kleen , linux-mm@kvack.org, linux-kernel@vger.kernel.org Subject: Re: [PATCHv2 0/4] x86: 5-level related changes into decompression code Message-ID: <20171129223103.in4qmtxbj2sawhpw@pd.tnic> References: <20171110220645.59944-1-kirill.shutemov@linux.intel.com> <20171129154908.6y4st6xc7hbsey2v@pd.tnic> <20171129161349.d7ksuhwhdamloty6@node.shutemov.name> <20171129170831.2iqpop2u534mgrbc@node.shutemov.name> <20171129174851.jk2ai37uumxve6sg@pd.tnic> <793b9c55-e85b-97b5-c857-dd8edcda4081@zytor.com> <20171129191902.2iamm3m23e3gwnj4@pd.tnic> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: NeoMutt/20170609 (1.8.3) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Nov 29, 2017 at 01:33:28PM -0800, H. Peter Anvin wrote: > You can't dump a message about *anything* if the bootloader bypasses the > checks that happen before we leave the firmware behind. This is what > this is about. For BIOS or EFI boot that go through the proper stub > functions we will print a message just fine, as we already validate the > "required features" structure (although please do verify that the > relevant words are indeed being checked.) A couple of points: * so this box here has a normal grub installation and apparently grub jumps to some other entry point. * I'm not convinced we need to do everything you typed because this is only a temporary issue and once X86_5LEVEL is complete, it should work. I mean, it needs to work otherwise forget single-system image and I don't think we want to give that up. > However, if the bootloader jumps straight into the code what do you > expect it to do? We have no real concept about what we'd need to do to > issue a message as we really don't know what devices are available on > the system, etc. If the screen_info field in struct boot_params has > been initialized then we actually *do* know how to write to the screen > -- if you are okay with including a text font etc. since modern systems > boot in graphics mode. We switch to text mode and dump our message. Can we do that? I wouldn't want to do any of this back'n'forth between kernel and boot loader because that sounds fragile, at least to me. And again, I'm not convinced we should spend too much energy on this as the issue is temporary AFAICT. -- Regards/Gruss, Boris. SUSE Linux GmbH, GF: Felix Imendörffer, Jane Smithard, Graham Norton, HRB 21284 (AG Nürnberg) --