Skip to content

Add a test OS #159

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

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

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?

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant