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

bootinfo: Add architecture-independent memory entry #212

Merged
merged 1 commit into from
Nov 28, 2022

Conversation

retrage
Copy link
Contributor

@retrage retrage commented Nov 27, 2022

E820Entry is used in boot::Info trait, which is x86_64-dependent memory entry data structure. It's confusing if e820 is used in other architectures. This commit introduces an architecture-independent memory entry bootinfo::MemoryEntry and replaces boot::Info with bootinfo::Info. This change is suggested in [1].

[1]
#205 (comment)

Signed-off-by: Akira Moroo [email protected]

@retrage retrage requested a review from rbradford November 27, 2022 10:19
`E820Entry` is used in `boot::Info` trait, which is x86_64-dependent
memory entry data structure. It's confusing if e820 is used in other
architectures. This commit introduces an architecture-independent memory
entry `bootinfo::MemoryEntry` and replaces `boot::Info` with
`bootinfo::Info`. This change is suggested in [1].

[1]
cloud-hypervisor#205 (comment)

Signed-off-by: Akira Moroo <[email protected]>
@retrage retrage merged commit a91205b into cloud-hypervisor:main Nov 28, 2022
@retrage retrage deleted the introduce-bootinfo branch November 28, 2022 13:43
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