Well I have a problem regarding SerialDxe because it breaks a bit QEMU since adding it without removing the PciSerial registers two ways of reading from serial. From microvm, you simply removed

Well, depends. A separate target is extra maintainance effort. But having to write code for runtime-switching where compile-time switching would work without additional code is extra maintainance

Sounds good, I'll start working on this. --------------------------------------------------------------------- Intel Corporation SAS (French simplified joint stock company) Registered headquarters:

Looking at current OvmfPkg today. We

Hi all, So far I've been able to patch the OvmfPkgX64 target to make it work for both QEMU and Cloud Hypervisor, but as I try to enable more features (EFI shell for instance) the gap is getting

Oh I see, that's great actually because as I mentioned earlier this is not Xen specific, therefore I really think the renaming you're doing

REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3D3796 Database.c: 1. Replace PcdGetExPtr with PcdGetExPtr. 2. Add FindAuthVariableData function to parse authenticated variable type = for

MdeModulePkg/HiiDatabaseDxe: Add Support for authenticated variable Database.c: 1. Replace PcdGetExPtr with PcdGetExPtr. 2. Add FindAuthVariableData function to parse authenticated variable type

Yes. Your patches add a PlatformInitHob because of that. I think right now it only has some tdx-specific variables, but we can move more variables into the hob to allow platform init code run in

Pushed via: PR - https://github.com/tianocore/edk2/pull/2399 Commit - https://github.com/tianocore/edk2/commit/6062002bd5a394fef46243dd866860c3480d918e Best Regards, Hao Wu

Should this be This instance "uses" ? Why opensbiSBI? Shouldn't this be just openSBI? Remove extra "This instance"

Any reason to have new GUID? I think space is required between multi line help string. Same comment for other places. Should this Hertz instead of Herz? Regards Sunil

belone -> belongs? find -> finds Regards Sunil

The comment should be size instead of base address. Also, better to be consistent to use EDK2 OpenSBI or just OpenSBI. This is not Root Firmware Domain, correct? Should it be explicit to

Update the comment to remove U5? Regards Sunil

Is there a reason not to remove this? Regards Sunil

REF: https://https://bugzilla.tianocore.org/show_bug.cgi?id=3795 CC: Dandan Bi <dandan.bi@...> CC: Liming Gao <gaoliming@...> Updated CoreInternalAllocatePages() to call

The SEV-ES bit of Fn800-001F[EAX] - Bit 3 is used for a host to determine support for running SEV-ES guests. It should not be checked by a guest to determine if it is running under SEV-ES. The guest

Cc: Liming Gao <gaoliming@...> Signed-off-by: Zhiguang Liu <zhiguang.liu@...> --- EDK-II-Development-Process.md | 60 +++++++++++++++++++++++++++++++++++++++= ++++++--------------- 1

Hi Gerd, What do you mean "the patch series"? Is this one? https://edk2.groups.io/g/devel/message/83488 It is still in review and hasn't been merged. Yes, we use CSM_ENABLE to switch the lapic timer