Re: [edk2-devel] [PATCH v2 0/6] Mark the File parameter as OPTIONAL

["Xu, Shiwei" <shiwei.xu@intel.com>]

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 tests.


--Thanks
Shiwei Xu

-----Original Message-----
From: Jiang, Guomin 
Sent: Monday, April 20, 2020 8:56 AM
To: devel@edk2.groups.io; Wang, Jian J <jian.j.wang@intel.com>; Yao, Jiewen <jiewen.yao@intel.com>; Zhang, Chao B <chao.b.zhang@intel.com>; Laszlo Ersek <lersek@redhat.com>
Cc: Xu, Shiwei <shiwei.xu@intel.com>; Gao, Liming <liming.gao@intel.com>
Subject: RE: [edk2-devel] [PATCH v2 0/6] Mark the File parameter as OPTIONAL
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 <jian.j.wang@intel.com>; Yao, 
> Jiewen <jiewen.yao@intel.com>; Zhang, Chao B <chao.b.zhang@intel.com>; 
> Laszlo Ersek <lersek@redhat.com>
> Cc: Xu, Shiwei <shiwei.xu@intel.com>; Gao, Liming 
> <liming.gao@intel.com>
> Subject: RE: [edk2-devel] [PATCH v2 0/6] Mark the File parameter as 
> OPTIONAL
> 
> Hi Laszlo,
> 
> According to your advice, i post new change. Can you help review the 
> series again?
> 
> Hi Jian, Chao, Jiewen,
> 
> Please help review the series.
> 
> Thanks,
> guomin
> 
> > -----Original Message-----
> > From: devel@edk2.groups.io <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 
> > 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
> >
> >
> >