Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752274AbcKQV6L (ORCPT ); Thu, 17 Nov 2016 16:58:11 -0500 Received: from mail-it0-f47.google.com ([209.85.214.47]:35798 "EHLO mail-it0-f47.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751532AbcKQV6J (ORCPT ); Thu, 17 Nov 2016 16:58:09 -0500 MIME-Version: 1.0 In-Reply-To: <147933287290.19316.3360403691390019935.stgit@warthog.procyon.org.uk> References: <147933283664.19316.12454053022687659937.stgit@warthog.procyon.org.uk> <147933287290.19316.3360403691390019935.stgit@warthog.procyon.org.uk> From: Ard Biesheuvel Date: Thu, 17 Nov 2016 21:58:07 +0000 Message-ID: Subject: Re: [PATCH 05/16] efi: Add EFI_SECURE_BOOT bit To: David Howells Cc: keyrings@vger.kernel.org, matthew.garrett@nebula.com, "linux-efi@vger.kernel.org" , "linux-kernel@vger.kernel.org" , linux-security-module@vger.kernel.org, Josh Boyer Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1868 Lines: 47 On 16 November 2016 at 21:47, David Howells wrote: > From: Josh Boyer > > UEFI machines can be booted in Secure Boot mode. Add a EFI_SECURE_BOOT bit > for use with efi_enabled. > > Signed-off-by: Josh Boyer > Signed-off-by: David Howells > --- > > arch/x86/kernel/setup.c | 1 + > include/linux/efi.h | 1 + > 2 files changed, 2 insertions(+) > > diff --git a/arch/x86/kernel/setup.c b/arch/x86/kernel/setup.c > index 9521acce8378..539f29587712 100644 > --- a/arch/x86/kernel/setup.c > +++ b/arch/x86/kernel/setup.c > @@ -1164,6 +1164,7 @@ void __init setup_arch(char **cmdline_p) > if (boot_params.secure_boot && > IS_ENABLED(CONFIG_EFI_SECURE_BOOT_LOCK_DOWN)) { > lock_kernel_down(); > + set_bit(EFI_SECURE_BOOT, &efi.flags); Why is this x86 only? And why is this bit only set if CONFIG_EFI_SECURE_BOOT_LOCK_DOWN is enabled? > pr_info("Secure boot enabled\n"); > } > > diff --git a/include/linux/efi.h b/include/linux/efi.h > index 11372fb8784c..5d7fb3e3400b 100644 > --- a/include/linux/efi.h > +++ b/include/linux/efi.h > @@ -1079,6 +1079,7 @@ extern int __init efi_setup_pcdp_console(char *); > #define EFI_ARCH_1 7 /* First arch-specific bit */ > #define EFI_DBG 8 /* Print additional debug info at runtime */ > #define EFI_NX_PE_DATA 9 /* Can runtime data regions be mapped non-executable? */ > +#define EFI_SECURE_BOOT 10 /* Are we in Secure Boot mode? */ > > #ifdef CONFIG_EFI > /* > > -- > 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