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

uefi(8): Refer to a related manual page #832

Closed
wants to merge 1 commit into from

Conversation

tomhukins
Copy link
Contributor

PR: 265834
Differential Revision: https://reviews.freebsd.org/D39577

@concussious
Copy link
Contributor

concussious commented Aug 31, 2023

I've been working on this for a while, I intended to do more [0], but since you're working on it now, how do you feel about this [1]?

Particularly:

  • I've tried to (concisely!) improve the explanation of what UEFI is, because people don't know and this is super fundamental.
  • legacy bios -> traditional bios. I have a real problem with that wording, I'm running SeaBIOS and that is much more advanced than UEFI in a totally different design philosophy. It's instantaneous, supports initializing every type of hardware I know of, it has a absolutely minuscule library (kbs, thousands of times smaller than the EFI gpu drivers) that can initialize any graphical hw I've ever seen. It has no application interface, so it's hardened by design massively more than EFI, and since it's so tiny it makes several copies of itself on the ROM in case of corruption. Really, this is so massively more advanced than EFI, I refuse to buy HW that doesn't support it.

[0] I think this page needs to explain more clearly the relationship between this and the other loaders. Does this loader load loader.lua or does it have the lua support in it?

[1] 498506d

@tomhukins
Copy link
Contributor Author

I've been working on this for a while, I intended to do more [0], but since you're working on it now, how do you feel about this [1]?

I don't have a great deal of knowledge about UEFI, but I welcome your efforts to improve FreeBSD's documentation.

In February, I noticed bug 265834 and submitted a small patch that would complete the work required to close that bug. After several rounds of patching, my work remains uncommitted, so I decided to submit a pull request for it. I've had more luck getting my work committed when I submit pull requests.

@mhorne
Copy link
Contributor

mhorne commented Sep 4, 2023

Thanks Tom. I grabbed your commit and pushed to main.

You should be able to close this PR, but I cannot.

@tomhukins
Copy link
Contributor Author

Merged to main in f49d6f5.

@tomhukins tomhukins closed this Sep 4, 2023
@tomhukins tomhukins deleted the loader_man branch September 5, 2023 17:16
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.

3 participants