From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail05.groups.io (mail05.groups.io [45.79.224.7]) by spool.mail.gandi.net (Postfix) with ESMTPS id 77D7AAC0864 for ; Tue, 7 May 2024 04:59:08 +0000 (UTC) DKIM-Signature: a=rsa-sha256; bh=m4WbKgHQjCUKHMeyGC88TenP9KBQB5XxnyrxPsUh8z8=; c=relaxed/simple; d=groups.io; h=From:To:CC:Subject:Thread-Topic:Thread-Index:Date:Message-ID:References:In-Reply-To:Accept-Language:msip_labels:MIME-Version:Precedence:List-Subscribe:List-Help:Sender:List-Id:Mailing-List:Delivered-To:Resent-Date:Resent-From:Reply-To:List-Unsubscribe-Post:List-Unsubscribe:Content-Language:Content-Type; s=20240206; t=1715057946; v=1; b=XngzRwYsXpdU6HeUIwpXV1OV9Q7qFK/jicYSXLT1o56+VQp6WL1jYyzHVoTLFG51ZeTnYJuc ONAeRXvJ0ZtZd812AQmxiEbmvhZVICgPGg6X8L3hXSWdrOq31en76BD6evUC7n7OQR1h8J0UPNH m17qpxAqRExXckxtx/ihJCTryJeOBedBNmITaOhMfcXDCyMoxlxhC7KwHu0mckNZuq8IfJQX2Zb f1ZZJny/NECJmHuOZIlBHE1COxpxLz9ssTCnfWIjpOW1gXBw67YbdbYor0/2kMIXoJQkR3UjdNa WbpfPvGjoJQBvyBXDHQRE6tPCD0R9OUEPIxbV1XrOMC/Q== X-Received: by 127.0.0.2 with SMTP id OwbPYY7687511xGLtQF2PmtY; Mon, 06 May 2024 21:59:06 -0700 X-Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.9]) by mx.groups.io with SMTP id smtpd.web11.4360.1715057945482081471 for ; Mon, 06 May 2024 21:59:05 -0700 X-CSE-ConnectionGUID: GFv41MhLTTyJKw1VMrDzYA== X-CSE-MsgGUID: j9UA2zcPRgeedlngKi8cig== X-IronPort-AV: E=McAfee;i="6600,9927,11065"; a="33335291" X-IronPort-AV: E=Sophos;i="6.07,260,1708416000"; d="scan'208,217";a="33335291" X-Received: from orviesa008.jf.intel.com ([10.64.159.148]) by orvoesa101.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 06 May 2024 21:59:05 -0700 X-CSE-ConnectionGUID: MRdGzcR+Sq2MRis8bvO/MQ== X-CSE-MsgGUID: xnqxPwk9TA66Ti/iu/lsfA== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.07,260,1708416000"; d="scan'208,217";a="28968552" X-Received: from fmsmsx602.amr.corp.intel.com ([10.18.126.82]) by orviesa008.jf.intel.com with ESMTP/TLS/AES256-GCM-SHA384; 06 May 2024 21:59:05 -0700 X-Received: from fmsmsx611.amr.corp.intel.com (10.18.126.91) by fmsmsx602.amr.corp.intel.com (10.18.126.82) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.35; Mon, 6 May 2024 21:59:04 -0700 X-Received: from fmsmsx610.amr.corp.intel.com (10.18.126.90) by fmsmsx611.amr.corp.intel.com (10.18.126.91) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.35; Mon, 6 May 2024 21:59:04 -0700 X-Received: from fmsedg602.ED.cps.intel.com (10.1.192.136) by fmsmsx610.amr.corp.intel.com (10.18.126.90) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.35 via Frontend Transport; Mon, 6 May 2024 21:59:04 -0700 X-Received: from NAM04-DM6-obe.outbound.protection.outlook.com (104.47.73.41) by edgegateway.intel.com (192.55.55.71) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2507.35; Mon, 6 May 2024 21:59:04 -0700 X-Received: from MW4PR11MB5872.namprd11.prod.outlook.com (2603:10b6:303:169::14) by PH7PR11MB7025.namprd11.prod.outlook.com (2603:10b6:510:208::12) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7544.41; Tue, 7 May 2024 04:59:00 +0000 X-Received: from MW4PR11MB5872.namprd11.prod.outlook.com ([fe80::6444:ca4c:aa3e:f8d2]) by MW4PR11MB5872.namprd11.prod.outlook.com ([fe80::6444:ca4c:aa3e:f8d2%4]) with mapi id 15.20.7544.041; Tue, 7 May 2024 04:59:00 +0000 From: "Yao, Jiewen" To: "Wu, Jiaxin" , "Ni, Ray" , "devel@edk2.groups.io" , Ard Biesheuvel CC: "Zeng, Star" , Gerd Hoffmann , "Kumar, Rahul R" , "Dong, Guo" , "Rhodes, Sean" , "Lu, James" , "Guo, Gua" , Abdul Lateef Attar , Abner Chang , Tom Lendacky Subject: Re: [edk2-devel] [PATCH v4 00/14] Add SmmRelocationLib Thread-Topic: [PATCH v4 00/14] Add SmmRelocationLib Thread-Index: AQHal9PA9IGV4Fma0kKigj4l3IqtTrGAVyLRgABAhnCACpdQ4IAAF9aw Date: Tue, 7 May 2024 04:59:00 +0000 Message-ID: References: <20240426121720.54308-1-jiaxin.wu@intel.com> In-Reply-To: Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: msip_labels: x-ms-publictraffictype: Email x-ms-traffictypediagnostic: MW4PR11MB5872:EE_|PH7PR11MB7025:EE_ x-ms-office365-filtering-correlation-id: 8ce1aceb-65d8-4e5a-a421-08dc6e5268d7 x-ld-processed: 46c98d88-e344-4ed4-8496-4ed7712e255d,ExtAddr x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam-message-info: =?us-ascii?Q?HGNw90c6/hAiyHLKOYxyxdvkUb/JgGaxPEjJilBIswnGrVGHlP9QQZkn7leF?= =?us-ascii?Q?DMbzTXdOiy/hggsgSXVsDXqC0iV3bCxXGnuRYu+EeYldDGR/eoypcPTPXRE+?= =?us-ascii?Q?iyr3PGxV0ia9/s9KuDPfhxHs9zUHypjBVsgSirsDspZ7eC9SZr+62aKdrD/O?= =?us-ascii?Q?w9EovYkE3tL/IcjQGYt9POnEgj6gnk/LbL2NXjSBuGEB9WlBFMxNG+QMv7sh?= =?us-ascii?Q?yFufwUDSiFzCoUR+5oEZE6xFzjSJERT6CVIlzXvAXZdZRbywhBTrguwsAGdJ?= =?us-ascii?Q?1A+rXqBpkBamv/OT1oasHUmr67pm3CescL6Xs5PV+e9PeaJF37PUPehX9uY2?= =?us-ascii?Q?LkkQgLbxmc240m/5BpbDc7W+inzh4Ou2ddcW9pxGiToTj9n+UnRnK+hnMsab?= =?us-ascii?Q?7Goan2DEZpL1wx1eQuz/9coemGpvCUrUeir5HCDOz8Cq0YW1vKb/A5oOZwB1?= =?us-ascii?Q?qO/5r7i8SPhsbrxiICU/FIqUKlsk5L02raiJ5hIfG5TnENB+EZ8Ljgnhvrv3?= =?us-ascii?Q?bK+4EIJ7N/zb+S0hgnfLaP1z9/vMDlMtaZetV10YYZ6s40rmqZ2NgJNNoVRy?= =?us-ascii?Q?8s0E0RYd3fjW5L6312Y0VJuDR6JXGF6pg0K8/v8bpPYBR2RWyxCbHt+gOJSU?= =?us-ascii?Q?vIbqO8NrTvR7Tu3n/VMxBU5Ga8AeYYR2/rPGWNklhKG/1FGRWmysJLg2uKUQ?= =?us-ascii?Q?1eSABPOjsOhieBmewiKRXT/ooKUwQ7J6y1wV8SS7QN+ezWatCZKzj4aa9pS7?= =?us-ascii?Q?sf6C2/fnxdz6qrKAWWU+yILDl4L4I+SY7LKI0oNu5ddGCh9/LgOBsqXvTrVB?= =?us-ascii?Q?YmpaA3Pg4+h+b5kTilMukpiJSZ5CYUooOipP1nN4rP6ZaTs0+hzRdL7lsdXO?= =?us-ascii?Q?+MP4aobj4ddbaaz+AH7gZh0UeUy5cI85uIJEl60UgpAaFdXtW92+VHqPrlat?= =?us-ascii?Q?DOTpKzy6y/tiReT68NGX8/DKpLViKSQOkJ1JN9VXr/pJbAHEEQO8xWk5c6ul?= =?us-ascii?Q?kIaUnAEYwqKU7ZZepJBOCECUizHMDDEoeHzabq+0MFsWqpgDWKDuuMF/Vg6Q?= =?us-ascii?Q?PxSDRh42Vg8L5N6jRajXWII/Xy01nlPoC8ykNVIqsXpte0ro3ZNKkykQ5Lul?= =?us-ascii?Q?PtVWm6fa678iVOzjy5XLapGtgvccPZDnrFzKloWMe+eMMzUzehri5dzvMMcd?= =?us-ascii?Q?PKdWWC0zlU2yN3cIineLZlu/R8PwdiacilRcPVMKSBDGASUuVZKbWm4hhii2?= =?us-ascii?Q?O6jMVVyt3dvFhJxm1MRQQec+0rM9TbBzUUNn+eCpUg=3D=3D?= x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?vDr1lZB/isHoL04WdU5nND3ef9Zg3qcCZ/fKS9CY51+9ucyk2qRP33hhC/CZ?= =?us-ascii?Q?6P9XKSTbXUUMyYpI6Vmhxf610uucMWOw98Qx+zgOrOWuP/GUFyabzCRdFmec?= =?us-ascii?Q?FMEQgzaj5tyRu5mFBcMGewjCNGDM8yU+SuUo8/Ko9LH+q3dfwWXmNwcIHDdT?= =?us-ascii?Q?XKuTM60FNRNnExdYsYPcdRq6w6BId9Gra+WGy/4bNI+ZD1gpmp2YzQXURtoJ?= =?us-ascii?Q?VOTlGRAlnz9bqcVB+TipzpV4LYj54FPX8yIe2WjvwWbK6Zp1HlirSci95th9?= =?us-ascii?Q?zQueqaOVrQnW25XflclFJRRF97T21YKG3M16mPrQdyz/ZGi/8PBl4wt0spjG?= =?us-ascii?Q?4TO0g48kmUhIXwLR9Ke5DvAwZNcAvU6r6qPxCPLQ3ZJ25BOxVMcx2Q+5TQzL?= =?us-ascii?Q?qMWF/YJWGs09l30rMsWSjqTeU3U3uEmuOCx72JdYZ2vBkwMt8jng03FPKzpG?= =?us-ascii?Q?y44xg/joUKbyT0ZZjDhP1LpinXzImOKEpksqUC3B+k2SQce4E4WV7LzqmT5i?= =?us-ascii?Q?N8h1mC1z5QTr4qRLK/l+PmlcyThImkoFRNYYlkRQu/ikZpwtOuw/UbMkJvPw?= =?us-ascii?Q?+Vg7vH7ltj+F4LYIjM0goWRmJHA98S8ujO4kG96AnJr1L62OCSWT8QvlO6SJ?= =?us-ascii?Q?vbvPppPRzVNSDDFVYwUDI3G0skV6k3koBjB1D1r6IPmWcntkC/WZrpVzYR+w?= =?us-ascii?Q?wYoGUMv4JVF//Rh0lZL+EjECrhVZjSAkBxEKBRFGo+3qYufTaoDSe1jfxO9Y?= =?us-ascii?Q?UhgsGNQx/54KsvuWCklxL46MVrif8ZiMA85kEXr4uv/ABhpRD6XqRfz+bjw3?= =?us-ascii?Q?ffp//CUBeY0nSSG5WXQLwVRrFgRYGm5VNxt+TrfOmd4jYHTvoLf7pxPRRjXu?= =?us-ascii?Q?bkkNTZ1b+QDM8vuNl0qraQDCxxPN77ayKjohsAlI7Eri+ajdNmdsyiFZ5hHo?= =?us-ascii?Q?bJoyT1MKTn6VbE3k498ZiemlZeLE3Ezma5wgqTEB71mPJSkAp3USPv0yFIZv?= =?us-ascii?Q?AcTHIBpSBb670u0Qu/pgHdjnFs852HYeSBieaee1zj1D2W3dcTxi30khi2ui?= =?us-ascii?Q?0FZXtbQfLhGyO18XplOCN1FoNkYj8Sdlvx9qQEP7McsaMOih1TJnR4ErQmPo?= =?us-ascii?Q?qVAf2QozoBG7bShwRLhLKnzCq05LXyJWNKmr8JIFzbivT+YoSRQJoEtE8tEZ?= =?us-ascii?Q?qjnIDAu2FQ2qTgIc/X41GVJivl8syGIfkYUDPAIh3J+enu+60cVtBylSYB1e?= =?us-ascii?Q?XmkbYCNbYacFbxbPQ/AUmpZVa+OrSEWmZtAOaJQ4Kvyczh/oM+GOBUWBU+X4?= =?us-ascii?Q?Ka1P3BQTjKbol4t8t/S7n5QsvcXU/RwMiWvLzsFMZ62msxohmuHltFrA8u7U?= =?us-ascii?Q?SECnFlY5QqfoLfw8hhk7TpUVKluYyU3Bjy/YpYP8xbXb9NBxiYzxKSjPZKH2?= =?us-ascii?Q?+Zhu9731sZq3QjOEzJXBBNzNnNXi+CCtbD4aqwQYLcpF+69Y8PCg0de7DCYE?= =?us-ascii?Q?VifI0JM/KXBMxW6uDOXTyn240pEPmYnBzPAZjw6z2GVe3fVGrkf2CfhnPXSa?= =?us-ascii?Q?AZZKg5s/6y7ZqP3CCJYvmFgUC7JVBtUoSisZ+29z?= MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: MW4PR11MB5872.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 8ce1aceb-65d8-4e5a-a421-08dc6e5268d7 X-MS-Exchange-CrossTenant-originalarrivaltime: 07 May 2024 04:59:00.4884 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: Kw3VE78LOkp679a+mmj5YC8i8hL7xmJblhEXNgVOANo3INUOu3/xjaP11NHMvhIVvQMwP7XIMPmDecipycob+A== X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH7PR11MB7025 X-OriginatorOrg: intel.com Precedence: Bulk List-Subscribe: List-Help: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Resent-Date: Mon, 06 May 2024 21:59:05 -0700 Resent-From: jiewen.yao@intel.com Reply-To: devel@edk2.groups.io,jiewen.yao@intel.com List-Unsubscribe-Post: List-Unsubscribe=One-Click List-Unsubscribe: X-Gm-Message-State: ALFhTqzqD5jMgYs8k3JDp3fvx7686176AA= Content-Language: en-US Content-Type: multipart/alternative; boundary="_000_MW4PR11MB5872F1715272B1A188AB1CDE8CE42MW4PR11MB5872namp_" X-GND-Status: LEGIT Authentication-Results: spool.mail.gandi.net; dkim=pass header.d=groups.io header.s=20240206 header.b=XngzRwYs; dmarc=fail reason="SPF not aligned (relaxed), DKIM not aligned (relaxed)" header.from=intel.com (policy=none); spf=pass (spool.mail.gandi.net: domain of bounce@groups.io designates 45.79.224.7 as permitted sender) smtp.mailfrom=bounce@groups.io --_000_MW4PR11MB5872F1715272B1A188AB1CDE8CE42MW4PR11MB5872namp_ Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Acked-by: Jiewen Yao From: Wu, Jiaxin Sent: Tuesday, May 7, 2024 11:39 AM To: Ni, Ray ; devel@edk2.groups.io; Ard Biesheuvel ; Yao, Jiewen Cc: Zeng, Star ; Gerd Hoffmann ; Ku= mar, Rahul R ; Dong, Guo ; Rho= des, Sean ; Lu, James ; Guo, Gua= ; Abdul Lateef Attar ; Abner= Chang ; Tom Lendacky Subject: RE: [PATCH v4 00/14] Add SmmRelocationLib Hi Jiewen and Ard, @Yao, Jiewen, @Ard Biesheuvel, do you agree we merge the change related to OVMF package= since you are the OVMF maintainers. Please help check /review. The patches have been acked/tested by the Gerd. [PATCH v4 08/14] OvmfPkg/SmmRelocationLib: Add library instance for OVMF [PATCH v4 09/14] OvmfPkg/PlatformInitLib: Create gEfiSmmSmramMemoryGuid [PATCH v4 10/14] OvmfPkg: Refine SmmAccess implementation [PATCH v4 11/14] OvmfPkg/SmmCpuFeaturesLib: Check Smbase Relocation is do= ne or not [PATCH v4 12/14] OvmfPkg/PlatformPei: Relocate SmBases in PEI phase Thanks, Jiaxin From: Wu, Jiaxin Sent: Tuesday, April 30, 2024 6:14 PM To: Ni, Ray >; devel@edk2.groups.= io Cc: Zeng, Star >; Gerd Hoff= mann >; Kumar, Rahul R >; Dong, Guo >; Rhodes, Sean >; Lu, James >; Guo, Gua >; Ard Bie= sheuvel >; Yao,= Jiewen >; Abdul Lateef A= ttar >; Abner C= hang >; Tom Lendacky > Subject: RE: [PATCH v4 00/14] Add SmmRelocationLib Thanks Ray, I missed to add some already reviewed-by tag in v4. All V4 patc= hes are tested & acted by Gerd: Tested-by: Gerd Hoffmann > Acked-by: Gerd Hoffmann > For each one: listed as below, *so need Ray "reviewed-by" tag on the patch:= 05/06/14, need Gerd "reviewed-by" tag on the patch: 08/09/10/11/12* [PATCH v4 01/14] UefiCpuPkg: Add SmmRelocationLib class Reviewed-by: Ray Ni > --> no cha= nge compared to V3 [PATCH v4 02/14] UefiCpuPkg/SmmRelocationLib: Add SmmRelocationLib librar= y instance Reviewed-by: Ray Ni > --> no chan= ge compared to V3 [PATCH v4 03/14] UefiCpuPkg/SmmRelocationLib: Rename global variables Reviewed-by: Ray Ni > --> no chan= ge compared to V3 [PATCH v4 04/14] UefiCpuPkg/SmmRelocationLib: Avoid unnecessary memory al= location Reviewed-by: Ray Ni > --> no chan= ge compared to V3 [PATCH v4 05/14] UefiCpuPkg/SmmRelocationLib: Remove unnecessary global v= ariable [PATCH v4 06/14] UefiCpuPkg/SmmRelocationLib: Remove unnecessary CpuIndex * Change Based on Ray's comment on V3: split the removal of CpuIndex parame= ter in a new patch, so add the 06 patch in V4.* [PATCH v4 07/14] UefiCpuPkg/SmmRelocationLib: Add library instance for AM= D Reviewed-by: Abdul Lateef Attar > --> no change compared to V3 [PATCH v4 08/14] OvmfPkg/SmmRelocationLib: Add library instance for OVMF [PATCH v4 09/14] OvmfPkg/PlatformInitLib: Create gEfiSmmSmramMemoryGuid [PATCH v4 10/14] OvmfPkg: Refine SmmAccess implementation [PATCH v4 11/14] OvmfPkg/SmmCpuFeaturesLib: Check Smbase Relocation is do= ne or not [PATCH v4 12/14] OvmfPkg/PlatformPei: Relocate SmBases in PEI phase *Change Based on Gerd's comment on V3: 1. Creating the EFI_SMM_SMRAM_MEMOR= Y_GUID HOB should be moved to its own function. 2) refine the comment in S= mmAccess 3) refine the commit log.* [PATCH v4 13/14] UefiPayloadPkg/UefiPayloadPkg.dsc: Include SmmRelocation= Lib Reviewed-by: Gua Guo > Reviewed-by: Guo Dong > [PATCH v4 14/14] UefiCpuPkg/PiSmmCpuDxeSmm: Remove SmBases relocation log= ic *Change Based on Ray's comment on V3: move the "TileSize" check just below = the original TileSize calculation logic* Thanks, Jiaxin From: Ni, Ray > Sent: Tuesday, April 30, 2024 2:01 PM To: Wu, Jiaxin >; devel@edk= 2.groups.io Cc: Zeng, Star >; Gerd Hoff= mann >; Kumar, Rahul R >; Dong, Guo >; Rhodes, Sean >; Lu, James >; Guo, Gua >; Ard Bie= sheuvel >; Yao,= Jiewen >; Abdul Lateef A= ttar >; Abner C= hang >; Tom Lendacky > Subject: Re: [PATCH v4 00/14] Add SmmRelocationLib Jiaxin, You need to add the missing Reviewed-by for each patch if the patch doesn't= change in V4. Can you reply to this mail to list what patches have already got Reviewed-b= y from whom? Thanks, Ray ________________________________ From: Wu, Jiaxin > Sent: Friday, April 26, 2024 20:17 To: devel@edk2.groups.io > Cc: Ni, Ray >; Zeng, Star >; Gerd Hoffmann >; Kumar, Rahul R >; Dong, Guo >; Rhodes, Sean >; Lu, James >; Guo, Gua <= gua.guo@intel.com>; Ard Biesheuvel >; Yao, Jiewen >; Abdul Lateef Attar >; Abner Chang >; Tom Lendacky > Subject: [PATCH v4 00/14] Add SmmRelocationLib PR: https://github.com/tianocore/edk2/pull/5546 Intel plans to separate the smbase relocation logic from PiSmmCpuDxeSmm driver, and the related behavior will be moved to the new interface defined by the SmmRelocationLib class. The SmmRelocationLib class provides the SmmRelocationInit() interface for platform to do the smbase relocation, which shall provide below 2 functionalities: 1. Relocate smbases for each processor. 2. Create the gSmmBaseHobGuid HOB. With SmmRelocationLib, PiSmmCpuDxeSmm driver (which runs at a later phase) can be simplfied as below for SMM init: 1. Consume the gSmmBaseHobGuid HOB for the relocated smbases for each Processor. 2. Execute the early SMM Init. Cc: Ray Ni > Cc: Zeng Star > Cc: Gerd Hoffmann > Cc: Rahul Kumar > Cc: Guo Dong > Cc: Sean Rhodes > Cc: James Lu > Cc: Gua Guo > Cc: Ard Biesheuvel > Cc: Jiewen Yao > Cc: Abdul Lateef Attar > Cc: Abner Chang > Cc: Tom Lendacky > Signed-off-by: Jiaxin Wu > Jiaxin Wu (14): UefiCpuPkg: Add SmmRelocationLib class UefiCpuPkg/SmmRelocationLib: Add SmmRelocationLib library instance UefiCpuPkg/SmmRelocationLib: Rename global variables UefiCpuPkg/SmmRelocationLib: Avoid unnecessary memory allocation UefiCpuPkg/SmmRelocationLib: Remove unnecessary global variable UefiCpuPkg/SmmRelocationLib: Remove unnecessary CpuIndex UefiCpuPkg/SmmRelocationLib: Add library instance for AMD OvmfPkg/SmmRelocationLib: Add library instance for OVMF OvmfPkg/PlatformInitLib: Create gEfiSmmSmramMemoryGuid OvmfPkg: Refine SmmAccess implementation OvmfPkg/SmmCpuFeaturesLib: Check Smbase Relocation is done or not OvmfPkg/PlatformPei: Relocate SmBases in PEI phase UefiPayloadPkg/UefiPayloadPkg.dsc: Include SmmRelocationLib UefiCpuPkg/PiSmmCpuDxeSmm: Remove SmBases relocation logic OvmfPkg/AmdSev/AmdSevX64.dsc | 1 + OvmfPkg/CloudHv/CloudHvX64.dsc | 1 + OvmfPkg/Library/PlatformInitLib/MemDetect.c | 135 +++-- .../Library/PlatformInitLib/PlatformInitLib.inf | 6 +- .../Library/SmmCpuFeaturesLib/SmmCpuFeaturesLib.c | 33 +- .../Library/SmmRelocationLib}/Ia32/Semaphore.c | 13 +- .../Library/SmmRelocationLib}/Ia32/SmmInit.nasm | 83 +++- .../SmmRelocationLib/InternalSmmRelocationLib.h | 127 +++++ .../Library/SmmRelocationLib/SmmRelocationLib.c | 549 +++++++++++++++++= ++++ .../Library/SmmRelocationLib/SmmRelocationLib.inf | 60 +++ .../SmmRelocationLib/SmramSaveStateConfig.c | 100 ++++ .../Library/SmmRelocationLib}/X64/Semaphore.c | 13 +- .../Library/SmmRelocationLib}/X64/SmmInit.nasm | 85 +++- OvmfPkg/Microvm/MicrovmX64.dsc | 1 + OvmfPkg/OvmfPkgIa32.dsc | 1 + OvmfPkg/OvmfPkgIa32X64.dsc | 1 + OvmfPkg/OvmfPkgX64.dsc | 1 + OvmfPkg/PlatformPei/Platform.c | 3 + OvmfPkg/PlatformPei/Platform.h | 5 + OvmfPkg/PlatformPei/PlatformPei.inf | 5 +- OvmfPkg/PlatformPei/SmmRelocation.c | 80 +++ OvmfPkg/SmmAccess/SmmAccess2Dxe.c | 4 +- OvmfPkg/SmmAccess/SmmAccess2Dxe.inf | 5 + OvmfPkg/SmmAccess/SmmAccessPei.c | 116 ++--- OvmfPkg/SmmAccess/SmmAccessPei.inf | 11 +- OvmfPkg/SmmAccess/SmramInternal.c | 72 +-- OvmfPkg/SmmAccess/SmramInternal.h | 19 +- UefiCpuPkg/Include/Library/SmmRelocationLib.h | 42 ++ .../SmmRelocationLib/AmdSmmRelocationLib.inf | 60 +++ .../SmmRelocationLib/AmdSmramSaveStateConfig.c | 125 +++++ .../SmmRelocationLib}/Ia32/Semaphore.c | 13 +- .../SmmRelocationLib}/Ia32/SmmInit.nasm | 83 +++- .../SmmRelocationLib/InternalSmmRelocationLib.h | 127 +++++ .../Library/SmmRelocationLib/SmmRelocationLib.c | 549 +++++++++++++++++= ++++ .../Library/SmmRelocationLib/SmmRelocationLib.inf | 61 +++ .../SmmRelocationLib/SmramSaveStateConfig.c | 136 +++++ .../SmmRelocationLib}/X64/Semaphore.c | 13 +- .../SmmRelocationLib}/X64/SmmInit.nasm | 85 +++- UefiCpuPkg/PiSmmCpuDxeSmm/CpuS3.c | 21 +- UefiCpuPkg/PiSmmCpuDxeSmm/MpService.c | 10 +- UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.c | 334 ++----------- UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.h | 103 +--- UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.inf | 4 - UefiCpuPkg/PiSmmCpuDxeSmm/SmmMpPerf.h | 2 +- UefiCpuPkg/PiSmmCpuDxeSmm/SmramSaveState.c | 69 --- UefiCpuPkg/UefiCpuPkg.dec | 3 + UefiCpuPkg/UefiCpuPkg.dsc | 2 + UefiPayloadPkg/UefiPayloadPkg.dsc | 2 + 48 files changed, 2578 insertions(+), 796 deletions(-) copy {UefiCpuPkg/PiSmmCpuDxeSmm =3D> OvmfPkg/Library/SmmRelocationLib}/Ia3= 2/Semaphore.c (71%) copy {UefiCpuPkg/PiSmmCpuDxeSmm =3D> OvmfPkg/Library/SmmRelocationLib}/Ia3= 2/SmmInit.nasm (47%) create mode 100644 OvmfPkg/Library/SmmRelocationLib/InternalSmmRelocationL= ib.h create mode 100644 OvmfPkg/Library/SmmRelocationLib/SmmRelocationLib.c create mode 100644 OvmfPkg/Library/SmmRelocationLib/SmmRelocationLib.inf create mode 100644 OvmfPkg/Library/SmmRelocationLib/SmramSaveStateConfig.c copy {UefiCpuPkg/PiSmmCpuDxeSmm =3D> OvmfPkg/Library/SmmRelocationLib}/X64= /Semaphore.c (78%) copy {UefiCpuPkg/PiSmmCpuDxeSmm =3D> OvmfPkg/Library/SmmRelocationLib}/X64= /SmmInit.nasm (59%) create mode 100644 OvmfPkg/PlatformPei/SmmRelocation.c create mode 100644 UefiCpuPkg/Include/Library/SmmRelocationLib.h create mode 100644 UefiCpuPkg/Library/SmmRelocationLib/AmdSmmRelocationLib= .inf create mode 100644 UefiCpuPkg/Library/SmmRelocationLib/AmdSmramSaveStateCo= nfig.c rename UefiCpuPkg/{PiSmmCpuDxeSmm =3D> Library/SmmRelocationLib}/Ia32/Sema= phore.c (71%) rename UefiCpuPkg/{PiSmmCpuDxeSmm =3D> Library/SmmRelocationLib}/Ia32/SmmI= nit.nasm (47%) create mode 100644 UefiCpuPkg/Library/SmmRelocationLib/InternalSmmRelocati= onLib.h create mode 100644 UefiCpuPkg/Library/SmmRelocationLib/SmmRelocationLib.c create mode 100644 UefiCpuPkg/Library/SmmRelocationLib/SmmRelocationLib.in= f create mode 100644 UefiCpuPkg/Library/SmmRelocationLib/SmramSaveStateConfi= g.c rename UefiCpuPkg/{PiSmmCpuDxeSmm =3D> Library/SmmRelocationLib}/X64/Semap= hore.c (78%) rename UefiCpuPkg/{PiSmmCpuDxeSmm =3D> Library/SmmRelocationLib}/X64/SmmIn= it.nasm (59%) -- 2.16.2.windows.1 -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#118634): https://edk2.groups.io/g/devel/message/118634 Mute This Topic: https://groups.io/mt/105748911/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- --_000_MW4PR11MB5872F1715272B1A188AB1CDE8CE42MW4PR11MB5872namp_ Content-Type: text/html; charset="us-ascii" Content-Transfer-Encoding: quoted-printable

