Skip to content

Split MM PEI IPL into normal IPL and MM Supervisor PEI Support#577

Draft
kuqin12 wants to merge 41 commits into
microsoft:mainfrom
kuqin12:pei_n_ipl
Draft

Split MM PEI IPL into normal IPL and MM Supervisor PEI Support#577
kuqin12 wants to merge 41 commits into
microsoft:mainfrom
kuqin12:pei_n_ipl

Conversation

@kuqin12

@kuqin12 kuqin12 commented Dec 1, 2025

Copy link
Copy Markdown
Contributor

Description

This change splits the MM supervisor communication PPI into its own module rather than a combined MM PEI IPL driver.

This will allow the adoption of the StandaloneMmPeiIpl from EDK2 for X64 platforms.

For details on how to complete these options and their meaning refer to CONTRIBUTING.md.

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

How This Was Tested

This was tested on QEMU Q35 virtual platform.

Integration Instructions

Add MmSupervisorPkg/Drivers/MmPeiLaunchers/MmPeiSupport.inf to platform dsc and fdf file.

@mu-automation mu-automation Bot added the impact:non-functional Does not have a functional impact label Dec 1, 2025
@kuqin12 kuqin12 marked this pull request as draft December 1, 2025 21:09
@mu-automation mu-automation Bot added impact:breaking-change Requires integration attention and removed impact:non-functional Does not have a functional impact labels Dec 8, 2025
@kuqin12 kuqin12 changed the title [test] Using PEI IPL from StandaloneMmPkg Split MM PEI IPL into normal IPL and MM Supervisor PEI Support Dec 8, 2025
@kuqin12 kuqin12 marked this pull request as ready for review December 8, 2025 22:06
@kuqin12 kuqin12 requested review from apop5 and makubacki December 8, 2025 22:06
@codecov-commenter

codecov-commenter commented Dec 8, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 145 lines in your changes missing coverage. Please review.
✅ Project coverage is 0.52%. Comparing base (0d068c0) to head (8272fa2).

Files with missing lines Patch % Lines
...upervisorPkg/Drivers/MmPeiLaunchers/MmPeiSupport.c 0.00% 122 Missing ⚠️
MmSupervisorPkg/Drivers/MmPeiLaunchers/MmIplPei.c 0.00% 23 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff            @@
##            main    #577      +/-   ##
========================================
- Coverage   0.52%   0.52%   -0.01%     
========================================
  Files        132     133       +1     
  Lines      20067   20126      +59     
  Branches      60      60              
========================================
  Hits         106     106              
- Misses     19956   20015      +59     
  Partials       5       5              
Flag Coverage Δ
MmSupervisorPkg 0.52% <0.00%> (-0.01%) ⬇️

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.

@kuqin12

kuqin12 commented Dec 10, 2025

Copy link
Copy Markdown
Contributor Author

Needs #585 to complete first.

@mu-automation

mu-automation Bot commented Feb 8, 2026

Copy link
Copy Markdown
Contributor

This PR has been automatically marked as stale because it has not had activity in 60 days. It will be closed if no further activity occurs within 7 days. Thank you for your contributions.

@mu-automation mu-automation Bot added the state:stale Has not been updated in a long time label Feb 8, 2026
@mu-automation

mu-automation Bot commented Feb 16, 2026

Copy link
Copy Markdown
Contributor

This pull request has been automatically been closed because it did not have any activity in 60 days and no follow up within 7 days after being marked stale. Thank you for your contributions.

@mu-automation mu-automation Bot closed this Feb 16, 2026
@kuqin12

kuqin12 commented Feb 17, 2026

Copy link
Copy Markdown
Contributor Author

not stale...

@kuqin12 kuqin12 reopened this Feb 17, 2026
@mu-automation mu-automation Bot removed the state:stale Has not been updated in a long time label Feb 17, 2026
@mu-automation

mu-automation Bot commented Apr 18, 2026

Copy link
Copy Markdown
Contributor

This PR has been automatically marked as stale because it has not had activity in 60 days. It will be closed if no further activity occurs within 7 days. Thank you for your contributions.

@mu-automation mu-automation Bot added the state:stale Has not been updated in a long time label Apr 18, 2026
@mu-automation

mu-automation Bot commented Apr 25, 2026

Copy link
Copy Markdown
Contributor

This pull request has been automatically been closed because it did not have any activity in 60 days and no follow up within 7 days after being marked stale. Thank you for your contributions.

@mu-automation mu-automation Bot closed this Apr 25, 2026
@kuqin12 kuqin12 reopened this Jun 11, 2026
@kuqin12

kuqin12 commented Jun 11, 2026

Copy link
Copy Markdown
Contributor Author

need to rebase the commits, but the idea did not change that we want to split the current IPL into 2 so that the user and supv communication channels are separate.

@kuqin12 kuqin12 marked this pull request as draft June 11, 2026 23:37
@mu-automation mu-automation Bot removed the state:stale Has not been updated in a long time label Jun 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

impact:breaking-change Requires integration attention

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants