From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 192.55.52.93, mailfrom: liming.gao@intel.com) Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by groups.io with SMTP; Sat, 27 Apr 2019 18:36:15 -0700 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by fmsmga102.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 27 Apr 2019 18:36:15 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.60,403,1549958400"; d="scan'208";a="168560232" Received: from fmsmsx108.amr.corp.intel.com ([10.18.124.206]) by fmsmga001.fm.intel.com with ESMTP; 27 Apr 2019 18:36:15 -0700 Received: from fmsmsx122.amr.corp.intel.com (10.18.125.37) by FMSMSX108.amr.corp.intel.com (10.18.124.206) with Microsoft SMTP Server (TLS) id 14.3.408.0; Sat, 27 Apr 2019 18:36:14 -0700 Received: from shsmsx152.ccr.corp.intel.com (10.239.6.52) by fmsmsx122.amr.corp.intel.com (10.18.125.37) with Microsoft SMTP Server (TLS) id 14.3.408.0; Sat, 27 Apr 2019 18:36:14 -0700 Received: from shsmsx104.ccr.corp.intel.com ([169.254.5.92]) by SHSMSX152.ccr.corp.intel.com ([169.254.6.42]) with mapi id 14.03.0415.000; Sun, 28 Apr 2019 09:36:12 +0800 From: "Liming Gao" To: "Yao, Jiewen" , "devel@edk2.groups.io" , "Zhang, Shenglei" CC: "Justen, Jordan L" , Laszlo Ersek , Ard Biesheuvel , "Carsey, Jaben" , "Ni, Ray" , "Kinney, Michael D" Subject: Re: [edk2-devel] [PATCH v2 0/4] Remove IPF and related code Thread-Topic: [edk2-devel] [PATCH v2 0/4] Remove IPF and related code Thread-Index: AQHU+9xmmNTaZAOyLUKlgTl2St4qM6ZN3G9ggALu5ZCAAAEO4IAAASvQ Date: Sun, 28 Apr 2019 01:36:11 +0000 Message-ID: <4A89E2EF3DFEDB4C8BFDE51014F606A14E4319F1@SHSMSX104.ccr.corp.intel.com> References: <20190426030142.7880-1-shenglei.zhang@intel.com> <1598EC71E5B05FD5.20378@groups.io> <4A89E2EF3DFEDB4C8BFDE51014F606A14E4319A4@SHSMSX104.ccr.corp.intel.com> <74D8A39837DF1E4DA445A8C0B3885C503F607085@shsmsx102.ccr.corp.intel.com> In-Reply-To: <74D8A39837DF1E4DA445A8C0B3885C503F607085@shsmsx102.ccr.corp.intel.com> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Return-Path: liming.gao@intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Jiewen: This is a good suggestion. I can submit another BZ for it. Do you know a= ny 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 ; Zhang, >Shenglei >Cc: Justen, Jordan L ; Laszlo Ersek >; Ard Biesheuvel ; Carsey, >Jaben ; Ni, Ray ; Kinney, >Michael D >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 ; Zhang, >> Shenglei >> Cc: Justen, Jordan L ; Laszlo Ersek >> ; Ard Biesheuvel ; Carsey= , >> Jaben ; Ni, Ray ; Kinney, >> Michael D >> 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 >> >> >> >-----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 ; devel@edk2.groups.io >> >Cc: Justen, Jordan L ; Laszlo Ersek >> >; Ard Biesheuvel ; >> Carsey, >> >Jaben ; Ni, Ray ; Kinney, >> >Michael D >> >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 ; Laszlo Ersek >> >>; Ard Biesheuvel ; >> Carsey, >> >>Jaben ; Ni, Ray ; Kinney, >> >>Michael D ; Gao, Liming >> > >> >>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=3D1560 >> >> >> >>v2: Withdraw the removal of Mps.h and related changes. >> >> >> >>Cc: Jordan Justen >> >>Cc: Laszlo Ersek >> >>Cc: Ard Biesheuvel >> >>Cc: Jaben Carsey >> >>Cc: Ray Ni >> >>Cc: Michael D Kinney >> >>Cc: Liming Gao >> >>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 >> > >> > >> > >> >> >>=20