From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965114AbcHJS4N (ORCPT ); Wed, 10 Aug 2016 14:56:13 -0400 Received: from mail.kernel.org ([198.145.29.136]:36872 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S964972AbcHJS4K (ORCPT ); Wed, 10 Aug 2016 14:56:10 -0400 From: Andy Lutomirski To: "H. Peter Anvin" , x86@kernel.org Cc: Mario Limonciello , Matthew Garrett , Borislav Petkov , Matt Fleming , linux-kernel@vger.kernel.org, Andy Lutomirski Subject: [PATCH v2 0/5] Allow the trampoline to use EFI boot services RAM Date: Wed, 10 Aug 2016 02:29:12 -0700 Message-Id: X-Mailer: git-send-email 2.7.4 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org As currently configured, my laptop cannot boot any existing kernel because the real mode trampoline can't be reserved. The ranges in which it could live are rejected by the kernel: one is EFI boot services data and the other is above the EBDA. Allowing use of RAM between the EBDA and 640k is scary: there are probably many quirky BIOSes out there, and, as currently structured, it would be awkward to allow it just on EFI boots because we currently reserve that range before we figure out whether we're using EFI. This series fixes it the other way: it allow the trampoline to live in boot services memory. It achieves this by deferring the panic due to failure to reserve a trampoline until early_initcall time and then adjusting the EFI boot services quirk to reserve space for the trampoline if we haven't already found it a home. I'm hoping this is okay for 4.8 even though it's late: it fixes a boot failure and it's fairly conservative -- the only significant changes in behavior should be on systems that currently fail to boot. I'm not currently proposing it for stable because AFAIK I'm the only person to have seen this issue. If it survives in Linus' tree for a while, though, I might propose it for -stable later on. Andy Lutomirski (4): x86/boot: Synchronize trampoline_cr4_features and mmu_cr4_features directly x86/boot: Defer setup_real_mode() to early_initcall time x86/boot: Rework reserve_real_mode() to allow multiple tries x86/efi: Allocate a trampoline if needed in efi_free_boot_services() Changes from v1: - Fix comment in the last patch (hpa) - Add missing first patch (I can't count...) - Rebase to v4.8-rc1, uneventfully Andy Lutomirski (5): x86/boot: Run reserve_bios_regions() after we initialize the memory map x86/boot: Synchronize trampoline_cr4_features and mmu_cr4_features directly x86/boot: Defer setup_real_mode() to early_initcall time x86/boot: Rework reserve_real_mode() to allow multiple tries x86/efi: Allocate a trampoline if needed in efi_free_boot_services() arch/x86/include/asm/realmode.h | 10 ++++++++- arch/x86/kernel/head32.c | 2 -- arch/x86/kernel/head64.c | 1 - arch/x86/kernel/setup.c | 19 ++++++++++------- arch/x86/platform/efi/quirks.c | 21 ++++++++++++++++++ arch/x86/realmode/init.c | 47 ++++++++++++++++++++++++++++++----------- 6 files changed, 76 insertions(+), 24 deletions(-) -- 2.7.4