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

Added some docs and an example for how to use the APIC #460

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

Voinea-Radu
Copy link

I do believe that this could be useful when migrating from version 0.9 to version 0.11, as well as a quick way to get things going if starting from nothing.

@Freax13
Copy link
Member

Freax13 commented Oct 12, 2024

Thank you for your contribution!

I'm not sure if we should adopt this example: This example goes way beyond what's needed to get started in setting up a small kernel with the bootloader. I'm not sure if we want this repo to serve as a place for various examples of next-steps one could take in kernel dev.

Perhaps this is better suited for the x2apic crate. Maybe it could live in its own repo entirely.

@Voinea-Radu
Copy link
Author

Hi there, thank you a lot for taking the time to look into this
I do understand that it might be a bit complex for a small kernel, but these were what I needed to port the 0.9 blog version to the 0.11 current version and would have liked to have had them just be available somewhere so I can read thru them. I do find that especially some of the structs that hold all the values are unnecessary for a small startup and would be more than happy to remove them along side any other future proofing

@Voinea-Radu Voinea-Radu marked this pull request as draft October 31, 2024 11:09
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