Good feedback. Thank you very much, Sami. Response inline. I proposed some naming change. Please let us know if that is OK. Thank you Yao, Jiewen

Thanks Gerd. I like the idea to add more platforms to CI. Series: Acked-by: Jiewen Yao <Jiewen.yao@...> I am not CI person. We need CI expert to review the detail. Thank you Yao Jiewen

Yes, I tried again today. It works. https://github.com/tianocore/edk2/pull/2080 Pushed: 36b561623a4b8a6c7fea0b1b01f6789f2adf97e0.. 11a4af85a438b5b3d8cde942a209908219c57363 Thank you Yao Jiewen

That is weird. But I am not tool person. HI Liming/Mike/Sean Would you please take a look and help us understand what's happening? Thank you Yao Jiewen

Hi This patch failed in PR - https://github.com/tianocore/edk2/pull/2076 Please take a look. You are encouraged to try PR before submit next time. Thank you Yao Jiewen

Sure. It works fine. May I know *when* I can get your initial feedback ? I hope in days or 1 or 2 weeks. I don’t want to wait for several months, as we have features depend upon it. Thank you Yao Jiew

Series: Acked-by: Jiewen Yao <Jiewen.yao@...>

Hi This patch fails in the P-R - https://github.com/tianocore/edk2/pull/2073. Please double check. You are encourage to try P-R by yourself before submit the patch. Thank you Yao Jiewen

Acked-by: Jiewen Yao <Jiewen.yao@...>

OK. I recommend to add those background info in comments. Thank you Yao Jiewen

OK. I recommend to add those info in Bugzilla and the description as well.

Ok. Please allow me to clarify my description. "high memory" in this context means the memory region just below the less than 4GiB DRAM limit. In a platform BIOS, it means the memory just below TOLUD

Would you please help me understand what the relationship is between VirtioMmio and VirtioFdt ? The Bugzilla just states "add support for virtio-mmio devices". How VirtioFdt comes into this case? Than

Hi I am not sure where the problem is. "FdtClient is unhappy without a device tree, so add an empty fdt" Do we have option 2 to fix the issue? If no device tree, why we need FdtClient? Why we need mak

Sounds good. Then Bret, you may change https://github.com/tianocore/edk2/blob/master/SecurityPkg/SecurityPkg.ci.yaml, to exclude the RC_NV_* naming check. Thank you Yao Jiewen

We have to relocate based upon best practice. The mailbox must be in ACPI-NVS. And the ACPI-NVS should be in high memory. The original mailbox location is in low memory. If we do not relocate to high

Good suggestion, Gerd. I agree with both suggestion. We should rename XenTimerDxe to LocalApicTimerDxe.

I have fixed typo and push to https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Release-Planning

Reviewed-by: Jiewen Yao <Jiewen.yao@...> Since this https://bugzilla.tianocore.org/show_bug.cgi?id=3691 (networkpkg) is separated from https://bugzilla.tianocore.org/show_bug.cgi?id=3679 (crypto

Hi Liming/Mike Do you have any suggestion here? How do we change CI to add the name to exception list ? Thank you Yao Jiewen