No concern at all. I have given R-B for the whole patch series. :-)
> -----Original Message----- > From: Ard Biesheuvel [mailto:[email protected]] > Sent: Thursday, September 20, 2018 5:47 AM > To: Laszlo Ersek <[email protected]> > Cc: [email protected]; Zeng, Star <[email protected]>; Wang, Jian J > <[email protected]>; Kinney, Michael D <[email protected]>; > Gao, Liming <[email protected]>; Zhang, Chao B > <[email protected]>; Yao, Jiewen <[email protected]>; Leif > Lindholm <[email protected]> > Subject: Re: [PATCH v2 3/3] SecurityPkg: remove PE/COFF header > workaround for ELILO on IPF > > On 7 September 2018 at 01:28, Laszlo Ersek <[email protected]> wrote: > > On 09/07/18 07:42, Ard Biesheuvel wrote: > >> Now that Itanium support has been dropped, we can remove the various > >> occurrences of the ELILO on Itanium PE/COFF header workaround. > >> > >> Link: https://bugzilla.tianocore.org/show_bug.cgi?id=816 > >> Contributed-under: TianoCore Contribution Agreement 1.1 > >> Signed-off-by: Ard Biesheuvel <[email protected]> > >> --- > >> > SecurityPkg/Library/DxeImageVerificationLib/DxeImageVerificationLib.c > | 47 ++++---------------- > >> > SecurityPkg/Library/DxeTpmMeasureBootLib/DxeTpmMeasureBootLib.c > | 27 +++-------- > >> SecurityPkg/Tcg/Tcg2Dxe/MeasureBootPeCoff.c > | 27 +++-------- > >> > SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfi > gImpl.c | 25 +++-------- > >> 4 files changed, 25 insertions(+), 101 deletions(-) > > > > Reviewed-by: Laszlo Ersek <[email protected]> > > > > Chao, Jiewen: any concerns? _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel
