public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Min Xu" <min.m.xu@intel.com>
To: devel@edk2.groups.io
Cc: Min Xu <min.m.xu@intel.com>,
	Michael D Kinney <michael.d.kinney@intel.com>,
	Brijesh Singh <brijesh.singh@amd.com>,
	Erdem Aktas <erdemaktas@google.com>,
	James Bottomley <jejb@linux.ibm.com>,
	Jiewen Yao <jiewen.yao@intel.com>,
	Tom Lendacky <thomas.lendacky@amd.com>,
	Gerd Hoffmann <kraxel@redhat.com>,
	Leif Lindholm <leif@nuviainc.com>,
	Ard Biesheuvel <ardb+tianocore@kernel.org>,
	Abner Chang <abner.chang@hpe.com>,
	Daniel Schaefer <daniel.schaefer@hpe.com>,
	Ray Ni <ray.ni@intel.com>
Subject: [PATCH V5 00/10] Introduce TDVF Config-B (basic) in OvmfPkg
Date: Sun, 13 Mar 2022 08:41:01 +0800	[thread overview]
Message-ID: <20220313004111.388-1-min.m.xu@intel.com> (raw)

RFC: https://bugzilla.tianocore.org/show_bug.cgi?id=3429

Intel's Trust Domain Extensions (Intel TDX) refers to an Intel technology
that extends Virtual Machines Extensions (VMX) and Multi-Key Total Memory
Encryption (MKTME) with a new kind of virutal machines guest called a
Trust Domain (TD). A TD is desinged to run in a CPU mode that protects the
confidentiality of TD memory contents and the TD's CPU state from other
software, including the hosting Virtual-Machine Monitor (VMM), unless
explicitly shared by the TD itself.

There are 2 configurations for TDVF to upstream. See below link for
the definitions of the 2 configurations.
https://edk2.groups.io/g/devel/message/76367

This patch-set is to enable the basic feature of Config-B in OvmfPkg.
 - Add a standalone IntelTdxX64.dsc to a TDX specific directory for a
   *full* feature TDVF. (Align with existing SEV)
 - IntelTdxX64.dsc includes TDX/normal OVMF basic boot capability. The
   final binary can run on TDX/normal OVMF.
 - PEI phase is skipped.

By design in Config-B there should be more advanced features, such as:
 - RTMR based measurement and measure boot.
 - Remove unnecessary drivers to reduce attack surface, such as
   network stack.

To make the code review more efficiency, Config-B is split into 2 waves:
 - Basic feature of Config-B
 - Advanced feature of Config-B
This patch-set is for *Basic feature of Config-B*.

Code at: https://github.com/mxu9/edk2/tree/tdvf_wave3.v5

Please be noted:
This patch-set is based on tdvf_wave2.v8 which is at:
https://github.com/mxu9/edk2/tree/tdvf_wave2.v8

v5 changes:
 - Move PeilessStartupLib to OvmfPkg/Library.
 - Split the changes of IncompatiblePciDeviceSupportDxe to 2 patch.
   Please see Patch 7 & 8.

