-
Notifications
You must be signed in to change notification settings - Fork 122
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
Add pcap example, add ixy-ci config, and fix some Virtio bugs #18
Conversation
Certain VM setups filter out packets that do this which breaks our example.
The descriptor size includes the net_hdr but isn't part of the data the user receives.
Fixes the issue where packets were duplicated when the tx queue was drained (e.g. by sleeping in pktgen between batches).
Test passed! pktgen logs
fwd logs
pcap logs
|
Cool |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please change indent from spaces to tabs to be consistent with the other code in the repo
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do you want me to squash it into a single commit or do you want separate commits? please rebase and squash into a few commits if you want more than one commit
Let's just squash it into a single commit. I'd also suggest to integrate ixy-ci into this repository while we're at it. See here for how to set up the webhook. |
@ixy-ci test |
1 similar comment
@ixy-ci test |
Test passed! pktgen logs
fwd logs
pcap logs
|
As stated in the title this adds everything needed for ixy-ci and also fixes the first bugs found by it :)
The IO handling of the pcap example is rather naive and not optimized for performance so it doesn't really satisfy the README wish list entry (
tcpdump
-like example based onmmap
/fallocate
). It does suffice for ixy-ci though.I'll manually start an ixy-ci run on this PR for demonstration purposes. Later you'll have to add a GitHub webhook for this repository to start test runs.