Totally agree with this. Confidential Computing is a very niche use case, and there is no justification for exploding the complexity of the standard OVMF build. If, several years from now, it ever

Thank you Laszlo for reviewing this. James explained the difference from Secure Boot setup and our choice of hash validation of kernel+initrd+cmdline. [Skipping...] Brijesh's approach mandates

thank you Laszlo. Your feedback is received. I am waiting for comment from other people. thank you! Yao, Jiewen

I mean option (B). Create a completely separate DSC+FDF for Intel TDX. In my mind, there are two (very high level) stages for developing the "Confidential Computing with TDX" feature in edk2. Stage

From SysTableInfo Hob, get ACPI table address, and creat gPldAcpiTableGuid = Hob to store it. Remove diretly adding ACPI table to ConfigurationTable. Dxe ACPI driver will parse it and install ACPI

V1: If HOB contains APCI table information, entry point of AcpiTableDxe.inf should parse the APCI table from HOB, and install these tables. We assume the whole ACPI table (starting with

Cc: Liming Gao <gaoliming@...> Cc: Jian J Wang <jian.j.wang@...> Cc: Hao A Wu <hao.a.wu@...> Signed-off-by: Zhiguang Liu <zhiguang.liu@...> ---

From SysTableInfo Hob, get Smbios table address, and creat gPldSmbiosTableG= uid Hob to store it. Remove diretly adding smbios table to ConfigurationTable. Dxe module SmbiosDxe will parse it and

V1: The default EfiSmbiosProtocol operates on an empty SMBIOS table. The SMBIOS tables are provided by the bootloader on UefiPayloadPkg. Scan for existing tables in SmbiosDxe and load them if they

Cc: Liming Gao <gaoliming@...> Cc: Jian J Wang <jian.j.wang@...> Cc: Hao A Wu <hao.a.wu@...> Signed-off-by: Zhiguang Liu <zhiguang.liu@...> ---

UefiPayload parse gPldPciRootBridgeInfoGuid Guid Hob to retrieve PCI root b= ridges information. gPldPciRootBridgeInfoGuid Guid Hob should be created by Bootlo= ader. Cc: Maurice Ma

Cc: Liming Gao <gaoliming@...> Cc: Jian J Wang <jian.j.wang@...> Cc: Hao A Wu <hao.a.wu@...> Signed-off-by: Zhiguang Liu <zhiguang.liu@...> ---

V1: This patch set is based on Universal Payload on https://universalpayload.github.io/documentation/payload-interfaces/index.html This patch set introduce one general header, three different hob

V1: Add Universal Payload general defination header file according to Universal Payload’s documentation V2: Add a macro funtion to check the Revision Cc: Liming Gao <gaoliming@...> Cc:

Hi Brijesh, That series ("[PATCH v3 00/13] Add GHCBv2 macro and helpers") has been merged at this point, as commit range dbc22a178546..adfa3327d4fc. [*] So, I'm having trouble applying this series.

Created a PR https://github.com/tianocore/edk2/pull/1688 Thanks, Bob

Internally reviewed this patch before sending the edk2 mailing list and it looks good to me. Reviewed-by: Sunny Wang <sunny.wang@...

Internally reviewed this patch before sending the edk2 mailing list and Greg already addressed all my comments, so It looks good to me. Reviewed-by: Sunny Wang <sunny.wang@...>

Internally reviewed this patch before sending the edk2 mailing list and It looks good to me. Please also address Pete's good catches/comments. Reviewed-by: Sunny Wang <sunny.wang@...>

Internally reviewed this patch before sending the edk2 mailing list and Greg already addressed all my comments, so It looks good to me. Reviewed-by: Sunny Wang <sunny.wang@...> As for Bret's