Acked-by: Jiewen Yao <Jiewen.yao@intel.com>

 

From: Wu, Jiaxin <jiaxin.wu@intel.com> =
Sent: Tuesday, May 7, 2024 11:39 AM
To: Ni, Ray <ray.ni@intel.com>; devel@edk2.groups.io; Ard Bies= heuvel <ardb+tianocore@kernel.org>; Yao, Jiewen <jiewen.yao@intel.= com>
Cc: Zeng, Star <star.zeng@intel.com>; Gerd Hoffmann <kraxel= @redhat.com>; Kumar, Rahul R <rahul.r.kumar@intel.com>; Dong, Guo = <guo.dong@intel.com>; Rhodes, Sean <sean@starlabs.systems>; Lu,= James <james.lu@intel.com>; Guo, Gua <gua.guo@intel.com>; Abdu= l Lateef Attar <AbdulLateef.Attar@amd.com>; Abner Chang <abner.chan= g@amd.com>; Tom Lendacky <thomas.lendacky@amd.com>
Subject: RE: [PATCH v4 00/14] Add SmmRelocationLib

 

Hi Jiewen and Ard,

 

@Yao, Jiewen, @Ard Biesheuvel, do you agree we merge the change related t= o OVMF package since you are the OVMF maintainers. Please help check /revie= w.

 

