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....@intel.com>; Zhang, >Shenglei <shenglei.zh...@intel.com> >Cc: Justen, Jordan L <jordan.l.jus...@intel.com>; Laszlo Ersek ><ler...@redhat.com>; Ard Biesheuvel <ard.biesheu...@linaro.org>; Carsey, >Jaben <jaben.car...@intel.com>; Ni, Ray <ray...@intel.com>; Kinney, >Michael D <michael.d.kin...@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....@intel.com>; Zhang, >> Shenglei <shenglei.zh...@intel.com> >> Cc: Justen, Jordan L <jordan.l.jus...@intel.com>; Laszlo Ersek >> <ler...@redhat.com>; Ard Biesheuvel <ard.biesheu...@linaro.org>; Carsey, >> Jaben <jaben.car...@intel.com>; Ni, Ray <ray...@intel.com>; Kinney, >> Michael D <michael.d.kin...@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....@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.zh...@intel.com>; devel@edk2.groups.io >> >Cc: Justen, Jordan L <jordan.l.jus...@intel.com>; Laszlo Ersek >> ><ler...@redhat.com>; Ard Biesheuvel <ard.biesheu...@linaro.org>; >> Carsey, >> >Jaben <jaben.car...@intel.com>; Ni, Ray <ray...@intel.com>; Kinney, >> >Michael D <michael.d.kin...@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.jus...@intel.com>; Laszlo Ersek >> >><ler...@redhat.com>; Ard Biesheuvel <ard.biesheu...@linaro.org>; >> Carsey, >> >>Jaben <jaben.car...@intel.com>; Ni, Ray <ray...@intel.com>; Kinney, >> >>Michael D <michael.d.kin...@intel.com>; Gao, Liming >> ><liming....@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.jus...@intel.com> >> >>Cc: Laszlo Ersek <ler...@redhat.com> >> >>Cc: Ard Biesheuvel <ard.biesheu...@linaro.org> >> >>Cc: Jaben Carsey <jaben.car...@intel.com> >> >>Cc: Ray Ni <ray...@intel.com> >> >>Cc: Michael D Kinney <michael.d.kin...@intel.com> >> >>Cc: Liming Gao <liming....@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 >> > >> > >> > >> >> >> -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#39706): https://edk2.groups.io/g/devel/message/39706 Mute This Topic: https://groups.io/mt/31349515/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-