devel@edk2.groups.io

[Patch V2 7/7] MdePkg/Test: Add port of BaseSafeIntLib unit tests to GoogleTest

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4134 Cc: Liming Gao <gaoliming@...> Cc: Zhiguang Liu <zhiguang.liu@...> Signed-off-by: Michael D Kinney

By Michael D Kinney · #95971 ·

[Patch V2 6/7] BaseTools/Plugin/HostBaseUnitTestRunner: Enable gtest xml output

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4134 Set environment variable GTEST_OUTPUT to specify the output format of XML and the output file name. Both CMOCKA_XML_FILE and GTEST_OUTPUT are

By Michael D Kinney · #95970 ·

[Patch V2 5/7] .pytool: Add googletest submodule to CISettings.py

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4134 Cc: Sean Brogan <sean.brogan@...> Cc: Michael Kubacki <mikuback@...> Cc: Liming Gao

By Michael D Kinney · #95969 ·

[Patch V2 4/7] UnitTestFrameworkPkg/Library/CmockaLib: Enable symbol files.

Cc: Michael Kubacki <mikuback@...> Cc: Sean Brogan <sean.brogan@...> Signed-off-by: Michael D Kinney <michael.d.kinney@...> Reviewed-by: Michael Kubacki

By Michael D Kinney · #95968 ·

[Patch V2 3/7] UnitTestFrameworkPkg: Add googletest submodule and GoogleTestLib

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4134 Add submodule for googletest and add GoogleTestLib that is required for GoogleTest based unit tests. Add GoogleTest documentation to Readme.md

By Michael D Kinney · #95967 ·

[Patch V2 2/7] MdePkg/Include/Library: Undefine _ASSERT() if already defined

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4134 When unit testing is enabled, make sure _ASSERT() is not already defined by the host environment before defining _ASSERT(). This avoids

By Michael D Kinney · #95966 ·

[Patch V2 1/7] MdePkg/Include: Update Base.h to improve C++ compatibility

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4134 * Map NULL to nullptr or __null when c++ compiler is used. * Map STATIC_ASSERT to static_assert when a c++ compiler is used. * Typecast

By Michael D Kinney · #95965 ·

[Patch V2 0/7] Add GoogleTest to UnitTestFrameworkPkg

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4134 PR: https://github.com/tianocore/edk2/pull/3595 Branch: https://github.com/mdkinney/edk2/tree/Bug_4134_AddGoogleTest_V2 Add GoogleTest

By Michael D Kinney · #95964 ·

Re: 回复: [PATCH v9 00/19] Add Raw algorithm support using Arm TRNG interface

<gaoliming@...> wrote: I will try to look at this before monday.

By Ard Biesheuvel · #95963 ·

Re: [Patch 3/7] UnitTestFrameworkPkg: Add googletest submodule and GoogleTestLib

Hi Michael, Here is a first pass at a feature matrix. Can you think of other features to add? ### Framework and GoogleTest Feature Comparison | Feature | Framework | GoogleTest

By Michael D Kinney · #95962 ·

[PATCH v2 1/2] MdePkg/IndustryStandard: add definitions for ACPI APMT

This adds #defines and struct typedefs for the various node types in the ACPI Arm Performance Monitoring Unit (APMT) table. Signed-off-by: Jeff Brasen <jbrasen@...> ---

By Jeff Brasen · #95961 ·

[PATCH v2 2/2] ShellPkg/AcpiView: APMT Parser

Add a new parser for the Arm Performance Monitoring Unit Table. The APMT table describes the properties of PMU support implemented by components in an Arm-based system. Signed-off-by: Jeff Brasen

By Jeff Brasen · #95960 ·

Re: [PATCH v2] CryptoPkg/Readme.md: typo and grammar fixes

Reviewed-by: Michael D Kinney <michael.d.kinney@...>

By Michael D Kinney · #95959 ·

Re: [Patch 5/7] .pytool: Add googletest submodule to CISettings.py

Reviewed-by: Michael Kubacki <michael.kubacki@...> The Cc list is out of date on this patch. Current .pytool maintainers: M: Sean Brogan <sean.brogan@...> [spbrogan] M: Michael Kubacki <mikuback@...>

By Michael Kubacki · #95958 ·

Re: [Patch 4/7] UnitTestFrameworkPkg/Library/CmockaLib: Enable symbol files.

Reviewed-by: Michael Kubacki <michael.kubacki@...>

By Michael Kubacki · #95957 ·

Re: [Patch 3/7] UnitTestFrameworkPkg: Add googletest submodule and GoogleTestLib

Reviewed-by: Michael Kubacki <michael.kubacki@...> There's a typo in UnitTestFrameworkPkg/ReadMe.md. You'll find it by searching for "ued". The ReadMe.md updates are very helpful. Something else to

By Michael Kubacki · #95956 ·

Re: [PATCH V2 1/9] UefiCpuPkg: Add CcExitLib

Sorry I didn't pick up on this before, but any reason this isn't CcExitLibHandleVc() to match the other APIs or have the other APIs just be CcExit (since the "Lib" part seems a bit excessive)? (Same

By Lendacky, Thomas · #95955 ·

Re: [Patch 6/7] BaseTools/Plugin/HostBaseUnitTestRunner: Enable gtest xml output

Reviewed-by: Bob Feng <bob.c.feng@...>

By Bob Feng · #95954 ·

Re: How to add a DXE driver to an OVMF image?

Using DebugLib instead to print a message, I was able to confirm that my driver is now running properly: cat debug.log | grep HelloThis works both when adding it in the OVMF build and also when

By d.meneses@... · #95953 ·

[PATCH v2] CryptoPkg/Readme.md: typo and grammar fixes

Commit 244ce33bdd2f ("CryptoPkg: Add Readme.md", 2022-10-24) had added the long-awaited documentation on the dynamic crypto services. Fix some of the typos and arguable grammar errors in "Readme.md".

By Laszlo Ersek · #95952 ·

Messages