Skip to content

[CHERRY-PICK] MdeModulePkg: Allow dynamic generation of HEST ACPI table#1659

Merged
os-d merged 1 commit intomicrosoft:release/202511from
os-d:hestdxe
Feb 26, 2026
Merged

[CHERRY-PICK] MdeModulePkg: Allow dynamic generation of HEST ACPI table#1659
os-d merged 1 commit intomicrosoft:release/202511from
os-d:hestdxe

Conversation

@os-d
Copy link
Contributor

@os-d os-d commented Feb 26, 2026

Description

Introduce the HEST table generation protocol that allows platforms to build the table with multiple error source descriptors and install the table. The protocol provides two interfaces. The first interface allows for adding multiple error source descriptors into the HEST table. The second interface can then be used to dynamically install the fully populated HEST table. This allows multiple drivers and/or libraries to dynamically register error source descriptors into the HEST table.

Note: This is cherry-picked from https://git.gitlab.arm.com/arm-reference-solutions/edk2/-/commit/2dcd7973c295ec7e2a16b17126e6c5f5bbdfe98c not edk2. The owners are being worked with to upstream this to edk2.

  • Impacts functionality?
  • Impacts security?
  • Breaking change?
  • Includes tests?
  • Includes documentation?

How This Was Tested

Tested on physical ARM64 platform.

Integration Instructions

N/A.

Copy link
Member

@makubacki makubacki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I know this is cherry-picked but I left some comments to fix and take as feedback. Looks like CI caught the library not being built in MdeModulePkg.dsc.

Introduce the HEST table generation protocol that allows platforms to
build the table with multiple error source descriptors and install the
table. The protocol provides two interfaces. The first interface allows
for adding multiple error source descriptors into the HEST table. The
second interface can then be used to dynamically install the fully
populated HEST table. This allows multiple drivers and/or libraries to
dynamically register error source descriptors into the HEST table.

Co-authored-by: Thomas Abraham <thomas.abraham@arm.com>
Signed-off-by: Omkar Anand Kulkarni <omkar.kulkarni@arm.com>
Signed-off-by: Oliver Smith-Denny <osde@microsoft.com>
@os-d os-d enabled auto-merge (rebase) February 26, 2026 21:52
@os-d os-d merged commit a460671 into microsoft:release/202511 Feb 26, 2026
51 checks passed
@os-d os-d deleted the hestdxe branch February 26, 2026 22:25
@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 0% with 176 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (release/202511@2085528). Learn more about missing BASE report.

Files with missing lines Patch % Lines
MdeModulePkg/Universal/Apei/HestDxe/HestDxe.c 0.00% 176 Missing ⚠️
Additional details and impacted files
@@                Coverage Diff                @@
##             release/202511    #1659   +/-   ##
=================================================
  Coverage                  ?    1.84%           
=================================================
  Files                     ?     1151           
  Lines                     ?   375732           
  Branches                  ?     3196           
=================================================
  Hits                      ?     6936           
  Misses                    ?   368740           
  Partials                  ?       56           
Flag Coverage Δ
FmpDevicePkg 9.53% <ø> (?)
MdeModulePkg 1.58% <0.00%> (?)
NetworkPkg 0.55% <ø> (?)
PolicyServicePkg 30.42% <ø> (?)
SecurityPkg 1.61% <ø> (?)
UefiCpuPkg 4.78% <ø> (?)
UnitTestFrameworkPkg 11.70% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants