From nobody Fri Apr 19 18:14:44 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of groups.io designates 66.175.222.12 as permitted sender) client-ip=66.175.222.12; envelope-from=bounce+27952+54407+1787277+3901457@groups.io; helo=web01.groups.io; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.12 as permitted sender) smtp.mailfrom=bounce+27952+54407+1787277+3901457@groups.io; dmarc=fail(p=none dis=none) header.from=intel.com Received: from web01.groups.io (web01.groups.io [66.175.222.12]) by mx.zohomail.com with SMTPS id 1581652822534809.613573141797; Thu, 13 Feb 2020 20:00:22 -0800 (PST) Return-Path: X-Received: by 127.0.0.2 with SMTP id 9fLVYY1788612x6HhaORBCWg; Thu, 13 Feb 2020 20:00:20 -0800 X-Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by mx.groups.io with SMTP id smtpd.web12.1946.1581652818960192032 for ; Thu, 13 Feb 2020 20:00:19 -0800 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False X-Received: from fmsmga008.fm.intel.com ([10.253.24.58]) by orsmga103.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 13 Feb 2020 20:00:17 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.70,439,1574150400"; d="scan'208";a="228363397" X-Received: from shwdeopenpsi787.ccr.corp.intel.com ([10.239.158.56]) by fmsmga008.fm.intel.com with ESMTP; 13 Feb 2020 20:00:16 -0800 From: "Siyuan, Fu" To: devel@edk2.groups.io Cc: Michael D Kinney , Ray Ni , Rangasai V Chaganty Subject: [edk2-devel] [PATCH v4] IntelSiliconPkg: FIT based shadow microcode PPI support. Date: Fri, 14 Feb 2020 12:00:11 +0800 Message-Id: MIME-Version: 1.0 Precedence: Bulk List-Unsubscribe: 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,siyuan.fu@intel.com X-Gm-Message-State: LKRTtY4oCKGU2TFudl9w1vM4x1787277AA= Content-Transfer-Encoding: quoted-printable DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=groups.io; q=dns/txt; s=20140610; t=1581652820; bh=WaKy7lKqTbFlYnoB3lIi35zB/B0K3n4MeVYawfUNEdI=; h=Cc:Date:From:Reply-To:Subject:To; b=urza6YTkiWYkTxSW0xzq5+utdNiKWJ2vSUoO8m/gYjwO6Ojvw78o5CB2UE0oEeGT1vm 7dta4nbVUg3XnsRMq7+qQLGqINcjXIcqJW/CE2DOh3nhkKzrqIDzf3avbtAtPxdTUnEyI R8ULUP+x6CnMMSbKDzhqdZK8AHB28tWbdXw= X-ZohoMail-DKIM: pass (identity @groups.io) Content-Type: text/plain; charset="utf-8" V4 Changes: Adjust EDKII_MICROCODE_SHADOW_INFO_HOB structure definition for better alignment and understanding. Add EFI_MICROCODE_STORAGE_TYPE_FLASH_CONTEXT structure definition. Fix a typo in EFI_MICROCODE_STORAGE_TYPE_FLASH_GUID. Merge ShadowMicrocodePei.h header into c file. Correct file header description and copy right year. V3 Changes: Remove the feature PCD PcdCpuShadowMicrocodeByFit because the whole FIT microcode shadow code is moved to this PEIM so platform could disable this feature by not include PEIM now. V2 Changes: Rename EDKII_PEI_CPU_MICROCODE_ID to EDKII_PEI_MICROCODE_CPU_ID. This patch adds a platform PEIM for FIT based shadow microcode PPI support. A detailed design doc can be found here: https://edk2.groups.io/g/devel/files/Designs/2020/0214/Support%20 the%202nd%20Microcode%20FV%20Flash%20Region.pdf TEST: Tested on FIT enabled platform. BZ: https://tianocore.acgmultimedia.com/show_bug.cgi?id=3D2449 Cc: Michael D Kinney Cc: Ray Ni Cc: Rangasai V Chaganty Signed-off-by: Siyuan Fu --- .../ShadowMicrocode/ShadowMicrocodePei.c | 435 ++++++++++++++++++ .../ShadowMicrocode/ShadowMicrocodePei.inf | 43 ++ .../Include/Guid/MicrocodeShadowInfoHob.h | 64 +++ .../Intel/IntelSiliconPkg/IntelSiliconPkg.dec | 8 +- .../Intel/IntelSiliconPkg/IntelSiliconPkg.dsc | 3 +- 5 files changed, 551 insertions(+), 2 deletions(-) create mode 100644 Silicon/Intel/IntelSiliconPkg/Feature/ShadowMicrocode/S= hadowMicrocodePei.c create mode 100644 Silicon/Intel/IntelSiliconPkg/Feature/ShadowMicrocode/S= hadowMicrocodePei.inf create mode 100644 Silicon/Intel/IntelSiliconPkg/Include/Guid/MicrocodeSha= dowInfoHob.h diff --git a/Silicon/Intel/IntelSiliconPkg/Feature/ShadowMicrocode/ShadowMi= crocodePei.c b/Silicon/Intel/IntelSiliconPkg/Feature/ShadowMicrocode/Shadow= MicrocodePei.c new file mode 100644 index 0000000000..26c5368d1e --- /dev/null +++ b/Silicon/Intel/IntelSiliconPkg/Feature/ShadowMicrocode/ShadowMicrocode= Pei.c @@ -0,0 +1,435 @@ +/** @file + FIT based microcode shadow PEIM. + +Copyright (c) 2020, Intel Corporation. All rights reserved.
+SPDX-License-Identifier: BSD-2-Clause-Patent + +**/ + +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include +// +// Data structure for microcode patch information +// +typedef struct { + UINTN Address; + UINTN Size; +} MICROCODE_PATCH_INFO; + +/** + Shadow microcode update patches to memory. + + The function is used for shadowing microcode update patches to a continu= ous memory. + It shall allocate memory buffer and only shadow the microcode patches fo= r those + processors specified by MicrocodeCpuId array. The checksum verification = may be + skiped in this function so the caller must perform checksum verification= before + using the microcode patches in returned memory buffer. + + @param[in] This The PPI instance pointer. + @param[in] CpuIdCount Number of elements in MicrocodeCpuId ar= ray. + @param[in] MicrocodeCpuId A pointer to an array of EDKII_PEI_MICR= OCODE_CPU_ID + structures. + @param[out] BufferSize Pointer to receive the total size of Bu= ffer. + @param[out] Buffer Pointer to receive address of allocated= memory + with microcode patches data in it. + + @retval EFI_SUCCESS The microcode has been shadowed to memo= ry. + @retval EFI_OUT_OF_RESOURCES The operation fails due to lack of reso= urces. + +**/ +EFI_STATUS +ShadowMicrocode ( + IN EDKII_PEI_SHADOW_MICROCODE_PPI *This, + IN UINTN CpuIdCount, + IN EDKII_PEI_MICROCODE_CPU_ID *MicrocodeCpuId, + OUT UINTN *BufferSize, + OUT VOID **Buffer + ); + + +EDKII_PEI_SHADOW_MICROCODE_PPI mPeiShadowMicrocodePpi =3D { + ShadowMicrocode +}; + + +EFI_PEI_PPI_DESCRIPTOR mPeiShadowMicrocodePpiList[] =3D { + { + EFI_PEI_PPI_DESCRIPTOR_PPI | EFI_PEI_PPI_DESCRIPTOR_TERMINATE_LIST, + &gEdkiiPeiShadowMicrocodePpiGuid, + &mPeiShadowMicrocodePpi + } +}; + +/** + Determine if a microcode patch matchs the specific processor signature a= nd flag. + + @param[in] CpuIdCount Number of elements in MicrocodeCpuId a= rray. + @param[in] MicrocodeCpuId A pointer to an array of EDKII_PEI_MIC= ROCODE_CPU_ID + structures. + @param[in] ProcessorSignature The processor signature field value + supported by a microcode patch. + @param[in] ProcessorFlags The prcessor flags field value support= ed by + a microcode patch. + + @retval TRUE The specified microcode patch will be loaded. + @retval FALSE The specified microcode patch will not be loaded. +**/ +BOOLEAN +IsProcessorMatchedMicrocodePatch ( + IN UINTN CpuIdCount, + IN EDKII_PEI_MICROCODE_CPU_ID *MicrocodeCpuId, + IN UINT32 ProcessorSignature, + IN UINT32 ProcessorFlags + ) +{ + UINTN Index; + + for (Index =3D 0; Index < CpuIdCount; Index++) { + if ((ProcessorSignature =3D=3D MicrocodeCpuId[Index].ProcessorSignatur= e) && + (ProcessorFlags & (1 << MicrocodeCpuId[Index].PlatformId)) !=3D 0)= { + return TRUE; + } + } + + return FALSE; +} + +/** + Check the 'ProcessorSignature' and 'ProcessorFlags' of the microcode + patch header with the CPUID and PlatformID of the processors within + system to decide if it will be copied into memory. + + @param[in] CpuIdCount Number of elements in MicrocodeCpuId a= rray. + @param[in] MicrocodeCpuId A pointer to an array of EDKII_PEI_MIC= ROCODE_CPU_ID + structures. + @param[in] MicrocodeEntryPoint The pointer to the microcode patch hea= der. + + @retval TRUE The specified microcode patch need to be loaded. + @retval FALSE The specified microcode patch dosen't need to be loaded. +**/ +BOOLEAN +IsMicrocodePatchNeedLoad ( + IN UINTN CpuIdCount, + IN EDKII_PEI_MICROCODE_CPU_ID *MicrocodeCpuId, + CPU_MICROCODE_HEADER *MicrocodeEntryPoint + ) +{ + BOOLEAN NeedLoad; + UINTN DataSize; + UINTN TotalSize; + CPU_MICROCODE_EXTENDED_TABLE_HEADER *ExtendedTableHeader; + UINT32 ExtendedTableCount; + CPU_MICROCODE_EXTENDED_TABLE *ExtendedTable; + UINTN Index; + + // + // Check the 'ProcessorSignature' and 'ProcessorFlags' in microcode patc= h header. + // + NeedLoad =3D IsProcessorMatchedMicrocodePatch ( + CpuIdCount, + MicrocodeCpuId, + MicrocodeEntryPoint->ProcessorSignature.Uint32, + MicrocodeEntryPoint->ProcessorFlags + ); + + // + // If the Extended Signature Table exists, check if the processor is in = the + // support list + // + DataSize =3D MicrocodeEntryPoint->DataSize; + TotalSize =3D (DataSize =3D=3D 0) ? 2048 : MicrocodeEntryPoint->TotalSiz= e; + if ((!NeedLoad) && (DataSize !=3D 0) && + (TotalSize - DataSize > sizeof (CPU_MICROCODE_HEADER) + + sizeof (CPU_MICROCODE_EXTENDED_TABLE_HEADER)= )) { + ExtendedTableHeader =3D (CPU_MICROCODE_EXTENDED_TABLE_HEADER *) ((UINT= 8 *) (MicrocodeEntryPoint) + + DataSize + sizeof (CPU_MICROCODE_HEADER)); + ExtendedTableCount =3D ExtendedTableHeader->ExtendedSignatureCount; + ExtendedTable =3D (CPU_MICROCODE_EXTENDED_TABLE *) (ExtendedTabl= eHeader + 1); + + for (Index =3D 0; Index < ExtendedTableCount; Index ++) { + // + // Check the 'ProcessorSignature' and 'ProcessorFlag' of the Extended + // Signature Table entry with the CPUID and PlatformID of the proces= sors + // within system to decide if it will be copied into memory + // + NeedLoad =3D IsProcessorMatchedMicrocodePatch ( + CpuIdCount, + MicrocodeCpuId, + ExtendedTable->ProcessorSignature.Uint32, + ExtendedTable->ProcessorFlag + ); + if (NeedLoad) { + break; + } + ExtendedTable ++; + } + } + + return NeedLoad; +} + +/** + Actual worker function that shadows the required microcode patches into = memory. + + @param[in] Patches The pointer to an array of information= on + the microcode patches that will be loa= ded + into memory. + @param[in] PatchCount The number of microcode patches that w= ill + be loaded into memory. + @param[in] TotalLoadSize The total size of all the microcode pa= tches + to be loaded. + @param[out] BufferSize Pointer to receive the total size of B= uffer. + @param[out] Buffer Pointer to receive address of allocate= d memory + with microcode patches data in it. +**/ +VOID +ShadowMicrocodePatchWorker ( + IN MICROCODE_PATCH_INFO *Patches, + IN UINTN PatchCount, + IN UINTN TotalLoadSize, + OUT UINTN *BufferSize, + OUT VOID **Buffer + ) +{ + UINTN Index; + VOID *MicrocodePatchInRam; + UINT8 *Walker; + EDKII_MICROCODE_SHADOW_INFO_HOB *MicrocodeShadowHob; + UINTN HobDataLength; + UINT64 *MicrocodeAddressInMemory; + EFI_MICROCODE_STORAGE_TYPE_FLASH_CONTEXT *Flashcontext; + + ASSERT ((Patches !=3D NULL) && (PatchCount !=3D 0)); + + // + // Init microcode shadow info HOB content. + // + HobDataLength =3D sizeof (EDKII_MICROCODE_SHADOW_INFO_HOB) + + sizeof (UINT64) * PatchCount * 2; + MicrocodeShadowHob =3D AllocatePool (HobDataLength); + if (MicrocodeShadowHob =3D=3D NULL) { + ASSERT (FALSE); + return; + } + MicrocodeShadowHob->MicrocodeCount =3D PatchCount; + CopyGuid ( + &MicrocodeShadowHob->StorageType, + &gEdkiiMicrocodeStorageTypeFlashGuid + ); + MicrocodeAddressInMemory =3D (UINT64 *) (MicrocodeShadowHob + 1); + Flashcontext =3D (EFI_MICROCODE_STORAGE_TYPE_FLASH_CONTEXT *) (Microcode= AddressInMemory + PatchCount); + + // + // Allocate memory for microcode shadow operation. + // + MicrocodePatchInRam =3D AllocatePages (EFI_SIZE_TO_PAGES (TotalLoadSize)= ); + if (MicrocodePatchInRam =3D=3D NULL) { + ASSERT (FALSE); + return; + } + + // + // Shadow all the required microcode patches into memory + // + for (Walker =3D MicrocodePatchInRam, Index =3D 0; Index < PatchCount; In= dex++) { + CopyMem ( + Walker, + (VOID *) Patches[Index].Address, + Patches[Index].Size + ); + MicrocodeAddressInMemory[Index] =3D (UINT64) Walker; + Flashcontext->MicrocodeAddressInFlash[Index] =3D (UINT64) Patches[Ind= ex].Address; + Walker +=3D Patches[Index].Size; + } + + // + // Update the microcode patch related fields in CpuMpData + // + *Buffer =3D (VOID *) (UINTN) MicrocodePatchInRam; + *BufferSize =3D TotalLoadSize; + + BuildGuidDataHob ( + &gEdkiiMicrocodeShadowInfoHobGuid, + MicrocodeShadowHob, + HobDataLength + ); + + DEBUG (( + DEBUG_INFO, + "%a: Required microcode patches have been loaded at 0x%lx, with size 0= x%lx.\n", + __FUNCTION__, *Buffer, *BufferSize + )); + + return; +} + +/** + Shadow the required microcode patches data into memory according + to FIT microcode entry. + +**/ +EFI_STATUS +ShadowMicrocodePatchByFit ( + IN UINTN CpuIdCount, + IN EDKII_PEI_MICROCODE_CPU_ID *MicrocodeCpuId, + OUT UINTN *BufferSize, + OUT VOID **Buffer + ) +{ + UINT64 FitPointer; + FIRMWARE_INTERFACE_TABLE_ENTRY *FitEntry; + UINT32 EntryNum; + UINT32 Index; + MICROCODE_PATCH_INFO *PatchInfoBuffer; + UINTN MaxPatchNumber; + CPU_MICROCODE_HEADER *MicrocodeEntryPoint; + UINTN PatchCount; + UINTN TotalSize; + UINTN TotalLoadSize; + + FitPointer =3D *(UINT64 *) (UINTN) FIT_POINTER_ADDRESS; + if ((FitPointer =3D=3D 0) || + (FitPointer =3D=3D 0xFFFFFFFFFFFFFFFF) || + (FitPointer =3D=3D 0xEEEEEEEEEEEEEEEE)) { + // + // No FIT table. + // + ASSERT (FALSE); + return EFI_NOT_FOUND; + } + FitEntry =3D (FIRMWARE_INTERFACE_TABLE_ENTRY *) (UINTN) FitPointer; + if ((FitEntry[0].Type !=3D FIT_TYPE_00_HEADER) || + (FitEntry[0].Address !=3D FIT_TYPE_00_SIGNATURE)) { + // + // Invalid FIT table, treat it as no FIT table. + // + ASSERT (FALSE); + return EFI_NOT_FOUND; + } + + EntryNum =3D *(UINT32 *)(&FitEntry[0].Size[0]) & 0xFFFFFF; + + // + // Calculate microcode entry number + // + MaxPatchNumber =3D 0; + for (Index =3D 0; Index < EntryNum; Index++) { + if (FitEntry[Index].Type =3D=3D FIT_TYPE_01_MICROCODE) { + MaxPatchNumber++; + } + } + if (MaxPatchNumber =3D=3D 0) { + return EFI_NOT_FOUND; + } + + PatchInfoBuffer =3D AllocatePool (MaxPatchNumber * sizeof (MICROCODE_PAT= CH_INFO)); + if (PatchInfoBuffer =3D=3D NULL) { + return EFI_OUT_OF_RESOURCES; + } + + // + // Fill up microcode patch info buffer according to FIT table. + // + PatchCount =3D 0; + TotalLoadSize =3D 0; + for (Index =3D 0; Index < EntryNum; Index++) { + if (FitEntry[Index].Type =3D=3D FIT_TYPE_01_MICROCODE) { + MicrocodeEntryPoint =3D (CPU_MICROCODE_HEADER *) (UINTN) FitEntry[In= dex].Address; + TotalSize =3D (MicrocodeEntryPoint->DataSize =3D=3D 0) ? 2048 : Micr= ocodeEntryPoint->TotalSize; + if (IsMicrocodePatchNeedLoad (CpuIdCount, MicrocodeCpuId, MicrocodeE= ntryPoint)) { + PatchInfoBuffer[PatchCount].Address =3D (UINTN) MicrocodeEntry= Point; + PatchInfoBuffer[PatchCount].Size =3D TotalSize; + TotalLoadSize +=3D TotalSize; + PatchCount++; + } + } + } + + if (PatchCount !=3D 0) { + DEBUG (( + DEBUG_INFO, + "%a: 0x%x microcode patches will be loaded into memory, with size 0x= %x.\n", + __FUNCTION__, PatchCount, TotalLoadSize + )); + + ShadowMicrocodePatchWorker (PatchInfoBuffer, PatchCount, TotalLoadSize= , BufferSize, Buffer); + } + + FreePool (PatchInfoBuffer); + return EFI_SUCCESS; +} + + +/** + Shadow microcode update patches to memory. + + The function is used for shadowing microcode update patches to a continu= ous memory. + It shall allocate memory buffer and only shadow the microcode patches fo= r those + processors specified by MicrocodeCpuId array. The checksum verification = may be + skiped in this function so the caller must perform checksum verification= before + using the microcode patches in returned memory buffer. + + @param[in] This The PPI instance pointer. + @param[in] CpuIdCount Number of elements in MicrocodeCpuId ar= ray. + @param[in] MicrocodeCpuId A pointer to an array of EDKII_PEI_MICR= OCODE_CPU_ID + structures. + @param[out] BufferSize Pointer to receive the total size of Bu= ffer. + @param[out] Buffer Pointer to receive address of allocated= memory + with microcode patches data in it. + + @retval EFI_SUCCESS The microcode has been shadowed to memo= ry. + @retval EFI_OUT_OF_RESOURCES The operation fails due to lack of reso= urces. + +**/ +EFI_STATUS +ShadowMicrocode ( + IN EDKII_PEI_SHADOW_MICROCODE_PPI *This, + IN UINTN CpuIdCount, + IN EDKII_PEI_MICROCODE_CPU_ID *MicrocodeCpuId, + OUT UINTN *BufferSize, + OUT VOID **Buffer + ) +{ + if (BufferSize =3D=3D NULL || Buffer =3D=3D NULL) { + return EFI_INVALID_PARAMETER; + } + + return ShadowMicrocodePatchByFit (CpuIdCount, MicrocodeCpuId, BufferSize= , Buffer); +} + + +/** + Platform Init PEI module entry point + + @param[in] FileHandle Not used. + @param[in] PeiServices General purpose services available to e= very PEIM. + + @retval EFI_SUCCESS The function completes successfully + @retval EFI_OUT_OF_RESOURCES Insufficient resources to create databa= se +**/ +EFI_STATUS +EFIAPI +ShadowMicrocodePeimInit ( + IN EFI_PEI_FILE_HANDLE FileHandle, + IN CONST EFI_PEI_SERVICES **PeiServices + ) +{ + EFI_STATUS Status; + + // + // Install EDKII Shadow Microcode PPI + // + Status =3D PeiServicesInstallPpi(mPeiShadowMicrocodePpiList); + ASSERT_EFI_ERROR (Status); + + return Status; +} diff --git a/Silicon/Intel/IntelSiliconPkg/Feature/ShadowMicrocode/ShadowMi= crocodePei.inf b/Silicon/Intel/IntelSiliconPkg/Feature/ShadowMicrocode/Shad= owMicrocodePei.inf new file mode 100644 index 0000000000..019400ab31 --- /dev/null +++ b/Silicon/Intel/IntelSiliconPkg/Feature/ShadowMicrocode/ShadowMicrocode= Pei.inf @@ -0,0 +1,43 @@ +### @file +# FIT based microcode shadow PEIM. +# +# Copyright (c) 2020, Intel Corporation. All rights reserved.
+# +# SPDX-License-Identifier: BSD-2-Clause-Patent +# +### + +[Defines] + INF_VERSION =3D 0x00010017 + BASE_NAME =3D ShadowMicrocodePei + FILE_GUID =3D 8af4cf68-ebe4-4b21-a008-0cb3da277be5 + VERSION_STRING =3D 1.0 + MODULE_TYPE =3D PEIM + ENTRY_POINT =3D ShadowMicrocodePeimInit + +[Sources] + ShadowMicrocodePei.c + +[LibraryClasses] + PeimEntryPoint + DebugLib + MemoryAllocationLib + BaseMemoryLib + HobLib + PeiServicesLib + +[Packages] + MdePkg/MdePkg.dec + MdeModulePkg/MdeModulePkg.dec + UefiCpuPkg/UefiCpuPkg.dec + IntelSiliconPkg/IntelSiliconPkg.dec + +[Ppis] + gEdkiiPeiShadowMicrocodePpiGuid ## PRODUCES + +[Guids] + gEdkiiMicrocodeShadowInfoHobGuid + gEdkiiMicrocodeStorageTypeFlashGuid + +[Depex] + TRUE diff --git a/Silicon/Intel/IntelSiliconPkg/Include/Guid/MicrocodeShadowInfo= Hob.h b/Silicon/Intel/IntelSiliconPkg/Include/Guid/MicrocodeShadowInfoHob.h new file mode 100644 index 0000000000..d1a9d79a51 --- /dev/null +++ b/Silicon/Intel/IntelSiliconPkg/Include/Guid/MicrocodeShadowInfoHob.h @@ -0,0 +1,64 @@ +/** @file + The definition for Microcode Shadow Info Hob. + + Copyright (c) 2019, Intel Corporation. All rights reserved.
+ SPDX-License-Identifier: BSD-2-Clause-Patent +**/ + + +#ifndef _MICROCODE_SHADOW_INFO_HOB_H_ +#define _MICROCODE_SHADOW_INFO_HOB_H_ + +/// +/// The Global ID of a GUIDed HOB used to pass microcode shadow info to DX= E Driver. +/// +#define EDKII_MICROCODE_SHADOW_INFO_HOB_GUID \ + { \ + 0x658903f9, 0xda66, 0x460d, { 0x8b, 0xb0, 0x9d, 0x2d, 0xdf, 0x65, 0x44= , 0x59 } \ + } + +extern EFI_GUID gEdkiiMicrocodeShadowInfoHobGuid; + +typedef struct { + // + // An EFI_GUID that defines the contents of StorageContext. + // + GUID StorageType; + // + // Number of the microcode patches which have been + // relocated to memory. + // + UINT64 MicrocodeCount; + // + // An array with MicrocodeCount elements that stores + // the shadowed microcode patch address in memory. + // + UINT64 MicrocodeAddrInMemory[]; + // + // A buffer which contains details about the storage information + // specific to StorageType. + // + // UINT8 StorageContext[]; +} EDKII_MICROCODE_SHADOW_INFO_HOB; + +// +// An EDKII_MICROCODE_SHADOW_INFO_HOB with StorageType set to below GUID w= ill have +// the StorageContext of a EFI_MICROCODE_STORAGE_TYPE_FLASH_CONTEXT strucu= tre. +// +#define EFI_MICROCODE_STORAGE_TYPE_FLASH_GUID \ + { \ + 0x2cba01b3, 0xd391, 0x4598, { 0x8d, 0x89, 0xb7, 0xfc, 0x39, 0x22, 0xfd= , 0x71 } \ + } + +extern EFI_GUID gEdkiiMicrocodeStorageTypeFlashGuid; + +typedef struct { + // + // An array with MicrocodeCount elements that stores the original + // microcode patch address on flash. The address is placed in same + // order as the microcode patches in MicrocodeAddrInMemory. + // + UINT64 MicrocodeAddressInFlash[]; +} EFI_MICROCODE_STORAGE_TYPE_FLASH_CONTEXT; + +#endif diff --git a/Silicon/Intel/IntelSiliconPkg/IntelSiliconPkg.dec b/Silicon/In= tel/IntelSiliconPkg/IntelSiliconPkg.dec index 22ebf19c4e..3c49fb289c 100644 --- a/Silicon/Intel/IntelSiliconPkg/IntelSiliconPkg.dec +++ b/Silicon/Intel/IntelSiliconPkg/IntelSiliconPkg.dec @@ -3,7 +3,7 @@ # # This package provides common open source Intel silicon modules. # -# Copyright (c) 2016 - 2019, Intel Corporation. All rights reserved.
+# Copyright (c) 2016 - 2020, Intel Corporation. All rights reserved.
# SPDX-License-Identifier: BSD-2-Clause-Patent # ## @@ -48,6 +48,12 @@ ## HOB GUID to get memory information after MRC is done. The hob data wi= ll be used to set the PMR ranges gVtdPmrInfoDataHobGuid =3D {0x6fb61645, 0xf168, 0x46be, { 0x80, 0xec, 0x= b5, 0x02, 0x38, 0x5e, 0xe7, 0xe7 } } =20 + ## Include/Guid/MicrocodeShadowInfoHob.h + gEdkiiMicrocodeShadowInfoHobGuid =3D { 0x658903f9, 0xda66, 0x460d, { 0x8= b, 0xb0, 0x9d, 0x2d, 0xdf, 0x65, 0x44, 0x59 } } + + ## Include/Guid/MicrocodeShadowInfoHob.h + gEdkiiMicrocodeStorageTypeFlashGuid =3D { 0x2cba01b3, 0xd391, 0x4598, { = 0x8d, 0x89, 0xb7, 0xfc, 0x39, 0x22, 0xfd, 0x71 } } + [Ppis] gEdkiiVTdInfoPpiGuid =3D { 0x8a59fcb3, 0xf191, 0x400c, { 0x97, 0x67, 0x6= 7, 0xaf, 0x2b, 0x25, 0x68, 0x4a } } =20 diff --git a/Silicon/Intel/IntelSiliconPkg/IntelSiliconPkg.dsc b/Silicon/In= tel/IntelSiliconPkg/IntelSiliconPkg.dsc index 0a6509d8b3..f995883691 100644 --- a/Silicon/Intel/IntelSiliconPkg/IntelSiliconPkg.dsc +++ b/Silicon/Intel/IntelSiliconPkg/IntelSiliconPkg.dsc @@ -1,7 +1,7 @@ ## @file # This package provides common open source Intel silicon modules. # -# Copyright (c) 2017 - 2019, Intel Corporation. All rights reserved.
+# Copyright (c) 2017 - 2020, Intel Corporation. All rights reserved.
# # SPDX-License-Identifier: BSD-2-Clause-Patent # @@ -84,6 +84,7 @@ IntelSiliconPkg/Feature/VTd/PlatformVTdInfoSamplePei/PlatformVTdInfoSamp= lePei.inf IntelSiliconPkg/Feature/Capsule/MicrocodeUpdateDxe/MicrocodeUpdateDxe.inf IntelSiliconPkg/Feature/Capsule/Library/MicrocodeFlashAccessLibNull/Micr= ocodeFlashAccessLibNull.inf + IntelSiliconPkg/Feature/ShadowMicrocode/ShadowMicrocodePei.inf IntelSiliconPkg/Library/PeiDxeSmmBootMediaLib/PeiFirmwareBootMediaLib.inf IntelSiliconPkg/Library/PeiDxeSmmBootMediaLib/DxeSmmFirmwareBootMediaLib= .inf =20 --=20 2.19.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 (#54407): https://edk2.groups.io/g/devel/message/54407 Mute This Topic: https://groups.io/mt/71262072/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-