Hi, We are working on a feature that leverages UEFI HTTP protocol and during our testing we found an issue with the underlying DNS implementation. The current DNS implementation in EDK2 do not parse

Can we always enable CRYPTO_ENABLE? If yes, can we remove the macro?

Sami: I suggest to keep EFI_ACPI_IO_REMAPPING_TABLE_REVISION. Its value can be EFI_ACPI_IO_REMAPPING_TABLE_REV5, because the structure has been updated, such as EFI_ACPI_6_0_IO_REMAPPING_RC_NODE.

Reviewed-by: Liming Gao <gaoliming@...> Architecture set for the below recipient, distribution by

Mateusz: To be compatible, I suggest to add one new UefiDevicePathLibPeim.inf for PEIM module. It will be same to UefiDevicePathLibStandaloneMm.inf except for module type and base name. The

Thank you so much for the help, Jiewen! On 7/6/2022 6:09 PM, Yao, Jiewen wrote:

Merged https://github.com/tianocore/edk2/pull/3050

Hi Mike, How can sizeof(size_t) != sizeof(UINTN)? It seems to be quite a stretch to interpret the standard in a way that the native word size will not be able to store "the maximum size of a

Hi Pedro, This is an interesting feature. It is backwards compatible since you are adding a format specifier to the PrintLib class. There is a 2nd lib instance that needs to be updated, and that is

Hi Jiewen, Yes, the "https://github.com/kuqin12/edk2/tree/secure_boot_enhance_v3" is the branch I generate these patch series. And they have not been changed after sending

On Wed, Jul 6, 2022 at 02:57 AM, Sami Mujawar wrote: -#define EFI_ACPI_IO_REMAPPING_TABLE_REVISION 0x0 +#define EFI_ACPI_IO_REMAPPING_TABLE_REV0 0x0 +#define EFI_ACPI_IO_REMAPPING_TABLE_REV5 0x5Based

Hi Sami, Yes, that would also be okay. Thanks.

Hi Pierre, Thank you for bringing this up. As it stands the NXP platform would be using ACPI 6.4 which is the default case, and this patch will not change that. So, downgrading this to ACPI 6.2 may

Hi Thomas, Please find my response inline marked [SAMI]. Regards, Sami Mujawar [SAMI] I checked the edk2 code base and apparenlty the protocol versioning follows the scheme you mention above. As

Hello Sami, The only configuration manager not using ACPI 6.4 tables is at: Platform/NXP/ConfigurationManagerPkg/ConfigurationManagerDxe/ConfigurationManager.c I think the minor version of its FADT

Reviewed-by: Thomas Abraham <thomas.abraham@...> Tested-by: Thomas Abraham <thomas.abraham@...> Thanks.

nit: Should this have been EFI_ACPI_IO_REMAPPING_TABLE_REVISION_0 to better align with all the other table revision macros? Reviewed-by: Thomas Abraham <thomas.abraham@...> Thanks.

Invoke the constructor in the SEC phase to call into initialization functions associated with libraries linked with this particular module. For instance, PrePeiCore's CEntryPoint function calls

Reviewed-by: Lean Sheng Tan<sheng.tan@...>

Currently, "push byte %[Vector]" causes nasm warning when Vector is larger than 0x7F. This is because push accepts a signed value, and byte means signed int8. Maximum signed int8 is 0x7F. When Vector