From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by mx.groups.io with SMTP id smtpd.web12.8154.1580970093692952246 for ; Wed, 05 Feb 2020 22:21:34 -0800 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 192.55.52.120, mailfrom: hao.a.wu@intel.com) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga002.jf.intel.com ([10.7.209.21]) by fmsmga104.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 05 Feb 2020 22:21:33 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.70,408,1574150400"; d="scan'208";a="249960152" Received: from fmsmsx103.amr.corp.intel.com ([10.18.124.201]) by orsmga002.jf.intel.com with ESMTP; 05 Feb 2020 22:21:32 -0800 Received: from fmsmsx602.amr.corp.intel.com (10.18.126.82) by FMSMSX103.amr.corp.intel.com (10.18.124.201) with Microsoft SMTP Server (TLS) id 14.3.439.0; Wed, 5 Feb 2020 22:21:32 -0800 Received: from fmsmsx602.amr.corp.intel.com (10.18.126.82) 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.1713.5; Wed, 5 Feb 2020 22:21:32 -0800 Received: from shsmsx108.ccr.corp.intel.com (10.239.4.97) by fmsmsx602.amr.corp.intel.com (10.18.126.82) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256) id 15.1.1713.5 via Frontend Transport; Wed, 5 Feb 2020 22:21:31 -0800 Received: from shsmsx104.ccr.corp.intel.com ([169.254.5.5]) by SHSMSX108.ccr.corp.intel.com ([169.254.8.98]) with mapi id 14.03.0439.000; Thu, 6 Feb 2020 14:21:30 +0800 From: "Wu, Hao A" To: "devel@edk2.groups.io" CC: "Kubacki, Michael A" , "Kinney, Michael D" , "Dong, Eric" , "Ni, Ray" , Laszlo Ersek Subject: Re: [PATCH v1 0/2] Fix backward incompatible CPU_MP_DATA struct change Thread-Topic: [PATCH v1 0/2] Fix backward incompatible CPU_MP_DATA struct change Thread-Index: AQHV3K2kT1aejy6hp0iNsG325jpF96gNor3w Date: Thu, 6 Feb 2020 06:21:29 +0000 Message-ID: References: <20200206052356.3672-1-hao.a.wu@intel.com> In-Reply-To: <20200206052356.3672-1-hao.a.wu@intel.com> Accept-Language: zh-CN, en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Return-Path: hao.a.wu@intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable > -----Original Message----- > From: Wu, Hao A > Sent: Thursday, February 06, 2020 1:24 PM > To: devel@edk2.groups.io > Cc: Wu, Hao A; Kubacki, Michael A; Kinney, Michael D; Dong, Eric; Ni, Ray= ; > Laszlo Ersek > Subject: [PATCH v1 0/2] Fix backward incompatible CPU_MP_DATA struct > change >=20 > The series will resolve a backward compatibility issue with pre-built > binaries (e.g. FSP) introduced by commit 88bd0661661. >=20 > The relocation of 'MicrocodePatchRegionSize' and 'MicrocodePatchAddress' > fields in structure CPU_MP_DATA may cause access issue for platforms that > use pre-built FSP binary, since the offset of these microcode related > fields in CPU_MP_DATA can be different between PEI phase (in the pre-buil= t > binary) and DXE phase (in current code implementation). >=20 > The series will use the newly introduced EDKII microcode patch HOB instea= d > for the DXE phase to get the information of the loaded (done in PEI phase= ) > microcode patches data. Sorry, I forgot to mention that The series is also available at: https://github.com/hwu25/edk2/tree/patch_mpinitlib_fsp_v1 Tests done for the series: 1. OS boot successfully on platform (not using FSP binary) when the EDKII microcode patch HOB is produced. Debug messages show that the microcode detection and application work properly. 2. OS boot successfully on platform (not using FSP binary) when the EDKII microcode patch HOB is NOT produced. Debug messages show that the microc= ode detection and application work properly. Note: At this moment, I am not able to verify the OS boot on platform that uses pre-built FSP binary. I am seeking help to verify this case. Best Regards, Hao Wu >=20 > Cc: Michael Kubacki > Cc: Michael D Kinney > Cc: Eric Dong > Cc: Ray Ni > Cc: Laszlo Ersek >=20 > Hao A Wu (2): > Revert UefiCpuPkg/MpInitLib: Relocate microcode patch fields in > CPU_MP_DATA > UefiCpuPkg/MpInitLib: Not pass microcode info between archs in > CPU_MP_DATA >=20 > UefiCpuPkg/Library/MpInitLib/DxeMpInitLib.inf | 3 +- > UefiCpuPkg/Library/MpInitLib/MpLib.h | 27 +++++++++++- > UefiCpuPkg/Library/MpInitLib/Microcode.c | 43 ++++++++++++++++++++ > UefiCpuPkg/Library/MpInitLib/MpLib.c | 20 +++++---- > UefiCpuPkg/Library/MpInitLib/PeiMpLib.c | 3 +- > 5 files changed, 82 insertions(+), 14 deletions(-) >=20 > -- > 2.12.0.windows.1