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

Experiment: enable USO and disable software checksums in winxdp #3809

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

Conversation

mtfriesen
Copy link
Contributor

Description

Estimate the best case performance of generic XDP on Windows with hardware USO, hardware checksums, and nearly zero-copy TX.

Testing

Absolutely none. I expect functional tests based on duonic will break, but hopefully the perf tests pass and produce a nice number.

Documentation

Experiment.

@codecov
Copy link

codecov bot commented Aug 12, 2023

Codecov Report

Merging #3809 (276b28f) into main (cf4537c) will increase coverage by 6.16%.
Report is 7 commits behind head on main.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #3809      +/-   ##
==========================================
+ Coverage   80.42%   86.58%   +6.16%     
==========================================
  Files          56       56              
  Lines       15694    16543     +849     
==========================================
+ Hits        12622    14324    +1702     
+ Misses       3072     2219     -853     
Files Changed Coverage Δ
src/core/packet_builder.c 91.82% <100.00%> (+10.98%) ⬆️
src/core/sent_packet_metadata.c 100.00% <100.00%> (ø)

... and 44 files with indirect coverage changes

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.

None yet

2 participants