Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754658AbdLFDY5 (ORCPT ); Tue, 5 Dec 2017 22:24:57 -0500 Received: from smtp.nue.novell.com ([195.135.221.5]:60347 "EHLO smtp.nue.novell.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754406AbdLFDYx (ORCPT ); Tue, 5 Dec 2017 22:24:53 -0500 Date: Wed, 6 Dec 2017 11:24:37 +0800 From: Gary Lin To: Josh Boyer Cc: x86 , "Linux-Kernel@Vger. Kernel. Org" , "linux-efi@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "H. Peter Anvin" , Thomas Gleixner , Ard Biesheuvel , Ingo Molnar , Matt Fleming , Catalin Marinas , Will Deacon , Joey Lee Subject: Re: [RFC v3 PATCH 0/2] Introduce Security Version to EFI Stub Message-ID: <20171206032437.pifxxcewgozlghfs@GaryWorkstation> References: <20171205100148.5757-1-glin@suse.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: NeoMutt/20170912 (1.9.0) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4408 Lines: 97 On Tue, Dec 05, 2017 at 04:14:26PM -0500, Josh Boyer wrote: > On Tue, Dec 5, 2017 at 5:01 AM, Gary Lin wrote: > > The series of patches introduce Security Version to EFI stub. > > > > Security Version is a monotonically increasing number and designed to > > prevent the user from loading an insecure kernel accidentally. The > > bootloader maintains a list of security versions corresponding to > > different distributions. After fixing a critical vulnerability, the > > distribution kernel maintainer bumps the "version", and the bootloader > > updates the list automatically. When the user tries to load a kernel > > with a lower security version, the bootloader shows a warning prompt > > to notify the user the potential risk. > > If a distribution releases a kernel with a higher security version and > that it automatically updated on boot, what happens if that kernel > contains a different bug that causes it to fail to boot or break > critical functionality? At that point, the user's machine would be in > a state where the higher security version is enforced but the only > kernel that provides that is broken. Wouldn't that make a bad > situation even worse by now requiring manual acceptance of the older > SV kernel boot physically at the machine? > > I feel like I'm missing a detail here or something. > If the new kernel fails to boot, then the user has to choose the kernel manually anyway, and there will be an option in the warning prompt to lower SV. Since Security Version is meant to be a warning, not a hard block, we can introduce a timeout to the warning prompt if necessary. The system still boots, but with a longer boot time. Gary Lin > josh > > > For more details: https://github.com/lcp/shim/wiki/Security-Version > > > > The original idea is to add a new PE/COFF section to store the data. > > However, there are some restrictions. > > > > 1. For x86, the size limit of the EFI header is 0x200, and a section entry > > in the section table takes 40 bytes. Currently, the EFI header already > > occupies the first 0x1da bytes, so there is no room for a new section > > entry. > > > > 2. The MemoryAttributes table sets the attributes of memory pages according > > to the section flags. For ARM64, the minimal granularity is 4KB, but > > Security Version only needs a few bytes, and it's pointless to allocate > > 4KB for it. > > > > Fortunately, there is a special section defined in PE/COFF: resource > > section. The only known user of the resource section in UEFI is the HII > > protocol which fetches data from "HII" directory. For Security Version, a > > new directory called "LinuxSV" is created and it contains the file offset > > to the struct of Security Version. The bootloader just follows the > > resource table to fetch the "version" from the image file. > > > > v3: > > - Move everything to the resource section to be compatible with both > > x86 and ARM64 > > v2: > > - Decrease the size of secdata_offset to 2 bytes since the setup header > > is limited to around 32KB. > > - Restructure the secdata section. The signer is now a null-terminated > > string. The type of distro_version changes to u32 in case the distro > > uses a long version. > > - Modify the Kconfig names and add help. > > - Remove the signer name hack in build.c. > > > > Cc: "H. Peter Anvin" > > Cc: Thomas Gleixner > > Cc: Ard Biesheuvel > > Cc: Ingo Molnar > > Cc: Matt Fleming > > Cc: Catalin Marinas > > Cc: Will Deacon > > Cc: Joey Lee > > Signed-off-by: Gary Lin > > > > Gary Lin (2): > > x86/efi: Introduce Security Version to x86 > > arm64/efi: Introduce Security Version to ARM64 > > > > arch/arm64/kernel/efi-header.S | 57 ++++++++++++++++++++++++++++++++++++++++++ > > arch/x86/boot/header.S | 55 ++++++++++++++++++++++++++++++++++++++++ > > drivers/firmware/efi/Kconfig | 40 +++++++++++++++++++++++++++++ > > 3 files changed, 152 insertions(+) > > > > -- > > 2.15.0 > > > > -- > > To unsubscribe from this list: send the line "unsubscribe linux-efi" in > > the body of a message to majordomo@vger.kernel.org > > More majordomo info at http://vger.kernel.org/majordomo-info.html >