The patches have been acked/tested by the Gerd.=

 

  [PATCH v4 08/14] OvmfPkg/SmmRelocationLib: Ad= d library instance for OVMF
  [PATCH v4 09/14] OvmfPkg/PlatformInitLib: Create gEfiSmmSmramMemoryG= uid
  [PATCH v4 10/14]  OvmfPkg: Refine SmmAccess implementation
  [PATCH v4 11/14] OvmfPkg/SmmCpuFeaturesLib: Check Smbase Relocation = is done or not
  [PATCH v4 12/14] OvmfPkg/PlatformPei: Relocate SmBases in PEI phase<= o:p>

 

Thanks,

Jiaxin

 

From: Wu, Jiaxin
Sent: Tuesday, April 30, 2024 6:14 PM
To: Ni, Ray <ray.ni@intel.com= >; devel@edk2.groups.io
Cc: Zeng, Star <star.zeng@= intel.com>; Gerd Hoffmann <k= raxel@redhat.com>; Kumar, Rahul R <rahul.r.kumar@intel.com>; Dong, Guo <guo.dong@intel.com>; Rhod= es, Sean <sean@starlabs.systems= >; Lu, James <james.lu@inte= l.com>; Guo, Gua <gua.guo@in= tel.com>; Ard Biesheuvel <ardb+tiano= core@kernel.org>; Yao, Jiewen <jiewen.yao@intel.com>; Abdul Lateef Attar <AbdulLateef.Attar@amd.com>; Abner Chang <abner.chang@amd.com= >; Tom Lendacky <thoma= s.lendacky@amd.com>
Subject: RE: [PATCH v4 00/14] Add SmmRelocationLib

 

