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: [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 > >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 (#39706): https://edk2.groups.io/g/devel/message/39706 Mute This Topic: https://groups.io/mt/31349515/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
