回复: [edk2-devel] [PATCH V2] MdeModulePkg/Decompress: Add missing source file to Brotli library

2023-01-28 Thread gaoliming via groups.io
Timothy: Thanks for your report. This is a real issue in Brotli decompress library. Which tool chain has been verified? GCC5 or VS2019 or Both? Thanks Liming 发件人: devel@edk2.groups.io 代表 Lin, Kuei-Hung (Timothy) via groups.io 发送时间: 2023年1月13日 6:18 收件人: He, Jiangang ; Chang, Abner ; Kinney,

回复: [edk2-devel] [PATCH v1 0/2] Add ACPI 6.5 header

2023-01-28 Thread gaoliming via groups.io
Reviewed-by: Liming Gao > -邮件原件- > 发件人: devel@edk2.groups.io 代表 Chao Li > 发送时间: 2023年1月17日 21:00 > 收件人: devel@edk2.groups.io > 抄送: Michael D Kinney ; Liming Gao > ; Zhiguang Liu > 主题: [edk2-devel] [PATCH v1 0/2] Add ACPI 6.5 header > > ACPI specification 6.5 has been released in August

Re: [edk2-devel] [PATCH v3 3/5] UefiCpuPkg/PiSmmCpuDxeSmm: Consume SMM Base Hob for SmBase info

2023-01-28 Thread Wu, Jiaxin
> > > +UINT32 mBspApicId = 0; > > This should be moved to a separate patch with commit message explaining > the reasons for the change. My guess would be this is required to allow > processors running SmmInitHandler in parallel. > Yes, it's part of work to combine 2 SMIs (gcSmmInitTemp

[edk2-devel] 回复: [PATCH v2 0/2] Added AfterReadyToBoot Event

2023-01-28 Thread gaoliming via groups.io
Reviewed-by: Liming Gao > -邮件原件- > 发件人: Robert Phelps > 发送时间: 2023年1月13日 4:07 > 收件人: devel@edk2.groups.io > 抄送: michael.d.kin...@intel.com; gaolim...@byosoft.com.cn; > zhiguang@intel.com > 主题: [PATCH v2 0/2] Added AfterReadyToBoot Event > > In the UEFI Specification version 2.9 a ne

[edk2-devel] 回复: [PATCH 2/2] MdePkg:IORT header update for IORT Rev E.e spec

