Re: [edk2-devel] [PATCH] UefiCpuPkg/PiSmmCpu: Remove hardcode 48 address size limitation

2021-05-15 Thread Laszlo Ersek
On 05/15/21 02:04, Ni, Ray wrote: > Laszlo, > Do you think that another API is also needed: GetPhysicalAddressWidth() that > returns number 36/52? No. The GetPhysicalAddressBits() function that I proposed already returns this information. It has three outputs: the number of bits (that is, the wi

Re: [edk2-devel] [PATCH] UefiCpuPkg/MpInitLib: Allocate a separate SEV-ES AP reset stack area

2021-05-15 Thread Laszlo Ersek
On 05/14/21 17:44, Marvin Häuser wrote: > On 14.05.21 17:23, Lendacky, Thomas wrote: >> On 5/14/21 10:04 AM, Marvin Häuser wrote: +  // Check to be sure that the "allocate below" behavior hasn't changed. +  // This will also catch a failed allocation, as "-1" is returne

[edk2-devel] [PATCH v1 1/1] Add MemoryFence implementation for RiscV64

2021-05-15 Thread Daniel Schaefer
Cc: Abner Chang Cc: Michael D Kinney Cc: Liming Gao Cc: Zhiguang Liu Cc: Leif Lindholm Signed-off-by: Daniel Schaefer --- MdePkg/Library/BaseLib/BaseLib.inf | 1 + MdePkg/Library/BaseLib/RiscV64/MemoryFence.S | 33 2 files changed, 34 insertions(+) diff --git

Re: [edk2-devel] [PATCH v1 3/3] CryptoPkg/BaseCryptLib: Fix possible uninitialized use

2021-05-15 Thread Sergei Dmitrouk
Hello Jiewen, I get the error only for GCC49 and not for GCC5 toolchain. CI uses GCC5. So I compared build commands and this seems to depend on LTO. Adding `-flto` impedes compiler's ability to detect such simple issues. I've found relevant bug report, there is even fix suggestion from last mo