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

Implement virtio network device #1074

Closed
wants to merge 2 commits into from
Closed

Conversation

basicer
Copy link
Contributor

@basicer basicer commented Jul 7, 2024

Implement a virtio networking card. Currently only supports one set of buffer pairs but seems to be a bit faster than ne2k.

Tested with linux4 and the alpine linux_virt kernel.

@copy copy mentioned this pull request Jul 13, 2024
@basicer basicer force-pushed the virtio-net branch 2 times, most recently from 71018e4 to 8d34247 Compare August 27, 2024 03:58
@basicer basicer changed the title virtio_net device Implement virtio network device Aug 27, 2024
@basicer basicer marked this pull request as ready for review August 27, 2024 15:56
@copy
Copy link
Owner

copy commented Aug 31, 2024

FYI, I'm reworking the V86 constructor (to make more device configurations possible, e.g. disabling some default devices and allowing multiple network cards (with different backends)). I'm planning to merge this afterwards.

@basicer
Copy link
Contributor Author

basicer commented Sep 1, 2024

FYI, I'm reworking the V86 constructor (to make more device configurations possible, e.g. disabling some default devices and allowing multiple network cards (with different backends)). I'm planning to merge this afterwards.

Awesome! I was going to suggest that. The main interface was really showing its inflexibility.

@copy
Copy link
Owner

copy commented Sep 15, 2024

cherry-picked into the wip branch. FYI, the api change is here: 60e0d5c (I haven't implemented multiple network cards yet). Feedback would be welcome.

@copy copy closed this Sep 15, 2024
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