Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix the system boot to a boot manager menu after priority boot option returns with a status of EFI_SUCCESS #614

Conversation

liushuai20
Copy link

Description

It will present a boot manager menu to the user after priority boot option returns with a status of EFI_SUCCESS.

  • Impacts functionality?
  • Impacts security?
  • Breaking change?
  • Includes tests?
  • Includes documentation?
  • Backport to release branch?

How This Was Tested

IPMI boot to USB shell, exit on shell, it not present a boot manager menu.

Integration Instructions

N/A

@codecov-commenter
Copy link

codecov-commenter commented Dec 16, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 11.59%. Comparing base (184d495) to head (d2af9c4).

Additional details and impacted files
@@             Coverage Diff             @@
##           dev/202405     #614   +/-   ##
===========================================
  Coverage       11.59%   11.59%           
===========================================
  Files             132      132           
  Lines           21561    21555    -6     
  Branches         2546     2546           
===========================================
  Hits             2499     2499           
+ Misses          19028    19022    -6     
  Partials           34       34           
Flag Coverage Δ
MsCorePkg 1.44% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@liushuai20
Copy link
Author

not need, close

@liushuai20 liushuai20 closed this Dec 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants