Hi Patrick, UefiPayloadPkg/Library/SpiFlashLib/SpiFlashLib.inf is a common module as the commit message mentioned as below: * UefiPayloadPkg: Add SpiFlashLib This is a common SPI Flash library used

Adds following changes 1) Added support for manufacturing mode in BIOS Characteristics Extension Byte 2. 2) Added support for slot height in System Slot Table (type 9). 3) Updated Built-in

Hi Liming, I didnt reliaze that patch v2 was empty. Sending patch v3 after rebase. Thanks AbduL Cc: Michael D Kinney <michael.d.kinney@...> Cc: Liming Gao <gaoliming@...> Cc:

It's quite hard to use VarStore.fdf.inc since I would need to replace only the first DATA section. And I can't include something specific like OvmfPkg/CloudHv/CloudHvElfHeader.fdf.inc inside

Well Cloud Hypervisor aims at supporting TDX, but based on a separate firmware. We're not trying to support both TDX and non-TDX use cases with the same binary, which simplifies the problem

I took care of that in the v3. --------------------------------------------------------------------- Intel Corporation SAS (French simplified joint stock company) Registered headquarters: "Les

From: Sebastien Boeuf <sebastien.boeuf@...> Add some documentation to the CloudHv target in order to clarify how to use it and what to expect from it. Signed-off-by: Sebastien Boeuf

From: Sebastien Boeuf <sebastien.boeuf@...> Instead of using the CMOS, the CloudHv platform relies on the list of memmap entries provided through the PVH boot protocol to determine the last RAM

From: Sebastien Boeuf <sebastien.boeuf@...> Instead of hardcoding the address of the RSDP in the firmware, let's rely on the PVH structure hvm_start_info to retrieve this

From: Sebastien Boeuf <sebastien.boeuf@...> Following the model from the Xen target, CloudHv is generated as a PVH ELF binary to take advantage of the PVH specification, which requires less

From: Sebastien Boeuf <sebastien.boeuf@...> Updating the fdf include file based on the run of the ELF header generator. The diff from this patch is the result of: $ gcc -o elf_gen

From: Sebastien Boeuf <sebastien.boeuf@...> Instead of having the PVH ELF header part of the fdf file directly, we move it to a dedicated include file. This is the first step in automating the

From: Sebastien Boeuf <sebastien.boeuf@...> Adding some flexibility to the program through optional parameters and global define, so that other targets can use the generator. * A global define

From: Sebastien Boeuf <sebastien.boeuf@...> Cloud Hypervisor aims at emulating the minimal amount of legacy devices and this is why the PVH boot specification is supported. The point is to be

(+ Hao Wu) <quic_tpilar@...> wrote: This is still not exactly what I had in mind, but I think it's actually better: the manipulation of the MMIO/IO decode and bus master attributes occurs in

Please update the copyright year to 2022 (line 5). With it addressed, Reviewed-by: Jian J Wang <jian.j.wang@...> Regards, Jian

Move the logic that stores starting PCI attributes and sets the EFI_PCI_IO_ATTRIBUTE_DUAL_ADDRESS_CYCLE attribute to DriverBindingStart() before the memory that backs the DMA engine is

Wait no. I just cannot read code, ignore that last bit. I'll prepare a patch.

Also, doesn't this code:   if (!EFI_ERROR (Status)) {     Supports &= (UINT64)EFI_PCI_DEVICE_ENABLE;     Status    = PciIo->Attributes (                         

Okay, I was wondering. My thought was that we probably want to re-enable bus mastering on reset so I kept that bit of code in the original location.