ARM-software / ebbr

Embedded Base Boot Requirements Specification
Creative Commons Attribution Share Alike 4.0 International
113 stars 37 forks source link

Boot Manager requirements #132

Closed vstehle closed 4 days ago

vstehle commented 2 months ago

Let's clarify the requirements on the UEFI Boot Manager to address #130.

vstehle commented 2 months ago

As per EBBR call of Aug 26 this must be respun with better wording for BootOptionsSupport.

vstehle commented 1 month ago

Changes:

vstehle commented 2 weeks ago

Reworked after @xypron's comments (thanks!) and our call of Oct. 7. Please review and let me know how you like it.

Changes:

vstehle commented 2 weeks ago

Rebased on CI fixes.

xypron commented 2 weeks ago

@vstehle

Concerning BootOptionSupport the UEFI spec has "If the global variable is not present, then an installer or application must act as if a value of 0 was returned."

What relevance has your warning about requiring BootOptionSupport in this context?

vstehle commented 6 days ago

What relevance has your warning about requiring BootOptionSupport in this context?

Hi @xypron, I think you are right; all the wording around BootOptionSupport is confusing. As this variable is optional in UEFI already, let's simply leave it as it is. While at it, we can do the same for all the other boot manager features, already optional in UEFI. I think this makes the addition lighter; let me know how you like it.

vstehle commented 6 days ago

Changes:

vstehle commented 6 days ago

Rebased.

vstehle commented 4 days ago

Approved during the EBBR call of Oct 23.