Re: [edk2-devel] [PATCH v2 0/4] Remove IPF and related code

["Liming Gao" <liming.gao@intel.com>]

Jiewen:
  This is a good suggestion. I can submit another BZ for it. Do you know any other definitions are for legacy CSM?

Thanks
Liming
>-----Original Message-----
>From: Yao, Jiewen
>Sent: Sunday, April 28, 2019 9:32 AM
>To: devel@edk2.groups.io; Gao, Liming <liming.gao@intel.com>; Zhang,
>Shenglei <shenglei.zhang@intel.com>
>Cc: Justen, Jordan L <jordan.l.justen@intel.com>; Laszlo Ersek
><lersek@redhat.com>; Ard Biesheuvel <ard.biesheuvel@linaro.org>; Carsey,
>Jaben <jaben.carsey@intel.com>; Ni, Ray <ray.ni@intel.com>; Kinney,
>Michael D <michael.d.kinney@intel.com>
>Subject: RE: [edk2-devel] [PATCH v2 0/4] Remove IPF and related code
>
>Hi
>MPS is not for IPF.
>However, MPS is a very legacy stuff.
>
>Can we consider removing it with CSM cleanup task?
>
>I don't think any modern OS will use MPS.
>
>Thank you
>Yao Jiewen
>
>> -----Original Message-----
>> From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of
>> Liming Gao
>> Sent: Saturday, April 27, 2019 6:28 PM
>> To: devel@edk2.groups.io; Gao, Liming <liming.gao@intel.com>; Zhang,
>> Shenglei <shenglei.zhang@intel.com>
>> Cc: Justen, Jordan L <jordan.l.justen@intel.com>; Laszlo Ersek
>> <lersek@redhat.com>; Ard Biesheuvel <ard.biesheuvel@linaro.org>; Carsey,
>> Jaben <jaben.carsey@intel.com>; Ni, Ray <ray.ni@intel.com>; Kinney,
>> Michael D <michael.d.kinney@intel.com>
>> Subject: Re: [edk2-devel] [PATCH v2 0/4] Remove IPF and related code
>>
>> I see the information has been added in BZ. This patch set is OK to me.
>> Reviewed-by: Liming Gao <liming.gao@intel.com>
>>
>>
>> >-----Original Message-----
>> >From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of
>> >Liming Gao
>> >Sent: Friday, April 26, 2019 12:41 PM
>> >To: Zhang, Shenglei <shenglei.zhang@intel.com>; devel@edk2.groups.io
>> >Cc: Justen, Jordan L <jordan.l.justen@intel.com>; Laszlo Ersek
>> ><lersek@redhat.com>; Ard Biesheuvel <ard.biesheuvel@linaro.org>;
>> Carsey,
>> >Jaben <jaben.carsey@intel.com>; Ni, Ray <ray.ni@intel.com>; Kinney,
>> >Michael D <michael.d.kinney@intel.com>
>> >Subject: Re: [edk2-devel] [PATCH v2 0/4] Remove IPF and related code
>> >
>> >Please add the comments in BZ on why Mps.h can't be removed.
>> >
>> >Thanks
>> >Liming
>> >>-----Original Message-----
>> >>From: Zhang, Shenglei
>> >>Sent: Friday, April 26, 2019 11:02 AM
>> >>To: devel@edk2.groups.io
>> >>Cc: Justen, Jordan L <jordan.l.justen@intel.com>; Laszlo Ersek
>> >><lersek@redhat.com>; Ard Biesheuvel <ard.biesheuvel@linaro.org>;
>> Carsey,
>> >>Jaben <jaben.carsey@intel.com>; Ni, Ray <ray.ni@intel.com>; Kinney,
>> >>Michael D <michael.d.kinney@intel.com>; Gao, Liming
>> ><liming.gao@intel.com>
>> >>Subject: [PATCH v2 0/4] Remove IPF and related code
>> >>
>> >>Itanium arch is not supported any longer. So remove IPF and
>> >>related code.
>> >>https://bugzilla.tianocore.org/show_bug.cgi?id=1560
>> >>
>> >>v2: Withdraw the removal of Mps.h and related changes.
>> >>
>> >>Cc: Jordan Justen <jordan.l.justen@intel.com>
>> >>Cc: Laszlo Ersek <lersek@redhat.com>
>> >>Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
>> >>Cc: Jaben Carsey <jaben.carsey@intel.com>
>> >>Cc: Ray Ni <ray.ni@intel.com>
>> >>Cc: Michael D Kinney <michael.d.kinney@intel.com>
>> >>Cc: Liming Gao <liming.gao@intel.com>
>> >>Shenglei Zhang (4):
>> >>  OvmfPkg/PlatformBootManagerLib: Remove dependency on Mps.h
>> >>  ShellPkg/UefiShellDebug1CommandsLib
>> >>  ShellPkg/UefiHandleParsingLib: Remove some unused Guids
>> >>  MdePkg: Removed IPF related code
>> >>
>> >> MdePkg/Include/Guid/SalSystemTable.h          |   25 -
>> >> MdePkg/Include/IndustryStandard/Pal.h         | 3296 -----------------
>> >> MdePkg/Include/IndustryStandard/Sal.h         |  909 -----
>> >> MdePkg/Include/Library/ExtendedSalLib.h       |  488 ---
>> >> MdePkg/Include/Library/PalLib.h               |   57 -
>> >> MdePkg/Include/Library/SalLib.h               |   53 -
>> >> .../Include/Protocol/ExtendedSalBootService.h |  211 --
>> >> .../Protocol/ExtendedSalServiceClasses.h      |  272 --
>> >> MdePkg/Include/Protocol/McaInitPmi.h          |  201 -
>> >> MdePkg/MdePkg.dec                             |   26 -
>> >> .../PlatformBootManagerLib/BdsPlatform.h      |    1 -
>> >> .../UefiHandleParsingLib.c                    |   17 -
>> >> .../UefiHandleParsingLib.inf                  |   17 -
>> >> .../UefiHandleParsingLib.uni                  |   17 -
>> >> .../Library/UefiShellDebug1CommandsLib/Dmem.c |    5 -
>> >> .../UefiShellDebug1CommandsLib.inf            |    1 -
>> >> 16 files changed, 5596 deletions(-)
>> >> delete mode 100644 MdePkg/Include/Guid/SalSystemTable.h
>> >> delete mode 100644 MdePkg/Include/IndustryStandard/Pal.h
>> >> delete mode 100644 MdePkg/Include/IndustryStandard/Sal.h
>> >> delete mode 100644 MdePkg/Include/Library/ExtendedSalLib.h
>> >> delete mode 100644 MdePkg/Include/Library/PalLib.h
>> >> delete mode 100644 MdePkg/Include/Library/SalLib.h
>> >> delete mode 100644
>> MdePkg/Include/Protocol/ExtendedSalBootService.h
>> >> delete mode 100644
>> >MdePkg/Include/Protocol/ExtendedSalServiceClasses.h
>> >> delete mode 100644 MdePkg/Include/Protocol/McaInitPmi.h
>> >>
>> >>--
>> >>2.18.0.windows.1
>> >
>> >
>> >
>>
>>
>>