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.43, mailfrom: jiewen.yao@intel.com) Received: from mga05.intel.com (mga05.intel.com [192.55.52.43]) by groups.io with SMTP; Sat, 27 Apr 2019 18:32:32 -0700 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga004.fm.intel.com ([10.253.24.48]) by fmsmga105.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 27 Apr 2019 18:32:32 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.60,403,1549958400"; d="scan'208";a="165686448" Received: from fmsmsx107.amr.corp.intel.com ([10.18.124.205]) by fmsmga004.fm.intel.com with ESMTP; 27 Apr 2019 18:32:33 -0700 Received: from fmsmsx151.amr.corp.intel.com (10.18.125.4) by fmsmsx107.amr.corp.intel.com (10.18.124.205) with Microsoft SMTP Server (TLS) id 14.3.408.0; Sat, 27 Apr 2019 18:32:32 -0700 Received: from shsmsx151.ccr.corp.intel.com (10.239.6.50) by FMSMSX151.amr.corp.intel.com (10.18.125.4) with Microsoft SMTP Server (TLS) id 14.3.408.0; Sat, 27 Apr 2019 18:32:31 -0700 Received: from shsmsx102.ccr.corp.intel.com ([169.254.2.206]) by SHSMSX151.ccr.corp.intel.com ([169.254.3.39]) with mapi id 14.03.0415.000; Sun, 28 Apr 2019 09:32:28 +0800 From: "Yao, Jiewen" 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 Thread-Topic: [edk2-devel] [PATCH v2 0/4] Remove IPF and related code Thread-Index: AQHU+9xmmNTaZAOyLUKlgTl2St4qM6ZN3G9ggALu5ZCAAAEO4A== Date: Sun, 28 Apr 2019 01:32:27 +0000 Message-ID: <74D8A39837DF1E4DA445A8C0B3885C503F607085@shsmsx102.ccr.corp.intel.com> References: <20190426030142.7880-1-shenglei.zhang@intel.com> <1598EC71E5B05FD5.20378@groups.io> <4A89E2EF3DFEDB4C8BFDE51014F606A14E4319A4@SHSMSX104.ccr.corp.intel.com> In-Reply-To: <4A89E2EF3DFEDB4C8BFDE51014F606A14E4319A4@SHSMSX104.ccr.corp.intel.com> Accept-Language: zh-CN, en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiZjkwNGZkNTctZDRhOS00OGM1LWEyNzQtZDJiNjM4Y2Q0MDRiIiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjEwLjE4MDQuNDkiLCJUcnVzdGVkTGFiZWxIYXNoIjoiRHNZSWtKWHkySWtZWXNhWFBcL1drZVVOXC9VY1wvU1JtNmNlT1lHMGFvYk1rSkVqMWEyV0JPcnVGeU1KcXNZRjR6TyJ9 x-ctpclassification: CTP_NT dlp-product: dlpe-windows dlp-version: 11.0.600.7 dlp-reaction: no-action x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Return-Path: jiewen.yao@intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable 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 >=20 > I see the information has been added in BZ. This patch set is OK to me. > Reviewed-by: Liming Gao >=20 >=20 > >-----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 >=20 >=20