Re: [Patch 0/2] Shadow microcode patch according to FIT microcode table.

["Siyuan, Fu" <siyuan.fu@intel.com>]

Hi, Liming

Yes I will make another patch to remove the edk2-platforms header file once this patch is committed.

Best Regards
Siyuan 

> -----Original Message-----
> From: Gao, Liming <liming.gao@intel.com>
> Sent: 2020年1月9日 10:04
> To: Fu, Siyuan <siyuan.fu@intel.com>; devel@edk2.groups.io
> Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Dong, Eric
> <eric.dong@intel.com>; Ni, Ray <ray.ni@intel.com>; lersek@redhat.com
> Subject: RE: [Patch 0/2] Shadow microcode patch according to FIT microcode
> table.
> 
> Siyuan:
>   This is new feature, please add it into
> https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Release-
> Planning
> 
>   And, FIT header file is moved from edk2-platform to edk2. After this change,
> you also need to remove the one in edk2-platform. Right?
> 
> Thanks
> Liming
> -----Original Message-----
> From: Fu, Siyuan <siyuan.fu@intel.com>
> Sent: 2020年1月8日 12:25
> To: devel@edk2.groups.io
> Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming
> <liming.gao@intel.com>; Dong, Eric <eric.dong@intel.com>; Ni, Ray
> <ray.ni@intel.com>; lersek@redhat.com
> Subject: [Patch 0/2] Shadow microcode patch according to FIT microcode
> table.
> 
> The existing MpInitLib will shadow the microcode update patches from flash
> to memory and this is done by searching microcode region specified by PCD
> PcdCpuMicrocodePatchAddress and PcdCpuMicrocodePatchRegionSize.
> This brings a limition to platform FW that all the microcode patches must be
> placed in one continuous flash space.
> 
> This patch set shadows microcode update according to FIT microcode entries
> if it's present, otherwise it will fallback to original logic (by PCD).
> 
> Patch 1/2: Add FIT header file to MdePkg.
> Patch 2/2: Update microcode loader to shadow microcode according to FIT.
> 
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Liming Gao <liming.gao@intel.com>
> Cc: Eric Dong <eric.dong@intel.com>
> Cc: Ray Ni <ray.ni@intel.com>
> Cc: Laszlo Ersek <lersek@redhat.com>
> 
> Siyuan Fu (2):
>   MdePkg: Add header file for Firmware Interface Table specification.
>   UefiCpuPkg: Shadow microcode patch according to FIT microcode entry.
> 
>  .../IndustryStandard/FirmwareInterfaceTable.h |  76 ++++++
>  UefiCpuPkg/Library/MpInitLib/Microcode.c      | 255 +++++++++++++-----
>  UefiCpuPkg/Library/MpInitLib/MpLib.c          |   4 +-
>  UefiCpuPkg/Library/MpInitLib/MpLib.h          |   7 +-
>  4 files changed, 276 insertions(+), 66 deletions(-)  create mode 100644
> MdePkg/Include/IndustryStandard/FirmwareInterfaceTable.h
> 
> --
> 2.19.1.windows.1