microsoft / mu_feature_mm_supv

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

Port Recent MpService Changes [Rebase & FF] #290

Closed makubacki closed 6 months ago

makubacki commented 6 months ago

Description

Primarily made to pull in this change UefiCpuPkg/PiSmmCpuDxeSmm: Use NonSmm BSP as default SMM BSP..

Also includes other optimizations to keep the file in sync with edk2. The history of the recent changes in the file in this repo vs edk2 are shown for reference.

Commit URL Title Notes
2ca8d55974... UefiCpuPkg/PiSmmCpuDxeSmm: Check BspIndex first before lock cmpxchg New incoming change
d698bcfe4f... UefiCpuPkg/PiSmmCpuDxeSmm: Avoid BspIndex typecasting New incoming change
58d9463939... UefiCpuPkg/PiSmmCpuDxeSmm: Reduce one round BSP & AP sync New incoming change
41d1c4475b... UefiCpuPkg/PiSmmCpuDxeSmm: Invert ReleaseAllAPs & InitializeDebugAgent New incoming change
3a4ec6de01... UefiCpuPkg/PiSmmCpuDxeSmm: Align BSP and AP sync logic for SMI exit New incoming change
e1b62f3e28... UefiCpuPkg/PiSmmCpuDxeSmm: Check SMM Debug Agent support or not Skipped since it touches other files and is unrelated.
a83d953dc2... UefiCpuPkg/PiSmmCpuDxeSmm: Consume SmmCpuSyncLib Cherry-picked to basecore. Included in StandaloneMmCpuSyncLib PR (#257).
cc698d0335... UefiCpuPkg/PiSmmCpuDxeSmm: Simplify RunningApCount decrement Cherry-picked to basecore. Included in StandaloneMmCpuSyncLib PR (#257).
e14a022246... UefiCpuPkg/PiSmmCpuDxeSmm: Optimize Semaphore Sync between BSP and AP Included in StandaloneMmCpuSyncLib PR (#257)
68d506e0d1... UefiCpuPkg/PiSmmCpuDxeSmm: Use NonSmm BSP as default SMM BSP. New incoming change
b4dde1ae6a... UefiCpuPkg: Use GenSmmPageTable() to create Smm S3 page table Last commit in the 202311 update.

How This Was Tested

Integration Instructions

codecov-commenter commented 6 months ago

Codecov Report

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

Project coverage is 0.31%. Comparing base (23f3068) to head (c7c8276).

Files Patch % Lines
...mSupervisorPkg/Core/Services/MpService/MpService.c 0.00% 16 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #290 +/- ## ======================================== - Coverage 0.47% 0.31% -0.17% ======================================== Files 144 144 Lines 21934 21924 -10 Branches 60 60 ======================================== - Hits 105 69 -36 - Misses 21824 21828 +4 - Partials 5 27 +22 ``` | [Flag](https://app.codecov.io/gh/microsoft/mu_feature_mm_supv/pull/290/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/290/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=microsoft) | `0.31% <0.00%> (-0.17%)` | :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.