v4 changes:
 - Rename TdxStartupLib to PeilessStartupLib.
 - Check HostBridgeDevId from PlatformInfoHob in DxeAcpiTimerLib.
 - Add 2 more patches (#8 #9) to ignore PCI OptionRom in Td guest.

v3 changes:
 - Drop the PcdTdxWorkAreaBase introduced in v2. Instead PcdSevEsWorkArea
   is used to point to the TDX WorkArea. PcdSevEsWorkArea will be renamed
   to PcdCcWorkAreaBase in the future.
 - In v2 TdHob is processed in OvmfPkg/IntelTdx/Sec/IntelTdx.c. Since
   the codes have been moved to PlatformInitLib, IntelTdx.c is deleted.
 - Add comments in DxeAcpiTimerLib to explain why both
   PcdOvmfHostBridgePciDevId and EFI_HOB_PLATFORM_INFO are checked to
   read HostBridgeDevId.

v2 changes:
 - In v1 PEI phase is skipped during boot process. But the code is still
   in the image. It is not a good way. So in v2 the code of PEI is
   removed as well.
 - In v1 PrePiLib in EmbeddedPkg are updated to meet the requirement of
   TDVF-Config-B. This is because PrePiLib in EmbeddedPkg is designed for
   the situation of PEI-less boot. So after PEI is removed, this change
   is not needed anymore.
 - In v1 only Tdx guest can be brought up. Now v2 brings up both Legacy
   guest and Tdx guest.
 - Platform initialization in TdxStartup is refactored with the
   PlatformInitLib.
 - OvmfPkg/IntelTdx/Sec is introduced. It is a simplied version of
   OvmfPkg/Sec.

Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Brijesh Singh <brijesh.singh@amd.com>
Cc: Erdem Aktas <erdemaktas@google.com>
Cc: James Bottomley <jejb@linux.ibm.com>
Cc: Jiewen Yao <jiewen.yao@intel.com>
Cc: Tom Lendacky <thomas.lendacky@amd.com>
Cc: Gerd Hoffmann <kraxel@redhat.com>
Cc: Leif Lindholm <leif@nuviainc.com>
Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
Cc: Abner Chang <abner.chang@hpe.com>
Cc: Daniel Schaefer <daniel.schaefer@hpe.com>
Cc: Ray Ni <ray.ni@intel.com>
Signed-off-by: Min Xu <min.m.xu@intel.com>

Min Xu (10):
  OvmfPkg: Add TdxWorkArea definition
  OvmfPkg: Add PrePiHobListPointerLibTdx
  OvmfPkg: Add PeilessStartupLib
  OvmfPkg/IntelTdx: Add Sec to bring up both Legacy and Tdx guest
  OvmfPkg: Update TdxDxe to set TDX PCDs
  OvmfPkg: Update DxeAcpiTimerLib to read HostBridgeDevId in
    PlatformInfoHob
  OvmfPkg/IncompatiblePciDeviceSupportDxe: Refine the configuration
  OvmfPkg/IncompatiblePciDeviceSupportDxe: Ignore OptionRom in Td guest
  MdeModulePkg: Update PciEnumeratorSupport to ignore OptionRom if
    needed
  OvmfPkg: Introduce IntelTdxX64 for TDVF Config-B

 .../Bus/Pci/PciBusDxe/PciEnumeratorSupport.c  |  23 +-
 .../Bus/Pci/PciBusDxe/PciEnumeratorSupport.h  |   4 +-
 OvmfPkg/Include/Library/PeilessStartupLib.h   |  34 +
 OvmfPkg/Include/WorkArea.h                    |  15 +
 .../IncompatiblePciDeviceSupport.c            | 108 +-
 OvmfPkg/IntelTdx/IntelTdxX64.dsc              | 721 ++++++++++++++
 OvmfPkg/IntelTdx/IntelTdxX64.fdf              | 401 ++++++++
 .../PrePiHobListPointer.c                     |  57 ++
 .../PrePiHobListPointerLibTdx.inf             |  29 +
 OvmfPkg/IntelTdx/Sec/SecMain.c                | 198 ++++
 OvmfPkg/IntelTdx/Sec/SecMain.inf              |  61 ++
 OvmfPkg/IntelTdx/Sec/X64/SecEntry.nasm        | 151 +++
 .../Library/AcpiTimerLib/DxeAcpiTimerLib.c    |  25 +-
 .../Library/AcpiTimerLib/DxeAcpiTimerLib.inf  |   7 +-
 OvmfPkg/Library/PeilessStartupLib/DxeLoad.c   | 218 ++++
 OvmfPkg/Library/PeilessStartupLib/Hob.c       | 135 +++
 .../PeilessStartupLib/PeilessStartup.c        | 223 +++++
 .../PeilessStartupInternal.h                  |  55 ++
 .../PeilessStartupLib/PeilessStartupLib.inf   |  86 ++
 .../PeilessStartupLib/X64/PageTables.h        | 206 ++++
 .../PeilessStartupLib/X64/VirtualMemory.c     | 935 ++++++++++++++++++
 OvmfPkg/OvmfPkg.dec                           |   4 +
 OvmfPkg/TdxDxe/TdxDxe.c                       |  66 +-
 OvmfPkg/TdxDxe/TdxDxe.inf                     |   4 +
 24 files changed, 3724 insertions(+), 42 deletions(-)
 create mode 100644 OvmfPkg/Include/Library/PeilessStartupLib.h
 create mode 100644 OvmfPkg/IntelTdx/IntelTdxX64.dsc
 create mode 100644 OvmfPkg/IntelTdx/IntelTdxX64.fdf
 create mode 100644 OvmfPkg/IntelTdx/PrePiHobListPointerLibTdx/PrePiHobListPointer.c
 create mode 100644 OvmfPkg/IntelTdx/PrePiHobListPointerLibTdx/PrePiHobListPointerLibTdx.inf
 create mode 100644 OvmfPkg/IntelTdx/Sec/SecMain.c
 create mode 100644 OvmfPkg/IntelTdx/Sec/SecMain.inf
 create mode 100644 OvmfPkg/IntelTdx/Sec/X64/SecEntry.nasm
 create mode 100644 OvmfPkg/Library/PeilessStartupLib/DxeLoad.c
 create mode 100644 OvmfPkg/Library/PeilessStartupLib/Hob.c
 create mode 100644 OvmfPkg/Library/PeilessStartupLib/PeilessStartup.c
 create mode 100644 OvmfPkg/Library/PeilessStartupLib/PeilessStartupInternal.h
 create mode 100644 OvmfPkg/Library/PeilessStartupLib/PeilessStartupLib.inf
 create mode 100644 OvmfPkg/Library/PeilessStartupLib/X64/PageTables.h
 create mode 100644 OvmfPkg/Library/PeilessStartupLib/X64/VirtualMemory.c

-- 
2.29.2.windows.2


             reply	other threads:[~2022-03-13  0:41 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-03-13  0:41 Min Xu [this message]
2022-03-13  0:41 ` [PATCH V5 01/10] OvmfPkg: Add TdxWorkArea definition Min Xu
2022-03-13  0:41 ` [PATCH V5 02/10] OvmfPkg: Add PrePiHobListPointerLibTdx Min Xu
2022-03-13  0:41 ` [PATCH V5 03/10] OvmfPkg: Add PeilessStartupLib Min Xu
2022-03-13  0:41 ` [PATCH V5 04/10] OvmfPkg/IntelTdx: Add Sec to bring up both Legacy and Tdx guest Min Xu
2022-03-13  0:41 ` [PATCH V5 05/10] OvmfPkg: Update TdxDxe to set TDX PCDs Min Xu
2022-03-13  0:41 ` [PATCH V5 06/10] OvmfPkg: Update DxeAcpiTimerLib to read HostBridgeDevId in PlatformInfoHob Min Xu
2022-03-15 13:46   ` Gerd Hoffmann
2022-03-13  0:41 ` [PATCH V5 07/10] OvmfPkg/IncompatiblePciDeviceSupportDxe: Refine the configuration Min Xu
2022-03-15 13:48   ` Gerd Hoffmann
2022-03-13  0:41 ` [PATCH V5 08/10] OvmfPkg/IncompatiblePciDeviceSupportDxe: Ignore OptionRom in Td guest Min Xu
2022-03-15 13:48   ` Gerd Hoffmann
2022-03-13  0:41 ` [PATCH V5 09/10] MdeModulePkg: Update PciEnumeratorSupport to ignore OptionRom if needed Min Xu
2022-03-15 13:49   ` Gerd Hoffmann
2022-03-13  0:41 ` [PATCH V5 10/10] OvmfPkg: Introduce IntelTdxX64 for TDVF Config-B Min Xu
2022-04-02  8:20 ` [PATCH V5 00/10] Introduce TDVF Config-B (basic) in OvmfPkg Yao, Jiewen

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=20220313004111.388-1-min.m.xu@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