Thanks Ray, I missed to add some already reviewed-by= tag in v4. All V4 patches are tested & acted by Gerd:

Tested-by: Gerd Hoffmann <kraxel@redhat.com>

Acked-by: Gerd Hoffmann <kraxel@redhat.com>

 

For each one: listed as below, *so need Ray ̶= 0;reviewed-by” tag on the patch: 05/06/14, need Gerd “reviewed-= by” tag on the patch:  08/09/10/11/12*

 

  [PATCH v4 01/14] UefiCpuPkg: Add SmmRelocatio= nLib class

Reviewed-by: Ray Ni <ray.ni@intel.com>  --> no change compared to V3=


  [PATCH v4 02/14] UefiCpuPkg/SmmRelocationLib: Add SmmRelocationLib l= ibrary instance

Reviewed-by: Ray Ni <ray.ni@intel.com> --> no change compared to V3


  [PATCH v4 03/14] UefiCpuPkg/SmmRelocationLib: Rename global variable= s

Reviewed-by: Ray Ni <ray.ni@intel.com> --> no change compared to V3


 [PATCH v4 04/14]  UefiCpuPkg/SmmRelocationLib: Avoid unnecessary= memory allocation

Reviewed-by: Ray Ni <ray.ni@intel.com> --> no change compared to V3


  [PATCH v4 05/14] UefiCpuPkg/SmmRelocationLib: Remove unnecessary glo= bal variable
  [PATCH v4 06/14] UefiCpuPkg/SmmRelocationLib: Remove unnecessary Cpu= Index

