Merged as commit 7e6f150b6902, via <https://github.com/tianocore/edk2/pull/890>. Thanks Laszlo

Whoops, I failed to file this away in my queue - apologies. Many thanks for fixing this in opensbi. Reviewed-by: Leif Lindholm <leif@...>

Minor typo below: Drop "a"? Reviewed-by: Leif Lindholm <leif@...>

Jiewen, I don’t completely agree with your second point. If the underlying issue is already out of embargo, and we’ve just failed to fix it, that’s not a new issue. I would want to see fixes

Hi Vladimir, I am inprog of going through the patch. There are some coding standard violations. For reference: https://edk2-docs.gitbook.io/edk-ii-c-coding-standards-specification/ Http.c: Lines

Reviewed-by: Jian J Wang <jian.j.wang@...> Regards, Jian

Reviewed-by: Jian J Wang <jian.j.wang@...> Regards, Jian

Jian, I want the BaseTools maintainers to evaluate these build requirements. We already have a number of tool dependencies. Adding one more for perl may be acceptable. Mike

Christopher, Can you provide the details on how to generate the IA32 and X64 files? What is the additional parameter? Thanks, Mike

I dug up the original report details. This was noted as a concern during a source code inspection. There was no demonstration of how it might be triggered. " There is an integer overflow

I agree with the process and withdraw my request, replacing it instead with a disapproving head shake and deep sigh. I’ll go back to pushing on this after the tag. - Bret

https://bugzilla.tianocore.org/show_bug.cgi?id=2902 Replace BSD 2-Clause License with BSD+Patent License. This change is based on https://bugzilla.tianocore.org/show_bug.cgi?id=1373 and

Hi Laszlo I agree with on your most points. It is all about *return of investment* or *risk control*. Like we cannot pursue 100% security because we will bankrupt ourselves if so. Here I just raise

Reviewed-by: Chasel Chiu <chasel.chiu@...>

From: Tom Lendacky <thomas.lendacky@...> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2901 The DoDecrement variable in ApWakeupFunction () wasn't always being initialized. Update the code

Copyright year of IntelFsp2WrapperPkg.dec can be extend to 2020 when pushing. Reviewed-by: Chasel Chiu <chasel.chiu@...>

Reviewed-by: Chasel Chiu <chasel.chiu@...>

Reviewed-by: Chasel Chiu <chasel.chiu@...>

Reviewed-by: Chasel Chiu <chasel.chiu@...>

Hi Liming, Yes, fix is on the way. Thanks, Tom