From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) by mx.groups.io with SMTP id smtpd.web12.10439.1600328607911466575 for ; Thu, 17 Sep 2020 00:43:28 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@intel.onmicrosoft.com header.s=selector2-intel-onmicrosoft-com header.b=hLCVn7+O; spf=pass (domain: intel.com, ip: 134.134.136.100, mailfrom: divneil.r.wadhawan@intel.com) IronPort-SDR: b4i8j0ljrKgEASay7ix0KAUGYeM0Rfplrg/k65J1h3SWPTAMoTsgXpfpA6qTGcSfmkMIsNFewN XY1PRYCSjnIw== X-IronPort-AV: E=McAfee;i="6000,8403,9746"; a="223826878" X-IronPort-AV: E=Sophos;i="5.76,436,1592895600"; d="scan'208,217";a="223826878" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga005.fm.intel.com ([10.253.24.32]) by orsmga105.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 17 Sep 2020 00:43:27 -0700 IronPort-SDR: GdLw6inqeeO0ErW1i/aJIPrgMlU/n/RCcILW1QCcQZe4ETlOMF2jPKutO4EYwAOnTxsFdvTvnq +BSvNFi2qETQ== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.76,436,1592895600"; d="scan'208,217";a="508306940" Received: from fmsmsx605.amr.corp.intel.com ([10.18.126.85]) by fmsmga005.fm.intel.com with ESMTP; 17 Sep 2020 00:43:26 -0700 Received: from fmsmsx603.amr.corp.intel.com (10.18.126.83) by fmsmsx605.amr.corp.intel.com (10.18.126.85) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.1713.5; Thu, 17 Sep 2020 00:43:26 -0700 Received: from fmsedg601.ED.cps.intel.com (10.1.192.135) by fmsmsx603.amr.corp.intel.com (10.18.126.83) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.1713.5 via Frontend Transport; Thu, 17 Sep 2020 00:43:26 -0700 Received: from NAM10-BN7-obe.outbound.protection.outlook.com (104.47.70.104) by edgegateway.intel.com (192.55.55.70) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.1713.5; Thu, 17 Sep 2020 00:43:18 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=T79WP76baeyoHLTPTi6VyKU0bmwelsAzMV00wioDMiqWIYy7lCVYJ9ytE2up55HvZ6kWKh7L1PAggt1XzaCjHzERmqqjwiq6heYDm8lxm7eMviEajXX9BLgSpQpTeGwz5psmpI0+o+N8FKXzpqF/lmbGiVhIKyNuqx5zTlqcqjGUEbiSQ9I2XyEOVCxgduySSgUs2ARVI7Q07Xxsff8RXO6iSEBkHuOo2U/uHnlfO5yLM8IL0ivYOLfuX8zs9hI1NmPlEgfzuqOdqcVFcLpUNxClhPHZyi6IzT3jGUSiri+CpeouH5aWs3uHrkmPp61/MHIvDoJjubww/ATu1zAIkg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=si5FiJ4JwoM7ZFHVAZnJVaJTJjNMZdKvZs4aRjLMprg=; b=FTxvbeOdo3dU01RPjL9YAesApbvlDLe82lnaXPqS+YYcprvsU7q7PBamW/Fna2HdELYAKSHvx/xrZTnHyWToHrwOIQr04JFXbQgOuJwdiYgZ+O8VpRUElLho+5WyGPu370qjXEzh6epLAyJ27M/TKgP37LvaP+9mZpPdidPVpAWFZTX/QGoReicwiBgTaErbxsptlrJn8UVYNfTprTyxt5WX30weN7zKHrp3PtCaRf61xf+2BOUUJRiM4dUhSX8bs9z0rkTj/DOaJciMd95M4RNQ0y9NaY0d84T5XMrs3Z2+TMuyo1eObuzI0ml9s1z5xEqcFNbXMSobPAEwRFnKuA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=intel.com; dmarc=pass action=none header.from=intel.com; dkim=pass header.d=intel.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=intel.onmicrosoft.com; s=selector2-intel-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=si5FiJ4JwoM7ZFHVAZnJVaJTJjNMZdKvZs4aRjLMprg=; b=hLCVn7+OwIJ22fmL4T02nTV6t1dgK/6OiRoFWWBfwKlLV5PloPJ0oOHQ0bKvFlYHWBMwo23TGmv+Phtla1K3sORSX2GfMVHp+cC3RP6ny83NMlpL+uvCjDE/kYvhZB5MdbPeBvRYh4lJwVc6Z8BE22K8eyIc3QBlXUDjMRYmgbM= Received: from DM6PR11MB4315.namprd11.prod.outlook.com (2603:10b6:5:201::28) by DM6PR11MB3436.namprd11.prod.outlook.com (2603:10b6:5:65::22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3370.16; Thu, 17 Sep 2020 07:43:04 +0000 Received: from DM6PR11MB4315.namprd11.prod.outlook.com ([fe80::4c5c:c6d0:dfd3:1e45]) by DM6PR11MB4315.namprd11.prod.outlook.com ([fe80::4c5c:c6d0:dfd3:1e45%4]) with mapi id 15.20.3370.019; Thu, 17 Sep 2020 07:43:04 +0000 From: "Wadhawan, Divneil R" To: "Ni, Ray" , "devel@edk2.groups.io" CC: gaoliming , 'Andrew Fish' , "Justen, Jordan L" , "Kinney, Michael D" , "Wadhawan, Divneil R" Subject: Re: [edk2-devel] [PATCH v2] EmulatorPkg: Enable support for Secure Boot Thread-Topic: [edk2-devel] [PATCH v2] EmulatorPkg: Enable support for Secure Boot Thread-Index: AdaMQS9sqEhJ/RVOQYWR22UtRyYHlAAgXCCgAABlWnA= Date: Thu, 17 Sep 2020 07:43:04 +0000 Message-ID: References: In-Reply-To: Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-product: dlpe-windows dlp-reaction: no-action dlp-version: 11.5.1.3 authentication-results: intel.com; dkim=none (message not signed) header.d=none;intel.com; dmarc=none action=none header.from=intel.com; x-originating-ip: [106.200.250.114] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 53f37ada-9586-42cd-6073-08d85add4fc8 x-ms-traffictypediagnostic: DM6PR11MB3436: x-ld-processed: 46c98d88-e344-4ed4-8496-4ed7712e255d,ExtAddr x-ms-exchange-transport-forked: True x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:8882; x-ms-exchange-senderadcheck: 1 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: 7JsLc7O4Gp8Roj4g2MfXCdOgWEBGeIVBbGVMwUKY2CG4KSqXWa0huwFhRXH+ibGx0weSx4vHPBFaWvKsDv2AWxTbaZfZd2OqO4fV6uMArNsl2QY4TggfG4agu6HUx6l5P5ff/3Nv9nFlJOzjNG+bpyUVwat+Akjc6jkzuuoTo8BLz5Cj9Ovh8VyY2w231+p215wrhvzQ8cez2GKfLfpt171uw6sKcwr+PqtKn7WekcZKXEjqI6rC8+bF25T5rksh6vnGRA0LeLPRfuXbKCG3DMGWGryn3G5QuqBOqNXMHkFo15juuxl3BnQp0ArSauLbsL4lVjwmCSOMua2bZ63AEe0FeRnBq9PLzQ+UNnHYp8HyxjT9FFeCFmiXuFIVd/eMWQyPn7w2AaBabenGM8VnGA== x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:DM6PR11MB4315.namprd11.prod.outlook.com;PTR:;CAT:NONE;SFS:(4636009)(136003)(346002)(376002)(396003)(366004)(39860400002)(33656002)(26005)(110136005)(107886003)(64756008)(55016002)(71200400001)(53546011)(7696005)(52536014)(2906002)(9326002)(66476007)(66556008)(83380400001)(66946007)(8676002)(54906003)(66446008)(8936002)(6506007)(186003)(316002)(9686003)(86362001)(76116006)(166002)(966005)(5660300002)(4326008)(478600001);DIR:OUT;SFP:1102; x-ms-exchange-antispam-messagedata: cGwaSB9ZPiuk4SyDr7k9khB8UHrZtnaG8tkQ3fCzP9mLycdQjqN+WxvHncAMEpEvORJNuLOsb7rUZwjW7z2znqFY2YkhahIVjdzV80bbPOr1DFYBpWVGnNJRpghX3+uWbcKPX/TQVxp7VYFgVdukcDyyJErySj1QNtNvQ3EKUJhCdwnuA/3t4/nQm/BchYg73dSuKcWmjsy3EC7rbyXygRsvXOGHqjnb+ZXc7B+LG6bnLv9hFpcfmDVBxDajAnIMCv70HFn+0rWEU3FAHbQ6ks2mMY6g3T9YQyXdYjrg+4ARiON36gn/V6fM7lvO5BriNrEoWvI3AjhFt6AtHveFap3h05qczRC3nW9JVgyhX/y+EDwNCtBOYB/Iiome+HXa1bVOF+zX3qhMx3bEF6gepSlH32MExkR8dOZikrqvpRR6jZ7xMlE99RaSjezmZxyDImFRtesIn+FHCluCPJERQKciVFo9aDutlc29gacdAn9PexAWvPT2/4PDmN7vPYkyFGcogmLvC0TCsT0hhDQ7O5ZcED9Q5jAPFeBN9i4JLtWSICLA2zwYvFb/tw+G5DCZ5+EFLCmg/7AHdYoQXzBLfICC2rAK1HqsOvEPXukGbYsTnWvdKdbBvPGTvzX7AviZz8gOczelw9OdHstY+FyVtg== MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: DM6PR11MB4315.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 53f37ada-9586-42cd-6073-08d85add4fc8 X-MS-Exchange-CrossTenant-originalarrivaltime: 17 Sep 2020 07:43:04.5119 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: /AEDkf4SPgsuNXJ/I2q3RkykOQRUfW4EvSPo8KfjH5dmMKF4MJyf8CBo90vXKHvnINxceNCTPKdFM/b8Twngdl67rQjgGquac0IeQyYoqJI= X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR11MB3436 Return-Path: divneil.r.wadhawan@intel.com X-OriginatorOrg: intel.com Content-Language: en-US Content-Type: multipart/alternative; boundary="_000_DM6PR11MB4315A962D49C49273E7EF098CB3E0DM6PR11MB4315namp_" --_000_DM6PR11MB4315A962D49C49273E7EF098CB3E0DM6PR11MB4315namp_ Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Hi Ray, Yes, I have tested the following: 1. SECURE_BOOT_ENABLE=3Dtrue * Key Enrollment (PK, KEK, db) via custom mode * Execution of unit test shell application (signed one works okay, unsi= gned gives an Access denied) 1. SECURE_BOOT_ENABLE=3Dfalse (default case) * Secure Boot Configuration menu is not visible (Same as existing defau= lt case) * Execution of Unit Test Application (Signed/Unsigned both works okay) I am planning to post the script in BZ: https://bugzilla.tianocore.org/show= _bug.cgi?id=3D2949 in a day or too. The script generates the full key hierarchy that makes it easy to test this= patch. The patch in BZ requires modifications as per Mike's comment, so, you can s= kip the patches in BZ for now. Regards, Divneil From: Ni, Ray Sent: Thursday, September 17, 2020 12:49 PM To: Wadhawan, Divneil R ; devel@edk2.groups.i= o Cc: gaoliming ; 'Andrew Fish' ; = Justen, Jordan L ; Kinney, Michael D Subject: RE: [edk2-devel] [PATCH v2] EmulatorPkg: Enable support for Secure= Boot Divneil, Just want to double confirm: did you test the secure boot and non-secure bo= ot? Thanks, Ray From: Wadhawan, Divneil R > Sent: Wednesday, September 16, 2020 11:53 PM To: devel@edk2.groups.io Cc: Ni, Ray >; gaoliming >; 'Andrew Fish' >; Justen, Jordan L >; Kinney, Michael D >; Wadhawan, Divneil R > Subject: [edk2-devel] [PATCH v2] EmulatorPkg: Enable support for Secure Boo= t SECURE_BOOT_ENABLE feature flag is introduced to enable Secure Boot. The following gets enabled with this patch: o Secure Boot Menu in "Device Manager" for enrolling keys o Storage space for Authenticated Variables o Authenticated execution of 3rd party images Signed-off-by: Divneil Rai Wadhawan > --- EmulatorPkg/EmulatorPkg.dsc | 37 +++++++++++++++++++++++++++++++++++-- EmulatorPkg/EmulatorPkg.fdf | 14 ++++++++++++++ 2 files changed, 49 insertions(+), 2 deletions(-) diff --git a/EmulatorPkg/EmulatorPkg.dsc b/EmulatorPkg/EmulatorPkg.dsc index 86a6271735..c6e25c745e 100644 --- a/EmulatorPkg/EmulatorPkg.dsc +++ b/EmulatorPkg/EmulatorPkg.dsc @@ -32,6 +32,7 @@ DEFINE NETWORK_TLS_ENABLE =3D FALSE DEFINE NETWORK_HTTP_BOOT_ENABLE =3D FALSE DEFINE NETWORK_ISCSI_ENABLE =3D FALSE + DEFINE SECURE_BOOT_ENABLE =3D FALSE [SkuIds] 0|DEFAULT @@ -106,12 +107,20 @@ LockBoxLib|MdeModulePkg/Library/LockBoxNullLib/LockBoxNullLib.inf CpuExceptionHandlerLib|MdeModulePkg/Library/CpuExceptionHandlerLibNull/C= puExceptionHandlerLibNull.inf TpmMeasurementLib|MdeModulePkg/Library/TpmMeasurementLibNull/TpmMeasurem= entLibNull.inf - AuthVariableLib|MdeModulePkg/Library/AuthVariableLibNull/AuthVariableLib= Null.inf VarCheckLib|MdeModulePkg/Library/VarCheckLib/VarCheckLib.inf SortLib|MdeModulePkg/Library/BaseSortLib/BaseSortLib.inf ShellLib|ShellPkg/Library/UefiShellLib/UefiShellLib.inf FileHandleLib|MdePkg/Library/UefiFileHandleLib/UefiFileHandleLib.inf +!if $(SECURE_BOOT_ENABLE) =3D=3D TRUE + IntrinsicLib|CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf + OpensslLib|CryptoPkg/Library/OpensslLib/OpensslLibCrypto.inf + PlatformSecureLib|SecurityPkg/Library/PlatformSecureLibNull/PlatformSecu= reLibNull.inf + AuthVariableLib|SecurityPkg/Library/AuthVariableLib/AuthVariableLib.inf +!else + AuthVariableLib|MdeModulePkg/Library/AuthVariableLibNull/AuthVariableLib= Null.inf +!endif + [LibraryClasses.common.SEC] PeiServicesLib|EmulatorPkg/Library/SecPeiServicesLib/SecPeiServicesLib.i= nf PcdLib|MdePkg/Library/BasePcdLibNull/BasePcdLibNull.inf @@ -162,6 +171,16 @@ TimerLib|EmulatorPkg/Library/DxeCoreTimerLib/DxeCoreTimerLib.inf EmuThunkLib|EmulatorPkg/Library/DxeEmuLib/DxeEmuLib.inf +[LibraryClasses.common.DXE_DRIVER, LibraryClasses.common.UEFI_DRIVER, Libr= aryClasses.common.UEFI_APPLICATION] +!if $(SECURE_BOOT_ENABLE) =3D=3D TRUE + BaseCryptLib|CryptoPkg/Library/BaseCryptLib/BaseCryptLib.inf +!endif + +[LibraryClasses.common.DXE_RUNTIME_DRIVER] +!if $(SECURE_BOOT_ENABLE) =3D=3D TRUE + BaseCryptLib|CryptoPkg/Library/BaseCryptLib/RuntimeCryptLib.inf +!endif + [LibraryClasses.common.DXE_RUNTIME_DRIVER, LibraryClasses.common.UEFI_DRIVE= R, LibraryClasses.common.DXE_DRIVER, LibraryClasses.common.UEFI_APPLICATION= ] HobLib|MdePkg/Library/DxeHobLib/DxeHobLib.inf PcdLib|MdePkg/Library/DxePcdLib/DxePcdLib.inf @@ -190,6 +209,10 @@ gEmulatorPkgTokenSpaceGuid.PcdEmuFirmwareFdSize|0x002a0000 gEmulatorPkgTokenSpaceGuid.PcdEmuFirmwareBlockSize|0x10000 gEmulatorPkgTokenSpaceGuid.PcdEmuFirmwareVolume|L"../FV/FV_RECOVERY.fd" +!if $(SECURE_BOOT_ENABLE) =3D=3D TRUE + gEfiMdeModulePkgTokenSpaceGuid.PcdMaxAuthVariableSize|0x2800 + gEfiSecurityPkgTokenSpaceGuid.PcdUserPhysicalPresence|TRUE +!endif gEmulatorPkgTokenSpaceGuid.PcdEmuMemorySize|L"64!64" @@ -306,7 +329,14 @@ EmulatorPkg/ResetRuntimeDxe/Reset.inf MdeModulePkg/Core/RuntimeDxe/RuntimeDxe.inf EmulatorPkg/FvbServicesRuntimeDxe/FvbServicesRuntimeDxe.inf - MdeModulePkg/Universal/SecurityStubDxe/SecurityStubDxe.inf + + MdeModulePkg/Universal/SecurityStubDxe/SecurityStubDxe.inf { + +!if $(SECURE_BOOT_ENABLE) =3D=3D TRUE + NULL|SecurityPkg/Library/DxeImageVerificationLib/DxeImageVerificatio= nLib.inf +!endif + } + MdeModulePkg/Universal/EbcDxe/EbcDxe.inf MdeModulePkg/Universal/MemoryTest/NullMemoryTestDxe/NullMemoryTestDxe.in= f EmulatorPkg/EmuThunkDxe/EmuThunk.inf @@ -315,6 +345,9 @@ EmulatorPkg/PlatformSmbiosDxe/PlatformSmbiosDxe.inf EmulatorPkg/TimerDxe/Timer.inf +!if $(SECURE_BOOT_ENABLE) =3D=3D TRUE + SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfigDx= e.inf +!endif MdeModulePkg/Universal/Variable/RuntimeDxe/VariableRuntimeDxe.inf { diff --git a/EmulatorPkg/EmulatorPkg.fdf b/EmulatorPkg/EmulatorPkg.fdf index 295f6f1db8..b256aa9397 100644 --- a/EmulatorPkg/EmulatorPkg.fdf +++ b/EmulatorPkg/EmulatorPkg.fdf @@ -46,10 +46,17 @@ DATA =3D { # Blockmap[1]: End 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, ## This is the VARIABLE_STORE_HEADER +!if $(SECURE_BOOT_ENABLE) =3D=3D FALSE #Signature: gEfiVariableGuid =3D # { 0xddcf3616, 0x3275, 0x4164, { 0x98, 0xb6, 0xfe, 0x85, 0x70, 0x7f, 0= xfe, 0x7d }} 0x16, 0x36, 0xcf, 0xdd, 0x75, 0x32, 0x64, 0x41, 0x98, 0xb6, 0xfe, 0x85, 0x70, 0x7f, 0xfe, 0x7d, +!else + # Signature: gEfiAuthenticatedVariableGuid =3D + # { 0xaaf32c78, 0x947b, 0x439a, { 0xa1, 0x80, 0x2e, 0x14, 0x4e, 0xc3, 0= x77, 0x92 }} + 0x78, 0x2c, 0xf3, 0xaa, 0x7b, 0x94, 0x9a, 0x43, + 0xa1, 0x80, 0x2e, 0x14, 0x4e, 0xc3, 0x77, 0x92, +!endif #Size: 0xc000 (gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageVariableS= ize) - 0x48 (size of EFI_FIRMWARE_VOLUME_HEADER) =3D 0xBFB8 # This can speed up the Variable Dispatch a bit. 0xB8, 0xBF, 0x00, 0x00, @@ -186,6 +193,13 @@ INF RuleOverride =3D UI MdeModulePkg/Application/UiAp= p/UiApp.inf INF MdeModulePkg/Application/BootManagerMenuApp/BootManagerMenuApp.inf INF MdeModulePkg/Universal/DriverSampleDxe/DriverSampleDxe.inf +# +# Secure Boot Key Enroll +# +!if $(SECURE_BOOT_ENABLE) =3D=3D TRUE +INF SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfig= Dxe.inf +!endif + # # Network stack drivers # -- 2.24.1.windows.2 --_000_DM6PR11MB4315A962D49C49273E7EF098CB3E0DM6PR11MB4315namp_ Content-Type: text/html; charset="us-ascii" Content-Transfer-Encoding: quoted-printable

Hi Ray,

 

Yes, I have tested the following:

 

  1. SECURE_BOOT_ENABLE=3Dtrue
  • Key Enrollment (PK, KEK, db) via custom mode
  • Exe= cution of unit test shell application (signed one works okay, unsigned give= s an Access denied)

 

  1. SECURE_BOOT_ENABLE=3Dfalse (default case)
  • Secure Boot Configuration menu is not visible (Same as existing defau= lt case)
  • Execution of Unit Test Application (Signed/Un= signed both works okay)

 

I am planning to post the script in BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=3D2949 in a day or too.<= o:p>

The script generates the full key hierarchy that mak= es it easy to test this patch.

The patch in BZ requires modifications as per Mike&#= 8217;s comment, so, you can skip the patches in BZ for now.

 

Regards,

Divneil

 

From: Ni, Ray <ray.ni@intel.com>
Sent: Thursday, September 17, 2020 12:49 PM
To: Wadhawan, Divneil R <divneil.r.wadhawan@intel.com>; devel@= edk2.groups.io
Cc: gaoliming <gaoliming@byosoft.com.cn>; 'Andrew Fish' <af= ish@apple.com>; Justen, Jordan L <jordan.l.justen@intel.com>; Kinn= ey, Michael D <michael.d.kinney@intel.com>
Subject: RE: [edk2-devel] [PATCH v2] EmulatorPkg: Enable support for= Secure Boot

 

Divneil,

Just want to double confirm: did you test the secure= boot and non-secure boot?

 

Thanks,

Ray

 

From: Wadhawan, Divneil R <divneil.r.wadhawan@intel.com>
Sent: Wednesday, September 16, 2020 11:53 PM
To: devel@edk2.groups.io=
Cc: Ni, Ray <ray.ni@intel.com= >; gaoliming <gaolimi= ng@byosoft.com.cn>; 'Andrew Fish' <afish@apple.com>; Justen, Jordan L <jordan.l.justen@intel.com>; Kinney, Michael D <michae= l.d.kinney@intel.com>; Wadhawan, Divneil R <divneil.r.wadhawan@intel.com>
Subject: [edk2-devel] [PATCH v2] EmulatorPkg: Enable support for Sec= ure Boot

 

SECURE_BOOT_ENABLE feature flag is introduced to ena= ble Secure Boot.

The following gets enabled with this patch:

o Secure Boot Menu in "Device Manager" for= enrolling keys

o Storage space for Authenticated Variables

o Authenticated execution of 3rd party images

 

Signed-off-by: Divneil Rai Wadhawan <divneil.r.wadhawan@intel.com>

---

EmulatorPkg/EmulatorPkg.dsc | 37 +++++++++++++++++++= ++++++++++++++++--

EmulatorPkg/EmulatorPkg.fdf | 14 ++++++++++++++=

2 files changed, 49 insertions(+), 2 deletions(-)

 

diff --git a/EmulatorPkg/EmulatorPkg.dsc b/EmulatorP= kg/EmulatorPkg.dsc

index 86a6271735..c6e25c745e 100644

--- a/EmulatorPkg/EmulatorPkg.dsc

+++ b/EmulatorPkg/EmulatorPkg.dsc

@@ -32,6 +32,7 @@

   DEFINE NETWORK_TLS_ENABLE  &n= bsp;    =3D FALSE

   DEFINE NETWORK_HTTP_BOOT_ENABLE =3D FAL= SE

   DEFINE NETWORK_ISCSI_ENABLE  =    =3D FALSE

+  DEFINE SECURE_BOOT_ENABLE   &= nbsp;   =3D FALSE

 

 [SkuIds]

   0|DEFAULT

@@ -106,12 +107,20 @@

   LockBoxLib|MdeModulePkg/Library/LockBox= NullLib/LockBoxNullLib.inf

   CpuExceptionHandlerLib|MdeModulePkg/Lib= rary/CpuExceptionHandlerLibNull/CpuExceptionHandlerLibNull.inf

   TpmMeasurementLib|MdeModulePkg/Library/= TpmMeasurementLibNull/TpmMeasurementLibNull.inf

-  AuthVariableLib|MdeModulePkg/Library/AuthVar= iableLibNull/AuthVariableLibNull.inf

   VarCheckLib|MdeModulePkg/Library/VarChe= ckLib/VarCheckLib.inf

   SortLib|MdeModulePkg/Library/BaseSortLi= b/BaseSortLib.inf

   ShellLib|ShellPkg/Library/UefiShellLib/= UefiShellLib.inf

   FileHandleLib|MdePkg/Library/UefiFileHa= ndleLib/UefiFileHandleLib.inf

 

+!if $(SECURE_BOOT_ENABLE) =3D=3D TRUE

+  IntrinsicLib|CryptoPkg/Library/IntrinsicLib/= IntrinsicLib.inf

+  OpensslLib|CryptoPkg/Library/OpensslLib/Open= sslLibCrypto.inf

+  PlatformSecureLib|SecurityPkg/Library/Platfo= rmSecureLibNull/PlatformSecureLibNull.inf

+  AuthVariableLib|SecurityPkg/Library/AuthVari= ableLib/AuthVariableLib.inf

+!else

+  AuthVariableLib|MdeModulePkg/Library/AuthVar= iableLibNull/AuthVariableLibNull.inf

+!endif

+

[LibraryClasses.common.SEC]

   PeiServicesLib|EmulatorPkg/Library/SecP= eiServicesLib/SecPeiServicesLib.inf

   PcdLib|MdePkg/Library/BasePcdLibNull/Ba= sePcdLibNull.inf

@@ -162,6 +171,16 @@

   TimerLib|EmulatorPkg/Library/DxeCoreTim= erLib/DxeCoreTimerLib.inf

  EmuThunkLib|EmulatorPkg/Library/DxeEmuLi= b/DxeEmuLib.inf

 

+[LibraryClasses.common.DXE_DRIVER, LibraryClasses.c= ommon.UEFI_DRIVER, LibraryClasses.common.UEFI_APPLICATION]

+!if $(SECURE_BOOT_ENABLE) =3D=3D TRUE

+  BaseCryptLib|CryptoPkg/Library/BaseCryptLib/= BaseCryptLib.inf

+!endif

+

+[LibraryClasses.common.DXE_RUNTIME_DRIVER]

+!if $(SECURE_BOOT_ENABLE) =3D=3D TRUE

+  BaseCryptLib|CryptoPkg/Library/BaseCryptLib/= RuntimeCryptLib.inf

+!endif

+

[LibraryClasses.common.DXE_RUNTIME_DRIVER, LibraryCl= asses.common.UEFI_DRIVER, LibraryClasses.common.DXE_DRIVER, LibraryClasses.= common.UEFI_APPLICATION]

   HobLib|MdePkg/Library/DxeHobLib/DxeHobL= ib.inf

   PcdLib|MdePkg/Library/DxePcdLib/DxePcdL= ib.inf

@@ -190,6 +209,10 @@

   gEmulatorPkgTokenSpaceGuid.PcdEmuFirmwa= reFdSize|0x002a0000

   gEmulatorPkgTokenSpaceGuid.PcdEmuFirmwa= reBlockSize|0x10000

   gEmulatorPkgTokenSpaceGuid.PcdEmuFirmwa= reVolume|L"../FV/FV_RECOVERY.fd"

+!if $(SECURE_BOOT_ENABLE) =3D=3D TRUE

+  gEfiMdeModulePkgTokenSpaceGuid.PcdMaxAuthVar= iableSize|0x2800

+  gEfiSecurityPkgTokenSpaceGuid.PcdUserPhysica= lPresence|TRUE

+!endif

 

   gEmulatorPkgTokenSpaceGuid.PcdEmuM= emorySize|L"64!64"

 

@@ -306,7 +329,14 @@

   EmulatorPkg/ResetRuntimeDxe/Reset.inf

   MdeModulePkg/Core/RuntimeDxe/RuntimeDxe= .inf

   EmulatorPkg/FvbServicesRuntimeDxe/FvbSe= rvicesRuntimeDxe.inf

-  MdeModulePkg/Universal/SecurityStubDxe/Secur= ityStubDxe.inf

+

+  MdeModulePkg/Universal/SecurityStubDxe/Secur= ityStubDxe.inf {

+    <LibraryClasses>

+!if $(SECURE_BOOT_ENABLE) =3D=3D TRUE

+      NULL|SecurityPkg/Lib= rary/DxeImageVerificationLib/DxeImageVerificationLib.inf

+!endif

+  }

+

   MdeModulePkg/Universal/EbcDxe/EbcDxe.in= f

   MdeModulePkg/Universal/MemoryTest/NullM= emoryTestDxe/NullMemoryTestDxe.inf

   EmulatorPkg/EmuThunkDxe/EmuThunk.inf

@@ -315,6 +345,9 @@

   EmulatorPkg/PlatformSmbiosDxe/PlatformS= mbiosDxe.inf

   EmulatorPkg/TimerDxe/Timer.inf

 

+!if $(SECURE_BOOT_ENABLE) =3D=3D TRUE

+  SecurityPkg/VariableAuthenticated/SecureBoot= ConfigDxe/SecureBootConfigDxe.inf

+!endif

 

   MdeModulePkg/Universal/Variable/Ru= ntimeDxe/VariableRuntimeDxe.inf {

     <LibraryClasses>=

diff --git a/EmulatorPkg/EmulatorPkg.fdf b/EmulatorP= kg/EmulatorPkg.fdf

index 295f6f1db8..b256aa9397 100644

--- a/EmulatorPkg/EmulatorPkg.fdf

+++ b/EmulatorPkg/EmulatorPkg.fdf

@@ -46,10 +46,17 @@ DATA =3D {

   # Blockmap[1]: End

   0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x0= 0, 0x00,

   ## This is the VARIABLE_STORE_HEADER

+!if $(SECURE_BOOT_ENABLE) =3D=3D FALSE

   #Signature: gEfiVariableGuid =3D

   #  { 0xddcf3616, 0x3275, 0x4164, {= 0x98, 0xb6, 0xfe, 0x85, 0x70, 0x7f, 0xfe, 0x7d }}

   0x16, 0x36, 0xcf, 0xdd, 0x75, 0x32, 0x6= 4, 0x41,

   0x98, 0xb6, 0xfe, 0x85, 0x70, 0x7f, 0xf= e, 0x7d,

+!else

+  # Signature: gEfiAuthenticatedVariableGuid = =3D

+  #  { 0xaaf32c78, 0x947b, 0x439a, { 0xa1= , 0x80, 0x2e, 0x14, 0x4e, 0xc3, 0x77, 0x92 }}

+  0x78, 0x2c, 0xf3, 0xaa, 0x7b, 0x94, 0x9a, 0x= 43,

+  0xa1, 0x80, 0x2e, 0x14, 0x4e, 0xc3, 0x77, 0x= 92,

+!endif

   #Size: 0xc000 (gEfiMdeModulePkgTokenSpa= ceGuid.PcdFlashNvStorageVariableSize) - 0x48 (size of EFI_FIRMWARE_VOLUME_H= EADER) =3D 0xBFB8

   # This can speed up the Variable Dispat= ch a bit.

   0xB8, 0xBF, 0x00, 0x00,

@@ -186,6 +193,13 @@ INF  RuleOverride =3D UI M= deModulePkg/Application/UiApp/UiApp.inf

INF  MdeModulePkg/Application/BootManagerMenuAp= p/BootManagerMenuApp.inf

INF  MdeModulePkg/Universal/DriverSampleDxe/Dri= verSampleDxe.inf

 

+#

+# Secure Boot Key Enroll

+#

+!if $(SECURE_BOOT_ENABLE) =3D=3D TRUE

+INF SecurityPkg/VariableAuthenticated/SecureBootCon= figDxe/SecureBootConfigDxe.inf

+!endif

+

#

# Network stack drivers

#

--

2.24.1.windows.2

--_000_DM6PR11MB4315A962D49C49273E7EF098CB3E0DM6PR11MB4315namp_--