Skip to content

Conversation

@torgeiru
Copy link
Contributor

Driver testing enhancement:
Making so that you can write drivers in your testing unikernel. Currently you need to modify large portions of the codebase to hack a driver and it needs to be part of source. This allows constructing driver objects directly in your main/service.

Minor refactoring:
Moving defines from source to header files

Driver testing enhancement:
Making so that you can write drivers in your testing unikernel.
Currently you need to modify large portions of the codebase
to hack a driver and it needs to be part of source. This
allows constructing driver objects directly in your main/service.

Minor refactoring:
Moving defines from source to header files
@torgeiru
Copy link
Contributor Author

#2328

@mazunki
Copy link
Contributor

mazunki commented Oct 28, 2025

It's not a problem this time, but do keep in mind squashing commits makes it harder to review the changes. Making one commit to move the defines from one file to another, and another to add new features would have made it easier to track changes and be able to verify what's new and what's just refactored.

I'll see if the tests pass in a minute.

Also, for reference: #2333.

@mazunki
Copy link
Contributor

mazunki commented Oct 28, 2025

Tests are green :)

Copy link
Member

@MagnusS MagnusS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm - see also previous discussion in #2328

@MagnusS MagnusS merged commit 20c3133 into includeos:main Oct 29, 2025
@torgeiru torgeiru deleted the driver-testing-enhancement branch October 29, 2025 14:28
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.

3 participants