From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by mx.groups.io with SMTP id smtpd.web10.10276.1587345162881373563 for ; Sun, 19 Apr 2020 18:12:43 -0700 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 134.134.136.65, mailfrom: shiwei.xu@intel.com) IronPort-SDR: bbe+wss+ScMFHBmGD/a0ymPwvhQJMkWN+7N4wXpaaHs+syLOOF59AZ/iB9LQvwBjBl1tXe3M7f OF7H4kWTKrfA== X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga007.jf.intel.com ([10.7.209.58]) by orsmga103.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 19 Apr 2020 18:12:41 -0700 IronPort-SDR: 4NpZ0MaUkKt6jB1qWVkyBHlj82tH5dTXXf5v1Q1A/MqxYjbaCtIPVVpJpZpXhdKdMR9XKpixXy xhX/GxU0lg9w== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.72,405,1580803200"; d="scan'208";a="243649211" Received: from fmsmsx107.amr.corp.intel.com ([10.18.124.205]) by orsmga007.jf.intel.com with ESMTP; 19 Apr 2020 18:12:41 -0700 Received: from fmsmsx156.amr.corp.intel.com (10.18.116.74) by fmsmsx107.amr.corp.intel.com (10.18.124.205) with Microsoft SMTP Server (TLS) id 14.3.439.0; Sun, 19 Apr 2020 18:12:41 -0700 Received: from shsmsx101.ccr.corp.intel.com (10.239.4.153) by fmsmsx156.amr.corp.intel.com (10.18.116.74) with Microsoft SMTP Server (TLS) id 14.3.439.0; Sun, 19 Apr 2020 18:12:41 -0700 Received: from shsmsx104.ccr.corp.intel.com ([169.254.5.225]) by SHSMSX101.ccr.corp.intel.com ([169.254.1.129]) with mapi id 14.03.0439.000; Mon, 20 Apr 2020 09:12:38 +0800 From: "Xu, Shiwei" To: "Jiang, Guomin" , "devel@edk2.groups.io" , "Wang, Jian J" , "Yao, Jiewen" , "Zhang, Chao B" , "Laszlo Ersek" CC: "Gao, Liming" Subject: Re: [edk2-devel] [PATCH v2 0/6] Mark the File parameter as OPTIONAL Thread-Topic: [edk2-devel] [PATCH v2 0/6] Mark the File parameter as OPTIONAL Thread-Index: AQHWE8F+UzZhGVhCQkqg+AeCbKs5A6h7wLCwgAV0YuCAAARLkA== Date: Mon, 20 Apr 2020 01:12:38 +0000 Message-ID: References: <16063C7FB70875D2.18921@groups.io> In-Reply-To: Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Return-Path: shiwei.xu@intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Thanks Guomin! This change is critical now, because it could fix OpROM dispatch issue, we= need this solution check-in as soon as possible to unblock validation test= s. --Thanks Shiwei Xu -----Original Message----- From: Jiang, Guomin=20 Sent: Monday, April 20, 2020 8:56 AM To: devel@edk2.groups.io; Wang, Jian J ; Yao, Jiewe= n ; Zhang, Chao B ; Laszlo Er= sek Cc: Xu, Shiwei ; Gao, Liming Subject: RE: [edk2-devel] [PATCH v2 0/6] Mark the File parameter as OPTION= AL Importance: High Hi Jian, Chao, Laszlo, Can you spend some time for this. Best Regard Guomin > -----Original Message----- > From: Jiang, Guomin > Sent: Thursday, April 16, 2020 9:41 PM > To: devel@edk2.groups.io; Wang, Jian J ; Yao,=20 > Jiewen ; Zhang, Chao B ;= =20 > Laszlo Ersek > Cc: Xu, Shiwei ; Gao, Liming=20 > > Subject: RE: [edk2-devel] [PATCH v2 0/6] Mark the File parameter as=20 > OPTIONAL >=20 > Hi Laszlo, >=20 > According to your advice, i post new change. Can you help review the=20 > series again? >=20 > Hi Jian, Chao, Jiewen, >=20 > Please help review the series. >=20 > Thanks, > guomin >=20 > > -----Original Message----- > > From: devel@edk2.groups.io On Behalf Of > Guomin > > Jiang > > Sent: Thursday, April 16, 2020 3:34 PM > > To: devel@edk2.groups.io > > Subject: [edk2-devel] [PATCH v2 0/6] Mark the File parameter as=20 > > OPTIONAL > > > > File paramter should be optional according to the descritpion. > > > > Guomin Jiang (6): > > SecurityPkg/TPM: measure UEFI images without associated device paths > > again > > SecurityPkg/DxeImageAuth: Mark the File parameter as option > > SecurityPkg/DxeImageVerificationLib: Mark the File parameter as > > OPTIONAL > > MdeModulePkg/SecurityManagementLib: Mark the File parameter as > > OPTIONAL > > MdeModulePkg/SecurityStubDxe: Mark the File parameter as OPTIONAL > > MdePkg/Security2: Mark the File parameter as OPTIONAL. > > > > .../Include/Library/SecurityManagementLib.h | 2 +- > > .../DxeSecurityManagementLib.c | 2 +- > > .../Universal/SecurityStubDxe/SecurityStub.c | 2 +- > > MdePkg/Include/Protocol/Security2.h | 2 +- > > .../DxeImageAuthenticationStatusLib.c | 2 +- > > .../DxeImageVerificationLib.c | 2 +- > > .../DxeTpm2MeasureBootLib.c | 20 +++++++++---------= - > > .../DxeTpmMeasureBootLib.c | 20 +++++++++---------= - > > 8 files changed, 26 insertions(+), 26 deletions(-) > > > > -- > > 2.25.1.windows.1 > > > > > >=20