* Change Based on Ray’s comment on V3: = spl= it the removal of CpuIndex parameter in a new patch, so add the 06 patch in= V4.*


  [PATCH v4 07/14] UefiCpuPkg/SmmRelocationLib: Add library instance f= or AMD

Reviewed-by: Abdul Lateef Attar <AbdulLateef.Atta= r@amd.com> --> no change compared to V3


  [PATCH v4 08/14] OvmfPkg/SmmRelocationLib: Add library instance for = OVMF
  [PATCH v4 09/14] OvmfPkg/PlatformInitLib: Create gEfiSmmSmramMemoryG= uid
  [PATCH v4 10/14]  OvmfPkg: Refine SmmAccess implementation
  [PATCH v4 11/14] OvmfPkg/SmmCpuFeaturesLib: Check Smbase Relocation = is done or not
  [PATCH v4 12/14] OvmfPkg/PlatformPei: Relocate SmBases in PEI phase<= o:p>

*Change Based on Gerd’s  comment on V3= : 1. Creating the EFI_SMM_SMRAM_MEMORY_GUID HOB should be moved to its own = function.  2) refine the comment in SmmAccess 3) refine the commit log= .*


  [PATCH v4 13/14] UefiPayloadPkg/UefiPayloadPkg.dsc: Include SmmReloc= ationLib

