microsoft / mu_feature_mm_supv

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

Update the core to compare hash with patched digest value during build #297

Closed kuqin12 closed 3 months ago

kuqin12 commented 4 months ago

Preface

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

Description

The change updated build plugin to calculate hashes for MMI entry block and MM supervisor core and generate the includable dsc file for the platform/build process to consume.

This change also updated the hash verification accordingly so as to facilitate the test application operation. The aux file, hashes are communicated to supervisor in lieu of being carried by the supervisor to avoid Jormungandr loop.

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 is tested on QEMU Q35 platform with a valid TPM interface and passed test application.

Integration Instructions

The release process should update the usage of plugin helper functions to generate the dsc function and include the generated DSC file into platform DSC and/or release builds.

codecov-commenter commented 4 months ago

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Please upload report for BASE (feature/spam@4c1c4fa). Learn more about missing BASE report.

Files Patch % Lines
...mSupervisorPkg/Library/SecurePolicyLib/MemPolicy.c 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## feature/spam #297 +/- ## ============================================== Coverage ? 0.30% ============================================== Files ? 144 Lines ? 22332 Branches ? 60 ============================================== Hits ? 69 Misses ? 22236 Partials ? 27 ``` | [Flag](https://app.codecov.io/gh/microsoft/mu_feature_mm_supv/pull/297/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/297/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=microsoft) | `0.30% <0.00%> (?)` | | 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.