Sounds good. I think MdePkg\Include\IndustryStandard\LegacyBiosMpTable.h can be removed.
> -----Original Message----- > From: Gao, Liming > Sent: Saturday, April 27, 2019 6:36 PM > To: Yao, Jiewen <[email protected]>; [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 > > 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 (#39707): https://edk2.groups.io/g/devel/message/39707 Mute This Topic: https://groups.io/mt/31349515/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
