microsoft / mu_feature_mm_supv

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

Integrate image record changes from MU_BASECORE #279

Closed kuqin12 closed 5 months ago

kuqin12 commented 5 months ago

Preface

Please ensure you have read the contribution docs prior to submitting the pull request. In particular, pull request guidelines.

Description

This change integrates 4 commits from MU_BASECORE to use the common library implementation for image record creation and resolve the corresponding override validation failures.

For each item, place an "x" in between [ and ] if true. Example: [x]. (you can also check items in the GitHub UI)

How This Was Tested

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

Integration Instructions

Platform needs to include ImagePropertiesRecordLib in their platform dsc file to pick up this change.

codecov-commenter commented 5 months ago

Codecov Report

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

Project coverage is 0.31%. Comparing base (496ea0b) to head (5ba26a2).

Files Patch % Lines
MmSupervisorPkg/Core/Misc/MemoryAttributesTable.c 0.00% 32 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #279 +/- ## ======================================== - Coverage 0.47% 0.31% -0.16% ======================================== Files 144 144 Lines 22335 21916 -419 Branches 60 60 ======================================== - Hits 105 69 -36 + Misses 22225 21820 -405 - Partials 5 27 +22 ``` | [Flag](https://app.codecov.io/gh/microsoft/mu_feature_mm_supv/pull/279/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/279/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.16%)` | :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.