microsoft / mu_feature_mm_supv

Project Mu - Feature Repo - MM Supervisor
https://microsoft.github.io/mu
Other
46 stars 28 forks source link

Integrate EDK2 PiSmmCpuDxeSmm driver change #292

Closed kuqin12 closed 4 months ago

kuqin12 commented 4 months ago

Description

This change integrated the update from EDK2 where the MP information of the system will be pulled from the HOBs instead of gBS, which aligns with our existing implementation.

How This Was Tested

This change was tested on QEMU Q35 and booted to UEFI shell.

Integration Instructions

N/A

kuqin12 commented 4 months ago

This change requires an update of BASECORE from this PR: https://github.com/microsoft/mu_basecore/pull/839/files

codecov-commenter commented 4 months ago

Codecov Report

Attention: Patch coverage is 0% with 150 lines in your changes are missing coverage. Please review.

Project coverage is 0.47%. Comparing base (2f513c3) to head (0c5cfaa).

Files Patch % Lines
MmSupervisorPkg/Core/Relocate/Relocate.c 0.00% 150 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #292 +/- ## ======================================== - Coverage 0.47% 0.47% -0.01% ======================================== Files 144 144 Lines 21942 22066 +124 Branches 60 60 ======================================== Hits 105 105 - Misses 21832 21956 +124 Partials 5 5 ``` | [Flag](https://app.codecov.io/gh/microsoft/mu_feature_mm_supv/pull/292/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=microsoft) | Coverage Δ | | |---|---|---| | [MmSupervisorPkg](https://app.codecov.io/gh/microsoft/mu_feature_mm_supv/pull/292/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=microsoft) | `0.47% <0.00%> (-0.01%)` | :arrow_down: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=microsoft#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

kuqin12 commented 4 months ago

Depending on https://github.com/microsoft/mu_tiano_plus/pull/272 to resolve the pipeline error.