Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Add (partial) safe protocol implementation for EFI_HII_DATABASE_PROTOCOL#1719

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
seijikun wants to merge1 commit intorust-osdev:main
base:main
Choose a base branch
Loading
fromseijikun:mr-hii-database

Conversation

seijikun
Copy link
Contributor

This only grants access to the HII-database's raw buffer for now.

Checklist

  • Sensible git history (for example, squash "typo" or "fix" commits). See theRewriting History guide for help.
  • Update the changelog (if necessary)

@seijikunseijikunforce-pushed themr-hii-database branch 3 times, most recently from144e2a9 to917c7a0CompareJuly 8, 2025 22:44
@seijikun
Copy link
ContributorAuthor

This is one of the last things we (IP-Projects) need ... for now.
Our end goal with this protocol is to be able to read the collection of available BIOS settings (+ their values), which requires to fully parse the hiidb datastructure.

Though I'm somewhat at a loss as to how to continue with this.
I think parsing hiidb without the help of some library is abig no-no, sincethis is what it looks likewith a helping library.

Would it be acceptable to add a new dependency on something likebinrw to add this to uefi-rs, or do you regard hiidb parsing as out-of-scope for uefi-rs alltogether?

phip1611 reacted with rocket emoji

@phip1611
Copy link
Member

phip1611 commentedJul 9, 2025
edited
Loading

Would it be acceptable to add a new dependency on something likebinrw to add this to uefi-rs, or do you regard hiidb parsing as out-of-scope for uefi-rs alltogether?

Not sure, we haven't done this so far. I have a light tendency towardsno. In the documentation of the corresponding "high level" abstraction inuefi (pub struct HiiDatabase(HiiDatabaseProtocol);, you could add some more context for higher-level usages?

@seijikunseijikunforce-pushed themr-hii-database branch 3 times, most recently frome721b59 toeb3bfe1CompareJuly 9, 2025 10:05
This only grants access to the HII-database's raw buffer for now.
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@phip1611phip1611phip1611 left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

2 participants
@seijikun@phip1611

[8]ページ先頭

©2009-2025 Movatter.jp