Thanks for the clean up.

Reviewed-by: Jiewen Yao <[email protected]>

> -----Original Message-----
> From: Ard Biesheuvel [mailto:[email protected]]
> Sent: Thursday, September 6, 2018 9:45 PM
> To: [email protected]
> Cc: Ard Biesheuvel <[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]>; Laszlo
> Ersek <[email protected]>; Leif Lindholm <[email protected]>
> Subject: [PATCH 0/4] remove all 11 occurrences of ELILO on IPF PE/COFF header
> hack
> 
> Now that Itanium support has been dropped from EDK2, we can remove all 11
> (!)
> occurrences of the ELILO PE/COFF loader hack from the code base.
> 
> Note that SecurityPkg appears to have four mostly identical implementations
> of the PE/COFF measuring routine, so this may be another area for cleanup
> later.
> 
> Link: https://bugzilla.tianocore.org/show_bug.cgi?id=816
> 
> Cc: Star Zeng <[email protected]>
> Cc: Jian J Wang <[email protected]>
> Cc: Michael D Kinney <[email protected]>
> Cc: Liming Gao <[email protected]>
> Cc: Chao Zhang <[email protected]>
> Cc: Jiewen Yao <[email protected]>
> Cc: Laszlo Ersek <[email protected]>
> Cc: Leif Lindholm <[email protected]>
> 
> Ard Biesheuvel (4):
>   MdeModulePkg: remove PE/COFF header workaround for ELILO on IPF
>   MdePkg/BasePeCoffLib: remove PE/COFF header workaround for ELILO on
>     IPF
>   SecurityPkg: remove PE/COFF header workaround for ELILO on IPF
>   EdkCompatibilityPkg: remove PE/COFF header workaround for ELILO on IPF
> 
>  .../Library/BasePeCoffLib/BasePeCoff.c        | 60 +++---------------
>  .../Core/Dxe/Mem/MemoryProfileRecord.c        | 31 +---------
>  MdeModulePkg/Core/Dxe/Misc/MemoryProtection.c | 17 +-----
>  MdeModulePkg/Core/Dxe/Misc/PropertiesTable.c  | 17 +-----
>  .../Core/PiSmmCore/MemoryAttributesTable.c    | 17 +-----
>  .../Core/PiSmmCore/SmramProfileRecord.c       | 31 +---------
>  MdePkg/Library/BasePeCoffLib/BasePeCoff.c     | 61 +++----------------
>  .../DxeImageVerificationLib.c                 | 47 +++-----------
>  .../DxeTpmMeasureBootLib.c                    | 27 ++------
>  SecurityPkg/Tcg/Tcg2Dxe/MeasureBootPeCoff.c   | 27 ++------
>  .../SecureBootConfigImpl.c                    | 25 ++------
>  11 files changed, 47 insertions(+), 313 deletions(-)
> 
> --
> 2.18.0

_______________________________________________
edk2-devel mailing list
[email protected]
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to