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: [email protected] [mailto:[email protected]] On Behalf Of
> Liming Gao
> Sent: Saturday, April 27, 2019 6:28 PM
> To: [email protected]; Gao, Liming <[email protected]>; Zhang,
> Shenglei <[email protected]>
> Cc: Justen, Jordan L <[email protected]>; Laszlo Ersek
> <[email protected]>; Ard Biesheuvel <[email protected]>; Carsey,
> Jaben <[email protected]>; Ni, Ray <[email protected]>; Kinney,
> Michael D <[email protected]>
> 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 <[email protected]>
> 
> 
> >-----Original Message-----
> >From: [email protected] [mailto:[email protected]] On Behalf Of
> >Liming Gao
> >Sent: Friday, April 26, 2019 12:41 PM
> >To: Zhang, Shenglei <[email protected]>; [email protected]
> >Cc: Justen, Jordan L <[email protected]>; Laszlo Ersek
> ><[email protected]>; Ard Biesheuvel <[email protected]>;
> Carsey,
> >Jaben <[email protected]>; Ni, Ray <[email protected]>; Kinney,
> >Michael D <[email protected]>
> >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: [email protected]
> >>Cc: Justen, Jordan L <[email protected]>; Laszlo Ersek
> >><[email protected]>; Ard Biesheuvel <[email protected]>;
> Carsey,
> >>Jaben <[email protected]>; Ni, Ray <[email protected]>; Kinney,
> >>Michael D <[email protected]>; Gao, Liming
> ><[email protected]>
> >>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 <[email protected]>
> >>Cc: Laszlo Ersek <[email protected]>
> >>Cc: Ard Biesheuvel <[email protected]>
> >>Cc: Jaben Carsey <[email protected]>
> >>Cc: Ray Ni <[email protected]>
> >>Cc: Michael D Kinney <[email protected]>
> >>Cc: Liming Gao <[email protected]>
> >>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
> >
> >
> >
> 
> 
> 


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#39705): https://edk2.groups.io/g/devel/message/39705
Mute This Topic: https://groups.io/mt/31349515/21656
Group Owner: [email protected]
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to