Reviewed-by: Gua Guo <gua.guo@intel.com>

Reviewed-by: Guo Dong <guo.dong@intel.com><= o:p>


  [PATCH v4 14/14] UefiCpuPkg/PiSmmCpuDxeSmm: Remove SmBases relocatio= n logic

*Change Based on Ray’s comment on V3: <= b>move= the "TileSize" check just below the original TileSize calculatio= n logic*

 

Thanks,

Jiaxin

 

From: Ni, Ray <ray.ni@intel.com>
Sent: Tuesday, April 30, 2024 2:01 PM
To: Wu, Jiaxin <jiaxin.wu@= intel.com>; devel@edk2.groups.io
Cc: Zeng, Star <star.zeng@= intel.com>; Gerd Hoffmann <k= raxel@redhat.com>; Kumar, Rahul R <rahul.r.kumar@intel.com>; Dong, Guo <guo.dong@intel.com>; Rhod= es, Sean <sean@starlabs.systems= >; Lu, James <james.lu@inte= l.com>; Guo, Gua <gua.guo@in= tel.com>; Ard Biesheuvel <ardb+tiano= core@kernel.org>; Yao, Jiewen <jiewen.yao@intel.com>; Abdul Lateef Attar <AbdulLateef.Attar@amd.com>; Abner Chang <abner.chang@amd.com= >; Tom Lendacky <thoma= s.lendacky@amd.com>
Subject: Re: [PATCH v4 00/14] Add SmmRelocationLib

 

