microsoft / mu_plus

Project Mu Microsoft Core UEFI Value
https://microsoft.github.io/mu/
Other
214 stars 100 forks source link

AdvLoggerPkg: Modify AdvancedLib Dxe initialization. #549

Closed VivianNK closed 2 months ago

VivianNK commented 2 months ago

Description

Modify Advanced Logger DXE Library. Detect invalid Advanced Logger Protocol earlier, and allow to fail gracefully and fallback to DebugPortProtocol.

How This Was Tested

Local CI

Integration Instructions

N/A

codecov-commenter commented 2 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Please upload report for BASE (release/202405@cf4b663). Learn more about missing BASE report.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## release/202405 #549 +/- ## ================================================= Coverage ? 11.04% ================================================= Files ? 144 Lines ? 22022 Branches ? 2355 ================================================= Hits ? 2432 Misses ? 19558 Partials ? 32 ``` | [Flag](https://app.codecov.io/gh/microsoft/mu_plus/pull/549/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=microsoft) | Coverage Δ | | |---|---|---| | [AdvLoggerPkg](https://app.codecov.io/gh/microsoft/mu_plus/pull/549/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=microsoft) | `3.65% <ø> (?)` | | 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.