2023-01-28 Thread gaoliming via groups.io
Reviewed-by: Liming Gao > -邮件原件- > 发件人: Name > 发送时间: 2023年1月26日 2:41 > 收件人: devel@edk2.groups.io; sami.muja...@arm.com; > alexei.fedo...@arm.com; pierre.gond...@arm.com; > michael.d.kin...@intel.com; gaolim...@byosoft.com.cn; > zhiguang@intel.com > 抄送: Swatisri Kantamsetti > 主题: [PA

Re: [edk2-devel] [PATCH v3 5/5] OvmfPkg/SmmCpuFeaturesLib: Skip SMBASE configuration

2023-01-28 Thread Wu, Jiaxin
> > * Why do you want do it in PEI instead? The cover letter claims this >improves boot performance. I don't buy that. You are not skipping >the relocation process, you are just doing it somewhere else. I clarified why it can improves boot performance, it's not just do somewhere else

[edk2-devel] 回复: [edk2][PATCH V1 1/1] MdePkg/IndustryStandard: add definitions for MPAM ACPI specification

2023-01-28 Thread gaoliming via groups.io
Acked-by: Liming Gao > -邮件原件- > 发件人: Rohit Mathew > 发送时间: 2023年1月25日 8:39 > 收件人: devel@edk2.groups.io > 抄送: Michael D Kinney ; Sami Mujawar > ; Liming Gao ; > Zhiguang Liu ; Thomas Abraham > ; Swatisri Kantamsetti > 主题: [edk2][PATCH V1 1/1] MdePkg/IndustryStandard: add definitions for >

回复: [edk2-devel] [PATCH v2 0/4] BaseTools,ArmPkg,ArmVirtPkg: Remove leftover RVCT and RealView Debugger support

2023-01-28 Thread gaoliming via groups.io
Reviewed-by: Liming Gao > -邮件原件- > 发件人: devel@edk2.groups.io 代表 Ard > Biesheuvel > 发送时间: 2023年1月28日 1:20 > 收件人: devel@edk2.groups.io; michael.d.kin...@intel.com > 抄送: rebe...@bsdio.com; Gao, Liming ; Feng, > Bob C ; Leif Lindholm ; > Chen, Christine ; Marvin Häuser > > 主题: Re: [edk2-dev

Re: [edk2-devel] [PATCH v3 1/5] UefiCpuPkg/SmmBaseHob.h: Add SMM Base HOB Data

2023-01-28 Thread Wu, Jiaxin
Thanks Gerd raise this open -- how to support more processors due to hob size limitation. Looks multiple hobs is the only way since we have to store each cpu's info? Sorry, allow me ask a stupid question: why DataLength in hob defined as UINT16 causing the hob size limitation? Any design backg

回复: [edk2-devel] [PATCH] MdeModulePkg: Correct memory type in PrePiDxeCis.h

2023-01-28 Thread gaoliming via groups.io
Reviewed-by: Liming Gao > -邮件原件- > 发件人: Dionna Amalie Glaze > 发送时间: 2023年1月28日 8:02 > 收件人: Xu, Min M > 抄送: devel@edk2.groups.io; Yao, Jiewen ; Kinney, > Michael D ; Gao, Liming > ; Liu, Zhiguang ; Wang, > Jian J > 主题: Re: [edk2-devel] [PATCH] MdeModulePkg: Correct memory type in > PreP

Re: [edk2-devel] [PATCH v3 1/5] UefiCpuPkg/SmmBaseHob.h: Add SMM Base HOB Data

2023-01-28 Thread Wu, Jiaxin
> > For the smbase case it might also be possible to store the base > address and the per-cpu entry size, then use > > smbase = base + size * cpuindex > > to calculate the per-cpu base address. Puts some restrictions on the > memory allocation (must be one big block), not sure this is pos

[edk2-devel] 回复: [PATCH 1/1] tools_def: Remove duplicated -Os

2023-01-28 Thread gaoliming via groups.io
Reviewed-by: Liming Gao > -邮件原件- > 发件人: Ard Biesheuvel > 发送时间: 2023年1月27日 22:00 > 收件人: pierre.gond...@arm.com > 抄送: devel@edk2.groups.io; Bob Feng ; Liming Gao > ; Yuwei Chen ; Leif > Lindholm ; Sami Mujawar > ; Laszlo Ersek > 主题: Re: [PATCH 1/1] tools_def: Remove duplicated -Os > > On

[edk2-devel] 回复: [PATCH 1/1] BaseTools: remove useless header inclusion

2023-01-28 Thread gaoliming via groups.io
With this change, does GenFv pass build on Linux environment? > -邮件原件- > 发件人: tlaro...@polynum.com > 发送时间: 2023年1月26日 18:03 > 收件人: devel@edk2.groups.io > 抄送: Bob Feng ; Liming Gao > ; Yuwei Chen > 主题: [PATCH 1/1] BaseTools: remove useless header inclusion > > GenFvInternalLib.c: useless

回复: [edk2-devel] [PATCH 1/1] tools_def: Remove duplicated -Os

2023-01-28 Thread gaoliming via groups.io
Reviewed-by: Liming Gao > -邮件原件- > 发件人: devel@edk2.groups.io 代表 Ard > Biesheuvel > 发送时间: 2023年1月27日 22:00 > 收件人: pierre.gond...@arm.com > 抄送: devel@edk2.groups.io; Bob Feng ; Liming Gao > ; Yuwei Chen ; Leif > Lindholm ; Sami Mujawar > ; Laszlo Ersek > 主题: Re: [edk2-devel] [PATCH 1/1] t

[edk2-devel] [PATCH EDK2 v1 0/1] NetworkPkg/Ip4Dxe:Null-checking before dereference

2023-01-28 Thread wenyi,xie via groups.io
Main Changes : 1.Null-checking first before a pointer is dereferenced. Wenyi Xie (1): NetworkPkg/Ip4Dxe:Null-checking before dereference NetworkPkg/Ip4Dxe/Ip4Config2Nv.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) -- 2.20.1.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Grou

[edk2-devel] [PATCH EDK2 v1 1/1] NetworkPkg/Ip4Dxe:Null-checking before dereference

2023-01-28 Thread wenyi,xie via groups.io
Null-checking first before a pointer is dereferenced. Cc: Maciej Rabeda Cc: Siyuan Fu Signed-off-by: Wenyi Xie --- NetworkPkg/Ip4Dxe/Ip4Config2Nv.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/NetworkPkg/Ip4Dxe/Ip4Config2Nv.c b/NetworkPkg/Ip4Dxe/Ip4Confi

[edk2-devel] [edk2-staging/RiscV64QemuVirt PATCH V7 19/20] OvmfPkg/RiscVVirt: Add build files for Qemu Virt platform

2023-01-28 Thread Sunil V L
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4076 Add infrastructure files to build edk2 for RISC-V qemu virt machine. - It follows PEI less design. - EDK2 for qemu virt is booted in S-mode as a payload for M-mode FW - Leveraged from ArmVirtQemu Cc: Ard Biesheuvel Cc: Jiewen Yao Cc: Jor

[edk2-devel] [edk2-staging/RiscV64QemuVirt PATCH V7 17/20] OvmfPkg/RiscVVirt: Add PciCpuIo2Dxe module

2023-01-28 Thread Sunil V L
Add PciCpuIo2Dxe driver to implement EFI_CPU_IO2_PROTOCOL to add the translation for IO access. This is copied from ArmPciCpuIo2Dxe driver. Cc: Ard Biesheuvel Cc: Jiewen Yao Cc: Jordan Justen Cc: Gerd Hoffmann Signed-off-by: Sunil V L Acked-by: Abner Chang --- OvmfPkg/RiscVVirt/PciCpuIo2Dxe

[edk2-devel] [edk2-staging/RiscV64QemuVirt PATCH V7 20/20] Maintainers.txt: Add entry for OvmfPkg/RiscVVirt

2023-01-28 Thread Sunil V L
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4076 RiscVVirt is created to support EDK2 for RISC-V qemu virt machine platform. Add maintainer entries. Cc: Andrew Fish Cc: Leif Lindholm Cc: Michael D Kinney Signed-off-by: Sunil V L --- Maintainers.txt | 4 1 file changed, 4 inserti

[edk2-devel] [edk2-staging/RiscV64QemuVirt PATCH V7 18/20] OvmfPkg/RiscVVirt: Add SEC module

2023-01-28 Thread Sunil V L
Add the SEC module for RISC-V Qemu virt machine support. It uses the PEI less design. Cc: Ard Biesheuvel Cc: Jiewen Yao Cc: Jordan Justen Cc: Gerd Hoffmann Signed-off-by: Sunil V L Acked-by: Abner Chang --- OvmfPkg/RiscVVirt/Sec/SecMain.inf | 66 + OvmfPkg/RiscVVirt/Sec/SecMain.h | 1

[edk2-devel] [edk2-staging/RiscV64QemuVirt PATCH V7 16/20] OvmfPkg/RiscVVirt: Add VirtNorFlashPlatformLib library

2023-01-28 Thread Sunil V L
Qemu NOR flash driver needs this library. Add this library for RISC-V leveraged from SbsaQemu. Cc: Ard Biesheuvel Cc: Jiewen Yao Cc: Jordan Justen Cc: Gerd Hoffmann Signed-off-by: Sunil V L Acked-by: Abner Chang --- OvmfPkg/RiscVVirt/Library/VirtNorFlashPlatformLib/VirtNorFlashStaticLib.inf

[edk2-devel] [edk2-staging/RiscV64QemuVirt PATCH V7 15/20] OvmfPkg/RiscVVirt: Add ResetSystemLib library

2023-01-28 Thread Sunil V L
RISC-V Qemu virt uses SBI calls to implement the reset. Add the base class library. Cc: Ard Biesheuvel Cc: Jiewen Yao Cc: Jordan Justen Cc: Gerd Hoffmann Signed-off-by: Sunil V L Acked-by: Abner Chang --- OvmfPkg/RiscVVirt/Library/ResetSystemLib/BaseResetSystemLib.inf | 38 ++ OvmfPkg/

[edk2-devel] [edk2-staging/RiscV64QemuVirt PATCH V7 14/20] OvmfPkg/RiscVVirt: Add PrePiHobListPointerLib library

2023-01-28 Thread Sunil V L
Add the PrePiHobListPointerLib required for RISC-V Qemu Virt machine since it follows PEIless design. Cc: Ard Biesheuvel Cc: Jiewen Yao Cc: Jordan Justen Cc: Gerd Hoffmann Signed-off-by: Sunil V L Acked-by: Abner Chang --- OvmfPkg/RiscVVirt/Library/PrePiHobListPointerLib/PrePiHobListPointer

[edk2-devel] [edk2-staging/RiscV64QemuVirt PATCH V7 13/20] OvmfPkg/RiscVVirt: Add PlatformBootManagerLib library

2023-01-28 Thread Sunil V L
RISC-V Qemu Virt platfform needs the PlatformBootManagerLib similar to the one in ArmVirtPlatform. Add the library in OvmfPkg/RiscVVirt leveraging the one from Arm. Cc: Ard Biesheuvel Cc: Jiewen Yao Cc: Jordan Justen Cc: Gerd Hoffmann Signed-off-by: Sunil V L Acked-by: Abner Chang --- OvmfP

[edk2-devel] [edk2-staging/RiscV64QemuVirt PATCH V7 10/20] EmbeddedPkg: Enable PcdPrePiCpuIoSize for RISC-V

2023-01-28 Thread Sunil V L
This PCD is required to be enabled so that PrePiLib can be used in RISC-V. Cc: Leif Lindholm Cc: Ard Biesheuvel Cc: Abner Chang Cc: Daniel Schaefer Signed-off-by: Sunil V L --- EmbeddedPkg/EmbeddedPkg.dec | 3 +++ 1 file changed, 3 insertions(+) diff --git a/EmbeddedPkg/EmbeddedPkg.dec b/Em

[edk2-devel] [edk2-staging/RiscV64QemuVirt PATCH V7 12/20] ArmVirtPkg: Fix up the location of PlatformHasAcpiDtDxe

2023-01-28 Thread Sunil V L
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4076 PlatformHasAcpiDtDxe is required by other architectures also. Hence, it is moved to OvmfPkg. So, update the consumers of this module with the new location. Cc: Ard Biesheuvel Cc: Leif Lindholm Cc: Sami Mujawar Cc: Gerd Hoffmann Signed-o

[edk2-devel] [edk2-staging/RiscV64QemuVirt PATCH V7 11/20] ArmVirtPkg/PlatformHasAcpiDtDxe: Move to OvmfPkg

2023-01-28 Thread Sunil V L
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4076 This module is required by other architectures like RISC-V. Hence, move this to OvmfPkg. Cc: Ard Biesheuvel Cc: Jiewen Yao Cc: Jordan Justen Cc: Gerd Hoffmann Signed-off-by: Sunil V L --- ArmVirtPkg/ArmVirtPkg.dec

[edk2-devel] [edk2-staging/RiscV64QemuVirt PATCH V7 09/20] UefiCpuPkg/UefiCpuPkg.ci.yaml: Ignore RISC-V file

2023-01-28 Thread Sunil V L
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4076 RISC-V register names do not follow the EDK2 formatting. So, add it to ignore list for now. Cc: Eric Dong Cc: Ray Ni Cc: Rahul Kumar Cc: Gerd Hoffmann Signed-off-by: Sunil V L Acked-by: Abner Chang --- UefiCpuPkg/UefiCpuPkg.ci.yaml |

[edk2-devel] [edk2-staging/RiscV64QemuVirt PATCH V7 08/20] UefiCpuPkg/CpuTimerLib: Add RISC-V instance

2023-01-28 Thread Sunil V L
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4076 This is mostly copied from edk2-platforms/Silicon/RISC-V/ProcessorPkg/Library/RiscVTimerLib Cc: Eric Dong Cc: Ray Ni Cc: Rahul Kumar Cc: Daniel Schaefer Cc: Abner Chang Cc: Gerd Hoffmann Signed-off-by: Sunil V L Acked-by: Abner Chang

[edk2-devel] [edk2-staging/RiscV64QemuVirt PATCH V7 07/20] UefiCpuPkg/CpuDxe: Add RISC-V instance

2023-01-28 Thread Sunil V L
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4076 This is copied from edk2-platforms/Silicon/RISC-V/ProcessorPkg/Universal/CpuDxe Cc: Eric Dong Cc: Ray Ni Cc: Rahul Kumar Cc: Daniel Schaefer Cc: Gerd Hoffmann Signed-off-by: Sunil V L Acked-by: Abner Chang --- UefiCpuPkg/UefiCpuPkg.

[edk2-devel] [edk2-staging/RiscV64QemuVirt PATCH V7 06/20] UefiCpuPkg/CpuExceptionHandlerLib: Add RISC-V instance

2023-01-28 Thread Sunil V L
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4076 Add Cpu Exception Handler library for RISC-V. This is copied from edk2-platforms/Silicon/RISC-V/ProcessorPkg/Library/RiscVExceptionLib Cc: Eric Dong Cc: Ray Ni Cc: Rahul Kumar Cc: Daniel Schaefer Cc: Abner Chang Cc: Gerd Hoffmann Sign

[edk2-devel] [edk2-staging/RiscV64QemuVirt PATCH V7 05/20] UefiCpuPkg: Add CpuTimerDxe module

2023-01-28 Thread Sunil V L
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4076 This DXE module initializes the timer interrupt handler and installs the Arch Timer protocol. Cc: Eric Dong Cc: Ray Ni Cc: Rahul Kumar Cc: Daniel Schaefer Cc: Gerd Hoffmann Signed-off-by: Sunil V L Acked-by: Abner Chang --- UefiCpuP

[edk2-devel] [edk2-staging/RiscV64QemuVirt PATCH V7 04/20] UefiCpuPkg: Add RISCV_EFI_BOOT_PROTOCOL related definitions

2023-01-28 Thread Sunil V L
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4076 RISC-V UEFI based platforms need to support RISCV_EFI_BOOT_PROTOCOL. Add this protocol GUID definition and the header file required. Cc: Eric Dong Cc: Ray Ni Cc: Rahul Kumar Cc: Daniel Schaefer Cc: Gerd Hoffmann Signed-off-by: Sunil V

[edk2-devel] [edk2-staging/RiscV64QemuVirt PATCH V7 03/20] MdePkg: Add BaseRiscVSbiLib Library for RISC-V

2023-01-28 Thread Sunil V L
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4076 This library is required to make SBI ecalls from the S-mode EDK2. This is mostly copied from edk2-platforms/Silicon/RISC-V/ProcessorPkg/Library/RiscVEdk2SbiLib Cc: Michael D Kinney Cc: Liming Gao Cc: Zhiguang Liu Signed-off-by: Sunil V L

[edk2-devel] [edk2-staging/RiscV64QemuVirt PATCH V7 00/20] Add support for RISC-V virt machine

2023-01-28 Thread Sunil V L
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4076 Add support for RISC-V qemu virt machine. Most of the changes are migrated from edk2-platforms repo and added qemu specific libraries under OvmfPkg. The series has passed CI tests (https://github.com/tianocore/edk2/pull/3962) The series

[edk2-devel] [edk2-staging/RiscV64QemuVirt PATCH V7 02/20] MdePkg/BaseLib: RISC-V: Add few more helper functions

2023-01-28 Thread Sunil V L
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4076 Few of the basic helper functions required for any RISC-V CPU were added in edk2-platforms. To support qemu virt, they need to be added in BaseLib. Cc: Michael D Kinney Cc: Liming Gao Cc: Zhiguang Liu Cc: Daniel Schaefer Signed-off-by:

[edk2-devel] [edk2-staging/RiscV64QemuVirt PATCH V7 01/20] MdePkg/Register: Add register definition header files for RISC-V

2023-01-28 Thread Sunil V L
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4076 Add register definitions and access routines for RISC-V. These headers are leveraged from opensbi repo. Cc: Daniel Schaefer Cc: Michael D Kinney Cc: Liming Gao Cc: Zhiguang Liu Signed-off-by: Sunil V L Acked-by: Abner Chang --- MdePk

Re: [edk2-devel] [PATCH 1/3] MdeModulePkg/BmBoot: Skip removable media if it is not present

2023-01-28 Thread Sean Rhodes
Hi Ray Would it be possible to merge this? Thanks Sean On Fri, 16 Dec 2022, 09:03 Ni, Ray, wrote: > Reviewed-by: Ray Ni > > > > -Original Message- > > From: devel@edk2.groups.io On Behalf Of Sean > Rhodes > > Sent: Friday, December 16, 2022 4:58 PM > > To: devel@edk2.groups.io > > C

[edk2-devel] [PATCH V5 13/13] OvmfPkg: Support Tdx measurement in OvmfPkgX64

2023-01-28 Thread Min Xu
From: Min M Xu BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=4243 This patch enables Tdx measurement in OvmfPkgX64 with below changes: 1) TDX_MEASUREMENT_ENABLE is introduced in OvmfPkgX64.dsc. This flag indicates if Intel TDX measurement is enabled in OvmfPkgX64. Its default value is

[edk2-devel] [PATCH V5 12/13] OvmfPkg/PlatformPei: Build GuidHob for Tdx measurement

2023-01-28 Thread Min Xu
From: Min M Xu BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=4243 TdxHelperBuildGuidHobForTdxMeasurement is called in PlatformPei to build GuidHob for Tdx measurement. Cc: Erdem Aktas Cc: James Bottomley Cc: Jiewen Yao Cc: Gerd Hoffmann Cc: Tom Lendacky Cc: Michael Roth Acked-by: Ger

[edk2-devel] [PATCH V5 11/13] OvmfPkg/OvmfPkgX64: Measure TdHob and Configuration FV in SecMain

2023-01-28 Thread Min Xu
From: Min M Xu BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=4243 TdHob and Configuration FV (Cfv) are external inputs from VMM. From the security perspective, they should be measured before they're consumed. This patch measures TdHob and Cfv and stores the measurement values in WorkArea.

[edk2-devel] [PATCH V5 10/13] OvmfPkg/IntelTdx: Add PeiTdxHelperLib

2023-01-28 Thread Min Xu
From: Min M Xu BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=4243 TdxHelperLib provides below helper functions for a td-guest. - TdxHelperProcessTdHob - TdxHelperMeasureTdHob - TdxHelperMeasureCfvImage - TdxHelperBuildGuidHobForTdxMeasurement PeiTdxHelperLib is the PEI instance of TdxH

[edk2-devel] [PATCH V5 09/13] OvmfPkg/PeilessStartupLib: Delete the duplicated tdx measurement

2023-01-28 Thread Min Xu
From: Min M Xu BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=4243 After TdHob and Configuration FV (Cfv) are measured in SecMain, the same measurements in PeilessStartupLib should be deleted. Cc: Erdem Aktas Cc: James Bottomley Cc: Jiewen Yao Cc: Gerd Hoffmann Cc: Tom Lendacky Cc: Mic

[edk2-devel] [PATCH V5 08/13] OvmfPkg/IntelTdx: Measure TdHob and Configuration FV in SecMain

2023-01-28 Thread Min Xu
From: Min M Xu BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=4243 TdHob and Configuration FV (Cfv) are external inputs from VMM. From the security perspective, they should be measured before they're consumed. This patch measures TdHob and Cfv and stores the measurement values in WorkArea.

[edk2-devel] [PATCH V5 07/13] OvmfPkg: Refactor ProcessHobList

2023-01-28 Thread Min Xu
From: Min M Xu BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=4243 ProcessHobList once was implemented in PlatformInitLib and it walks thru TdHob list and accept un-accepted memories. This patch moves the codes to SecTdxHelperLib and rename ProcessHobList as TdxHelperProcessTdHob After Tdx

[edk2-devel] [PATCH V5 06/13] OvmfPkg: Refactor MeaureFvImage

2023-01-28 Thread Min Xu
From: Min M Xu BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=4243 MeasureFvImage once was implemented in PeilessStartupLib and it does measurement and logging for Configuration FV (Cfv) image in one go, using TpmMeasureAndLogData(). But it doesn't work in SEC. This patch splits MeasureFvIm

[edk2-devel] [PATCH V5 05/13] OvmfPkg: Refactor MeasureHobList

2023-01-28 Thread Min Xu
From: Min M Xu BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=4243 MeasureHobList once was implemented in PeilessStartupLib and it does measurement and logging for TdHob in one go, using TpmMeasureAndLogData(). But it doesn't work in SEC. This patch splits MeasureHobList into 2 functions an

[edk2-devel] [PATCH V5 04/13] OvmfPkg/PeilessStartupLib: Update the define of FV_HANDOFF_TABLE_POINTERS2

2023-01-28 Thread Min Xu
From: Min M Xu BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=4243 FV_HANDOFF_TABLE_POINTERS2 once was defined in IntelTdx.c. Its structure is same as PLATFORM_FIRMWARE_BLOB2_STRUCT which is defined in Library/TcgEventLogRecordLib.h. So this patch reuse the define of PLATFORM_FIRMWARE_BLOB2_

[edk2-devel] [PATCH V5 03/13] OvmfPkg/IntelTdx: Add SecTdxHelperLib

2023-01-28 Thread Min Xu
From: Min M Xu BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=4243 TdxHelperLib provides below helper functions for a td-guest. - TdxHelperProcessTdHob - TdxHelperMeasureTdHob - TdxHelperMeasureCfvImage - TdxHelperBuildGuidHobForTdxMeasurement SecTdxHelperLib is the SEC instance of TdxH

[edk2-devel] [PATCH V5 02/13] OvmfPkg/IntelTdx: Add TdxHelperLibNull

2023-01-28 Thread Min Xu
From: Min M Xu BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=4243 TdxHelperLib provides below helper functions for a td-guest. - TdxHelperProcessTdHob - TdxHelperMeasureTdHob - TdxHelperMeasureCfvImage - TdxHelperBuildGuidHobForTdxMeasurement TdxHelperLibNull is the NULL instance of Td

[edk2-devel] [PATCH V5 01/13] OvmfPkg: Add Tdx measurement data structure in WorkArea

2023-01-28 Thread Min Xu
From: Min M Xu BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=4243 >From the perspective of security any external input should be measured and extended to some registers (TPM PCRs or TDX RTMR registers). There are below 2 external input in a Td guest: - TdHob - Configuration FV (CFV) TdH

[edk2-devel] [PATCH V5 00/13] Enable Tdx measurement in OvmfPkgX64

2023-01-28 Thread Min Xu
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=4243 Tdx measurement (RTMR based measurement) is enabled in OvmfPkg/IntelTdx. This patch-set enables the feature in OvmfPkgX64 as well. Patch #1: Introduce TDX_MEASUREMETNS_DATA in SEC_TDX_WORK_AREA. That is because the RTMR measurement of Td

Re: [edk2-devel] [PATCH V4 06/12] OvmfPkg/PeilessStartupLib: Build GuidHob for Tdx measurement

2023-01-28 Thread Min Xu
On January 27, 2023 3:54 PM, Gerd Hoffmann wrote: > On Fri, Jan 27, 2023 at 08:11:00AM +0800, Min Xu wrote: > > From: Min M Xu > > > > BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=4243 > > > > 2 new functions are added in PeilessStartupLib/IntelTdx.c. > > - BuildTdxMeasurementGuidHob > > -