From nobody Sat Feb 7 04:47:04 2026 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) client-ip=66.175.222.108; envelope-from=bounce+27952+111687+1787277+3901457@groups.io; helo=mail02.groups.io; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+111687+1787277+3901457@groups.io; dmarc=fail(p=none dis=none) header.from=intel.com ARC-Seal: i=1; a=rsa-sha256; t=1700794604; cv=none; d=zohomail.com; s=zohoarc; b=hAwmbTer4ttP1xVqKHZuNvUzX+X9KjK73JsVA0DAGHBCzYW/KJH8pGjmBJE1tx6KTGIJbyREp7+BLtEempaxMtvC2618vb/tkN+W+gjiaOeFw4OFpxvks/RyIclvpYW8sWtYr/BAH9Nw/YH6D8l4zDFcid9HpnF3uWOBZNQlgCU= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1700794604; h=Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:List-Subscribe:List-Id:List-Help:List-Unsubscribe:MIME-Version:Message-ID:Reply-To:Reply-To:References:Sender:Subject:Subject:To:To:Message-Id; bh=0Hx3vCWN1bYV+lc6JF83oMECo4MpSFdDSfU2u7lj+Tk=; b=nJzO5Le3EUI4o5Pd8c2YLqRhT8rogGKUkcxleGQiAVTTL2oXtjt4UstCrsONOW3yUMDwLhXUat4s7AZkeGNYM79bsyww+VTjDU+HauihT63FFRFM0pRul4RCb1+Y8baCrD50RPtwt+e2wttMqorJpJxxAlT3djnHy7TqLsGYBVs= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+111687+1787277+3901457@groups.io; dmarc=fail header.from= (p=none dis=none) Received: from mail02.groups.io (mail02.groups.io [66.175.222.108]) by mx.zohomail.com with SMTPS id 17007946040451012.5118785184854; Thu, 23 Nov 2023 18:56:44 -0800 (PST) Return-Path: DKIM-Signature: a=rsa-sha256; bh=LphvU15hfpNmLWCEqaFWZzJVOtbGVtFfIu5AvMHeCZk=; c=relaxed/simple; d=groups.io; h=From:To:Cc:Subject:Date:Message-Id:In-Reply-To:References:MIME-Version:Precedence:List-Subscribe:List-Help:Sender:List-Id:Mailing-List:Delivered-To:Reply-To:List-Unsubscribe-Post:List-Unsubscribe:Content-Transfer-Encoding; s=20140610; t=1700794603; v=1; b=Mf1roj9N7YpiHo11kPbjFR4cKMV37gfykEHVd7fIRpLkF9dyy5gmjQGGkbDs5IXd/R9BrJdi FNAH8Uh+qb3eDUgYuaUf/im41WyJp8g9cZqhAUx9PVbHiyU1Ohf+54p1TDBxTKSrn2xA7WjTgML NV3d8T7m/SMhpKIPL7xSqkjM= X-Received: by 127.0.0.2 with SMTP id 4uNUYY1788612x0G8lxHGVuK; Thu, 23 Nov 2023 18:56:43 -0800 X-Received: from mgamail.intel.com (mgamail.intel.com [134.134.136.126]) by mx.groups.io with SMTP id smtpd.web11.128508.1700794600506959702 for ; Thu, 23 Nov 2023 18:56:43 -0800 X-IronPort-AV: E=McAfee;i="6600,9927,10902"; a="377386849" X-IronPort-AV: E=Sophos;i="6.04,223,1695711600"; d="scan'208";a="377386849" X-Received: from orsmga001.jf.intel.com ([10.7.209.18]) by orsmga106.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 23 Nov 2023 18:56:43 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10902"; a="802087268" X-IronPort-AV: E=Sophos;i="6.04,223,1695711600"; d="scan'208";a="802087268" X-Received: from shwdeopenlab813.ccr.corp.intel.com ([10.239.55.230]) by orsmga001.jf.intel.com with ESMTP; 23 Nov 2023 18:56:40 -0800 From: "Yuanhao Xie" To: devel@edk2.groups.io Cc: xieyuanh , Ray Ni , Eric Dong , Rahul Kumar , Tom Lendacky , Laszlo Ersek Subject: [edk2-devel] [Patch V4 1/3] UefiCpuPkg/MpInitLib: Eliminate redundant microcode loading in DXE. Date: Fri, 24 Nov 2023 10:56:33 +0800 Message-Id: <20231124025635.3629-2-yuanhao.xie@intel.com> In-Reply-To: <20231124025635.3629-1-yuanhao.xie@intel.com> References: <20231124025635.3629-1-yuanhao.xie@intel.com> MIME-Version: 1.0 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 Reply-To: devel@edk2.groups.io,yuanhao.xie@intel.com List-Unsubscribe-Post: List-Unsubscribe=One-Click List-Unsubscribe: X-Gm-Message-State: SyvGDUAOn2YPZWARONZNu3OTx1787277AA= Content-Transfer-Encoding: quoted-printable X-ZohoMail-DKIM: pass (identity @groups.io) X-ZM-MESSAGEID: 1700794604373100005 Content-Type: text/plain; charset="utf-8" The DXE stage's Microcode loading process has been elimincated by: 1. Let ShadowMicrocodeUpdatePatch and MicrocodeDetect for BSP performed only during the PEI phase. DXE skip those actions. 2. BSP in DXE WakeUpAp only for synchronizing MTRR settings, not loading microcode. Synchronizing the MTRRs setting to the APs is always essential. The modification of the MTRRs on the BSP may happens during timing in the early DXE phase, while the CpuMp PPI service and the CpuMp Protocol both not available. Cc: Ray Ni Cc: Eric Dong Cc: Rahul Kumar Cc: Tom Lendacky Cc: Laszlo Ersek Signed-off-by: Yuanhao Xie --- UefiCpuPkg/Library/MpInitLib/Microcode.c | 46 ----------------------------= ------------------ UefiCpuPkg/Library/MpInitLib/MpLib.c | 39 ++++++++++++++++++++++++----= ----------- UefiCpuPkg/Library/MpInitLib/MpLib.h | 21 --------------------- 3 files changed, 24 insertions(+), 82 deletions(-) diff --git a/UefiCpuPkg/Library/MpInitLib/Microcode.c b/UefiCpuPkg/Library/= MpInitLib/Microcode.c index 11720560af..8faa93024d 100644 --- a/UefiCpuPkg/Library/MpInitLib/Microcode.c +++ b/UefiCpuPkg/Library/MpInitLib/Microcode.c @@ -338,49 +338,3 @@ ShadowMicrocodeUpdatePatch ( ShadowMicrocodePatchByPcd (CpuMpData); } } - -/** - Get the cached microcode patch base address and size from the microcode = patch - information cache HOB. - - @param[out] Address Base address of the microcode patches data. - It will be updated if the microcode patch - information cache HOB is found. - @param[out] RegionSize Size of the microcode patches data. - It will be updated if the microcode patch - information cache HOB is found. - - @retval TRUE The microcode patch information cache HOB is found. - @retval FALSE The microcode patch information cache HOB is not found. - -**/ -BOOLEAN -GetMicrocodePatchInfoFromHob ( - UINT64 *Address, - UINT64 *RegionSize - ) -{ - EFI_HOB_GUID_TYPE *GuidHob; - EDKII_MICROCODE_PATCH_HOB *MicrocodePathHob; - - GuidHob =3D GetFirstGuidHob (&gEdkiiMicrocodePatchHobGuid); - if (GuidHob =3D=3D NULL) { - DEBUG ((DEBUG_INFO, "%a: Microcode patch cache HOB is not found.\n", _= _func__)); - return FALSE; - } - - MicrocodePathHob =3D GET_GUID_HOB_DATA (GuidHob); - - *Address =3D MicrocodePathHob->MicrocodePatchAddress; - *RegionSize =3D MicrocodePathHob->MicrocodePatchRegionSize; - - DEBUG (( - DEBUG_INFO, - "%a: MicrocodeBase =3D 0x%lx, MicrocodeSize =3D 0x%lx\n", - __func__, - *Address, - *RegionSize - )); - - return TRUE; -} diff --git a/UefiCpuPkg/Library/MpInitLib/MpLib.c b/UefiCpuPkg/Library/MpIn= itLib/MpLib.c index 9a6ec5db5c..1fd6440a53 100644 --- a/UefiCpuPkg/Library/MpInitLib/MpLib.c +++ b/UefiCpuPkg/Library/MpInitLib/MpLib.c @@ -451,12 +451,20 @@ ApInitializeSync ( CpuMpData =3D (CPU_MP_DATA *)Buffer; Status =3D GetProcessorNumber (CpuMpData, &ProcessorNumber); ASSERT_EFI_ERROR (Status); + ASSERT (CpuMpData->InitFlag =3D=3D ApInitReconfig || CpuMpData->InitFlag= =3D=3D ApInitDone); + if (CpuMpData->InitFlag !=3D ApInitReconfig) { + // + // Load microcode on AP for PEI phase. + // During the DXE phase, it cannot omitted. + // + MicrocodeDetect (CpuMpData, ProcessorNumber); + } + // - // Load microcode on AP - // - MicrocodeDetect (CpuMpData, ProcessorNumber); - // - // Sync BSP's MTRR table to AP + // Synchronizing the MTRRs setting to the APs is always essential. + // The modification of the MTRRs on the BSP may happens during + // timing in the early DXE phase, while the CpuMp PPI service and + // the CpuMp Protocol are both not available. // MtrrSetAllMtrrs (&CpuMpData->MtrrTable); } @@ -2224,29 +2232,25 @@ MpInitLibInitialize ( } } =20 - if (!GetMicrocodePatchInfoFromHob ( - &CpuMpData->MicrocodePatchAddress, - &CpuMpData->MicrocodePatchRegionSize - )) - { + if (MpHandOff =3D=3D NULL) { // // The microcode patch information cache HOB does not exist, which mea= ns // the microcode patches data has not been loaded into memory yet // ShadowMicrocodeUpdatePatch (CpuMpData); + // + // Detect and apply Microcode on BSP + // + MicrocodeDetect (CpuMpData, CpuMpData->BspNumber); } =20 - // - // Detect and apply Microcode on BSP - // - MicrocodeDetect (CpuMpData, CpuMpData->BspNumber); // // Store BSP's MTRR setting // MtrrGetAllMtrrs (&CpuMpData->MtrrTable); =20 // - // Wakeup APs to do some AP initialize sync (Microcode & MTRR) + // Wakeup APs to do some AP initialize sync (MTRR and/or Microcode). // if (CpuMpData->CpuCount > 1) { if (MpHandOff !=3D NULL) { @@ -2258,6 +2262,11 @@ MpInitLibInitialize ( CpuMpData->InitFlag =3D ApInitReconfig; } =20 + // + // Wake up APs to perform AP initialization synchronization. + // 1. For the PEI stage, load microcode and synchronize MTRRs, + // 2. For the DXE phase, only synchronize MTRRs. + // WakeUpAP (CpuMpData, TRUE, 0, ApInitializeSync, CpuMpData, TRUE); // // Wait for all APs finished initialization diff --git a/UefiCpuPkg/Library/MpInitLib/MpLib.h b/UefiCpuPkg/Library/MpIn= itLib/MpLib.h index 763db4963d..01b5b9c113 100644 --- a/UefiCpuPkg/Library/MpInitLib/MpLib.h +++ b/UefiCpuPkg/Library/MpInitLib/MpLib.h @@ -744,27 +744,6 @@ ShadowMicrocodeUpdatePatch ( IN OUT CPU_MP_DATA *CpuMpData ); =20 -/** - Get the cached microcode patch base address and size from the microcode = patch - information cache HOB. - - @param[out] Address Base address of the microcode patches data. - It will be updated if the microcode patch - information cache HOB is found. - @param[out] RegionSize Size of the microcode patches data. - It will be updated if the microcode patch - information cache HOB is found. - - @retval TRUE The microcode patch information cache HOB is found. - @retval FALSE The microcode patch information cache HOB is not found. - -**/ -BOOLEAN -GetMicrocodePatchInfoFromHob ( - UINT64 *Address, - UINT64 *RegionSize - ); - /** Detect whether Mwait-monitor feature is supported. =20 --=20 2.39.1.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 (#111687): https://edk2.groups.io/g/devel/message/111687 Mute This Topic: https://groups.io/mt/102775771/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-