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.

Sign up for GitHub

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

Jump to bottom

Add a test OS #159

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

Add a test OS #159

wants to merge 122 commits into from

Conversation

toku-sa-n
Copy link
Member

@toku-sa-n toku-sa-n commented Nov 25, 2023 •
edited
Loading

The code is taken from https://github.com/toku-sa-n/ramen/tree/00193f21cdb72f37e5b4059a1e47c9be88992f5c. It is GPL-licensed, but this revision does not include contributions from the other contributors except bots and me, and thus it does not violate GPL as I'm the author of it.

  • Add a changelog?
  • Set up CI
  • Manually build QEMU with -DDEBUG_XHCI on CI with debug mode enabled
  • Add comments to refer xHCI specification for each step?
  • Is this actually necessary?

@toku-sa-n
Copy link
Member Author

I'm in doubt if we really need a test OS.

  • It has lots of code and is too difficult to read through.
  • We can test just a few structures. All the others are left untested.
  • First and foremost, what do we test? Structures have correct field getters/setters?

@toku-sa-n toku-sa-n mentioned this pull request Dec 20, 2023
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@toku-sa-n