Jiaxin,

You need to add the missing Reviewed-by for each patch if = the patch doesn't change in V4.

 

Can you reply to this mail to list what patches have alrea= dy got Reviewed-by from whom?

 

 

 

Thanks,

Ray


From: Wu, Jiaxin <jiaxin.wu@intel.com>
Sent: Friday, April 26, 2024 20:17
To: devel@edk2.groups.io= <devel@edk2.groups.io> Cc: Ni, Ray <ray.ni@intel.com= >; Zeng, Star <star.zeng@i= ntel.com>; Gerd Hoffmann <kr= axel@redhat.com>; Kumar, Rahul R <rahul.r.kumar@intel.com>; Dong, Guo <guo.dong@intel.com= >; Rhodes, Sean <sean@starla= bs.systems>; Lu, James <jam= es.lu@intel.com>; Guo, Gua <= gua.guo@intel.com>; Ard Biesheuvel <ardb+tiano= core@kernel.org>; Yao, Jiewen <jiewen.yao@intel.com>; Abdul Lateef Attar <AbdulLateef.Attar@amd.com>; Abner Chang <abner.chang@amd.com= >; Tom Lendacky <thoma= s.lendacky@amd.com>
Subject: [PATCH v4 00/14] Add SmmRelocationLib

 

PR: https://github.com/tianocore/edk2/pull/5546

Intel plans to separate the smbase relocation logic from
PiSmmCpuDxeSmm driver, and the related behavior will be
moved to the new interface defined by the SmmRelocationLib
class.

The SmmRelocationLib class provides the SmmRelocationInit()
interface for platform to do the smbase relocation, which
shall provide below 2 functionalities:
1. Relocate smbases for each processor.
2. Create the gSmmBaseHobGuid HOB.

With SmmRelocationLib, PiSmmCpuDxeSmm driver (which runs at
a later phase) can be simplfied as below for SMM init:
1. Consume the gSmmBaseHobGuid HOB for the relocated smbases
for each Processor.
2. Execute the early SMM Init.

Cc: Ray Ni <ray.ni@intel.com>=
Cc: Zeng Star <star.zeng@intel.co= m>
Cc: Gerd Hoffmann <kraxel@redhat.co= m>
Cc: Rahul Kumar <rahul1.kumar@= intel.com>
Cc: Guo Dong <guo.dong@intel.com>
Cc: Sean Rhodes <
sean@starlabs.= systems>
Cc: James Lu <james.lu@intel.com>
Cc: Gua Guo <
gua.guo@intel.com&= gt;
Cc: Ard Biesheuvel <ardb+ti= anocore@kernel.org>
Cc: Jiewen Yao <jiewen.yao@intel= .com>
Cc: Abdul Lateef Attar <Abd= ulLateef.Attar@amd.com>
Cc: Abner Chang <abner.chang@amd.= com>
Cc: Tom Lendacky <thomas.lend= acky@amd.com>
Signed-off-by: Jiaxin Wu <jiaxin.= wu@intel.com>

