public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Siyuan, Fu" <siyuan.fu@intel.com>
To: devel@edk2.groups.io
Cc: michael.d.kinney@intel.com, liming.gao@intel.com,
	eric.dong@intel.com, ray.ni@intel.com, lersek@redhat.com,
	Tom Lendacky <thomas.lendacky@amd.com>
Subject: [PATCH v2 0/2] Shadow microcode patch according to FIT microcode table.
Date: Thu,  9 Jan 2020 10:14:03 +0800	[thread overview]
Message-ID: <cover.1578535686.git.siyuan.fu@intel.com> (raw)

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).

V2:
Add Feature PCD for Patch 2/2 for enabling/disabling this support.

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>
Cc: Tom Lendacky <thomas.lendacky@amd.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/DxeMpInitLib.inf |   1 +
 UefiCpuPkg/Library/MpInitLib/Microcode.c      | 262 +++++++++++++-----
 UefiCpuPkg/Library/MpInitLib/MpLib.c          |   4 +-
 UefiCpuPkg/Library/MpInitLib/MpLib.h          |   7 +-
 UefiCpuPkg/Library/MpInitLib/PeiMpInitLib.inf |   4 +-
 UefiCpuPkg/UefiCpuPkg.dec                     |   6 +
 7 files changed, 292 insertions(+), 68 deletions(-)
 create mode 100644 MdePkg/Include/IndustryStandard/FirmwareInterfaceTable.h

-- 
2.19.1.windows.1


             reply	other threads:[~2020-01-09  2:14 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-01-09  2:14 Siyuan, Fu [this message]
2020-01-09  2:14 ` [PATCH v2 1/2] MdePkg: Add header file for Firmware Interface Table specification Siyuan, Fu
2020-01-10  3:09   ` Liming Gao
2020-01-10 10:41     ` Ni, Ray
2020-01-13  0:56       ` Liming Gao
2020-01-09  2:14 ` [PATCH v2 2/2] UefiCpuPkg: Shadow microcode patch according to FIT microcode entry Siyuan, Fu
2020-01-09 13:32   ` Laszlo Ersek
2020-01-10  3:38   ` [edk2-devel] " Dong, Eric

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=cover.1578535686.git.siyuan.fu@intel.com \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox