From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 134.134.136.126, mailfrom: jiewen.yao@intel.com) Received: from mga18.intel.com (mga18.intel.com [134.134.136.126]) by groups.io with SMTP; Sat, 27 Apr 2019 18:46:30 -0700 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga008.fm.intel.com ([10.253.24.58]) by orsmga106.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 27 Apr 2019 18:46:29 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.60,403,1549958400"; d="scan'208";a="144189874" Received: from fmsmsx105.amr.corp.intel.com ([10.18.124.203]) by fmsmga008.fm.intel.com with ESMTP; 27 Apr 2019 18:46:29 -0700 Received: from fmsmsx153.amr.corp.intel.com (10.18.125.6) by FMSMSX105.amr.corp.intel.com (10.18.124.203) with Microsoft SMTP Server (TLS) id 14.3.408.0; Sat, 27 Apr 2019 18:46:28 -0700 Received: from shsmsx106.ccr.corp.intel.com (10.239.4.159) by FMSMSX153.amr.corp.intel.com (10.18.125.6) with Microsoft SMTP Server (TLS) id 14.3.408.0; Sat, 27 Apr 2019 18:46:28 -0700 Received: from shsmsx102.ccr.corp.intel.com ([169.254.2.206]) by SHSMSX106.ccr.corp.intel.com ([169.254.10.21]) with mapi id 14.03.0415.000; Sun, 28 Apr 2019 09:46:26 +0800 From: "Yao, Jiewen" To: "Gao, Liming" , "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+9xmmNTaZAOyLUKlgTl2St4qM6ZN3G9ggALu5ZCAAAEO4IAAASvQgAACpLA= Date: Sun, 28 Apr 2019 01:46:25 +0000 Message-ID: <74D8A39837DF1E4DA445A8C0B3885C503F60710D@shsmsx102.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> <4A89E2EF3DFEDB4C8BFDE51014F606A14E4319F1@SHSMSX104.ccr.corp.intel.com> In-Reply-To: <4A89E2EF3DFEDB4C8BFDE51014F606A14E4319F1@SHSMSX104.ccr.corp.intel.com> Accept-Language: zh-CN, en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiMDBmYzMwMDUtYTI4OS00ZDFkLTlkMDAtOWU0ODdjYmNkMGQ1IiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjEwLjE4MDQuNDkiLCJUcnVzdGVkTGFiZWxIYXNoIjoicUhIaGJXSXpEdjlMXC9aZWFCUXE0cVQ2WnNxUVpJM2JBQ0dETlwvQ0xTVjRyYXZqMjJUSHo1MHdRbEZkZDB3QjhhIn0= 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 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 ; 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 >=20 > Jiewen: > This is a good suggestion. I can submit another BZ for it. Do you know= any > other definitions are for legacy CSM? >=20 > 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 m= e. > >> 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