Jiaxin Wu (14):
  UefiCpuPkg: Add SmmRelocationLib class
  UefiCpuPkg/SmmRelocationLib: Add SmmRelocationLib library instance   UefiCpuPkg/SmmRelocationLib: Rename global variables
  UefiCpuPkg/SmmRelocationLib: Avoid unnecessary memory allocation
  UefiCpuPkg/SmmRelocationLib: Remove unnecessary global variable
  UefiCpuPkg/SmmRelocationLib: Remove unnecessary CpuIndex
  UefiCpuPkg/SmmRelocationLib: Add library instance for AMD
  OvmfPkg/SmmRelocationLib: Add library instance for OVMF
  OvmfPkg/PlatformInitLib: Create gEfiSmmSmramMemoryGuid
  OvmfPkg: Refine SmmAccess implementation
  OvmfPkg/SmmCpuFeaturesLib: Check Smbase Relocation is done or not   OvmfPkg/PlatformPei: Relocate SmBases in PEI phase
  UefiPayloadPkg/UefiPayloadPkg.dsc: Include SmmRelocationLib
  UefiCpuPkg/PiSmmCpuDxeSmm: Remove SmBases relocation logic

 OvmfPkg/AmdSev/AmdSevX64.dsc       = ;            &n= bsp;   |   1 +
 OvmfPkg/CloudHv/CloudHvX64.dsc      &nb= sp;            =   |   1 +
 OvmfPkg/Library/PlatformInitLib/MemDetect.c    &n= bsp;   | 135 +++--
 .../Library/PlatformInitLib/PlatformInitLib.inf    |&n= bsp;  6 +-
 .../Library/SmmCpuFeaturesLib/SmmCpuFeaturesLib.c  |  33 +-=
 .../Library/SmmRelocationLib}/Ia32/Semaphore.c    = ; |  13 +-
 .../Library/SmmRelocationLib}/Ia32/SmmInit.nasm    |&n= bsp; 83 +++-
 .../SmmRelocationLib/InternalSmmRelocationLib.h    | 1= 27 +++++
 .../Library/SmmRelocationLib/SmmRelocationLib.c    | 5= 49 +++++++++++++++++++++
 .../Library/SmmRelocationLib/SmmRelocationLib.inf  |  60 ++= +
 .../SmmRelocationLib/SmramSaveStateConfig.c    &n= bsp;   | 100 ++++
 .../Library/SmmRelocationLib}/X64/Semaphore.c    =   |  13 +-
 .../Library/SmmRelocationLib}/X64/SmmInit.nasm    = ; |  85 +++-
 OvmfPkg/Microvm/MicrovmX64.dsc      &nb= sp;            =   |   1 +
 OvmfPkg/OvmfPkgIa32.dsc       &nbs= p;            &= nbsp;       |   1 +
 OvmfPkg/OvmfPkgIa32X64.dsc       &= nbsp;           &nbs= p;     |   1 +
 OvmfPkg/OvmfPkgX64.dsc        = ;            &n= bsp;        |   1 +
 OvmfPkg/PlatformPei/Platform.c      &nb= sp;            =   |   3 +
 OvmfPkg/PlatformPei/Platform.h      &nb= sp;            =   |   5 +
 OvmfPkg/PlatformPei/PlatformPei.inf     &nbs= p;          |   5 +-=
 OvmfPkg/PlatformPei/SmmRelocation.c     &nbs= p;          |  80 +++
 OvmfPkg/SmmAccess/SmmAccess2Dxe.c      =             | &= nbsp; 4 +-
 OvmfPkg/SmmAccess/SmmAccess2Dxe.inf     &nbs= p;          |   5 +<= br>  OvmfPkg/SmmAccess/SmmAccessPei.c      &= nbsp;            | 1= 16 ++---
 OvmfPkg/SmmAccess/SmmAccessPei.inf      = ;           |  11 +-=
 OvmfPkg/SmmAccess/SmramInternal.c      =             |  = 72 +--
 OvmfPkg/SmmAccess/SmramInternal.h      =             |  = 19 +-
 UefiCpuPkg/Include/Library/SmmRelocationLib.h    =   |  42 ++
 .../SmmRelocationLib/AmdSmmRelocationLib.inf    &= nbsp;  |  60 +++
 .../SmmRelocationLib/AmdSmramSaveStateConfig.c    = ; | 125 +++++
 .../SmmRelocationLib}/Ia32/Semaphore.c     &= nbsp;       |  13 +-
 .../SmmRelocationLib}/Ia32/SmmInit.nasm     =        |  83 +++-
 .../SmmRelocationLib/InternalSmmRelocationLib.h    | 1= 27 +++++
 .../Library/SmmRelocationLib/SmmRelocationLib.c    | 5= 49 +++++++++++++++++++++
 .../Library/SmmRelocationLib/SmmRelocationLib.inf  |  61 ++= +
 .../SmmRelocationLib/SmramSaveStateConfig.c    &n= bsp;   | 136 +++++
 .../SmmRelocationLib}/X64/Semaphore.c     &n= bsp;        |  13 +-
 .../SmmRelocationLib}/X64/SmmInit.nasm     &= nbsp;       |  85 +++-
 UefiCpuPkg/PiSmmCpuDxeSmm/CpuS3.c      =             |  = 21 +-
 UefiCpuPkg/PiSmmCpuDxeSmm/MpService.c     &n= bsp;        |  10 +-
 UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.c    &nb= sp;    | 334 ++-----------
 UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.h    &nb= sp;    | 103 +---
 UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.inf    &= nbsp;  |   4 -
 UefiCpuPkg/PiSmmCpuDxeSmm/SmmMpPerf.h     &n= bsp;        |   2 +-
 UefiCpuPkg/PiSmmCpuDxeSmm/SmramSaveState.c    &nb= sp;    |  69 ---
 UefiCpuPkg/UefiCpuPkg.dec       &n= bsp;            = ;      |   3 +
 UefiCpuPkg/UefiCpuPkg.dsc       &n= bsp;            = ;      |   2 +
 UefiPayloadPkg/UefiPayloadPkg.dsc      =             | &= nbsp; 2 +
 48 files changed, 2578 insertions(+), 796 deletions(-)
 copy {UefiCpuPkg/PiSmmCpuDxeSmm =3D> OvmfPkg/Library/SmmRelocation= Lib}/Ia32/Semaphore.c (71%)
 copy {UefiCpuPkg/PiSmmCpuDxeSmm =3D> OvmfPkg/Library/SmmRelocation= Lib}/Ia32/SmmInit.nasm (47%)
 create mode 100644 OvmfPkg/Library/SmmRelocationLib/InternalSmmReloca= tionLib.h
 create mode 100644 OvmfPkg/Library/SmmRelocationLib/SmmRelocationLib.= c
 create mode 100644 OvmfPkg/Library/SmmRelocationLib/SmmRelocationLib.= inf
 create mode 100644 OvmfPkg/Library/SmmRelocationLib/SmramSaveStateCon= fig.c
 copy {UefiCpuPkg/PiSmmCpuDxeSmm =3D> OvmfPkg/Library/SmmRelocation= Lib}/X64/Semaphore.c (78%)
 copy {UefiCpuPkg/PiSmmCpuDxeSmm =3D> OvmfPkg/Library/SmmRelocation= Lib}/X64/SmmInit.nasm (59%)
 create mode 100644 OvmfPkg/PlatformPei/SmmRelocation.c
 create mode 100644 UefiCpuPkg/Include/Library/SmmRelocationLib.h
 create mode 100644 UefiCpuPkg/Library/SmmRelocationLib/AmdSmmRelocati= onLib.inf
 create mode 100644 UefiCpuPkg/Library/SmmRelocationLib/AmdSmramSaveSt= ateConfig.c
 rename UefiCpuPkg/{PiSmmCpuDxeSmm =3D> Library/SmmRelocationLib}/I= a32/Semaphore.c (71%)
 rename UefiCpuPkg/{PiSmmCpuDxeSmm =3D> Library/SmmRelocationLib}/I= a32/SmmInit.nasm (47%)
 create mode 100644 UefiCpuPkg/Library/SmmRelocationLib/InternalSmmRel= ocationLib.h
 create mode 100644 UefiCpuPkg/Library/SmmRelocationLib/SmmRelocationL= ib.c
 create mode 100644 UefiCpuPkg/Library/SmmRelocationLib/SmmRelocationL= ib.inf
 create mode 100644 UefiCpuPkg/Library/SmmRelocationLib/SmramSaveState= Config.c
 rename UefiCpuPkg/{PiSmmCpuDxeSmm =3D> Library/SmmRelocationLib}/X= 64/Semaphore.c (78%)
 rename UefiCpuPkg/{PiSmmCpuDxeSmm =3D> Library/SmmRelocationLib}/X= 64/SmmInit.nasm (59%)

--
2.16.2.windows.1

_._,_._,_

Groups.io Links:

=20 You receive all messages sent to this group. =20 =20

View/Reply Online (#118634) | =20 | Mute= This Topic | New Topic
Your Subscriptio= n | Contact Group Owner | Unsubscribe [rebecca@openfw.io]

_._,_._,_
--_000_MW4PR11MB5872F1715272B1A188AB1CDE8CE42MW4PR11MB5872namp_--