Skip to content

Add Missing 2502 MTC Var Policy Change [Rebase & FF] - #1887

Merged
Michael Kubacki (makubacki) merged 2 commits into
microsoft:release/202511from
makubacki:cp_hddp_mtc_var_policy_2511
Aug 17, 2026
Merged

Add Missing 2502 MTC Var Policy Change [Rebase & FF]#1887
Michael Kubacki (makubacki) merged 2 commits into
microsoft:release/202511from
makubacki:cp_hddp_mtc_var_policy_2511

Conversation

@makubacki

Copy link
Copy Markdown
Member

Description

This 2502 commit had two changes missing from 2511 c087d24.

The first is the constructor missing in the UefiBootManagerLib.inf file and the MTC changes not in 2511.

  • The constructor is missing in edk2, apparently because the constructor was a separate Mu change made from the change cherry-picked to edk2. I've created a PR in edk2 to simply add the constructor there.

  • The MTC var policy changes are just missing from 2502 (not part of edk2). I'll go ahead and upstream them to edk2 to reduce the delta in the future.

  • Impacts functionality?

  • Impacts security?

  • Breaking change?

  • Includes tests?

  • Includes documentation?

How This Was Tested

  • Build and boot Q35

MTC variable before

image

MTC variable after

image

Integration Instructions

  • N/A

UefiBootManagerLibConstructor() is a constructor function in BmBoot.c
that is missing as the constructor in the INF file.

Signed-off-by: Michael Kubacki <michael.kubacki@microsoft.com>
@mu-automation

mu-automation Bot commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

✅ QEMU Validation Passed

Source Dependencies

Repository Commit
mu_basecore e2a6f5f
mu_tiano_platforms 7e81e99

Results

Platform Target Build Boot Overall Boot Time Build Logs Boot Logs
Q35 DEBUG ✅ success ✅ success 0m 19s Build Logs Boot Logs
ArmVirt DEBUG ✅ success ✅ success 0m 13s Build Logs Boot Logs

Workflow run: https://github.com/microsoft/mu_basecore/actions/runs/32068557354

This comment was automatically generated by the Mu QEMU PR Validation workflow.

@codecov-commenter

Codecov Comments Bot (codecov-commenter) commented Aug 17, 2026

Copy link
Copy Markdown

Codecov Report

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

Files with missing lines Patch % Lines
...rsal/MonotonicCounterRuntimeDxe/MonotonicCounter.c 0.00% 4 Missing ⚠️
Additional details and impacted files
@@                Coverage Diff                @@
##             release/202511    #1887   +/-   ##
=================================================
  Coverage                  ?    1.39%           
=================================================
  Files                     ?      802           
  Lines                     ?   305330           
  Branches                  ?     2308           
=================================================
  Hits                      ?     4267           
  Misses                    ?   301032           
  Partials                  ?       31           
Flag Coverage Δ
MdeModulePkg 1.65% <0.00%> (?)
NetworkPkg 0.55% <ø> (?)

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

☔ View full report in Codecov by Harness.
📢 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.

@makubacki
Michael Kubacki (makubacki) merged commit 762a65d into microsoft:release/202511 Aug 17, 2026
64 checks passed
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.

5 participants