-
Notifications
You must be signed in to change notification settings - Fork 673
[2/3] Integrate XSK maps into the core XDP datapath #6003
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
Open
ProjectsByJackHe
wants to merge
43
commits into
main
Choose a base branch
from
jackhe/xdp_map_dp_int_pr_2_core
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+1,710
−49
Open
Changes from 17 commits
Commits
Show all changes
43 commits
Select commit
Hold shift + click to select a range
0020948
add tools
ProjectsByJackHe 9830250
phase 2 looks good so far
ProjectsByJackHe 7ba0bf1
phase 2 improvements
ProjectsByJackHe b364f47
better diagnostics
ProjectsByJackHe 01df062
phase 3 plumbing...
ProjectsByJackHe 3097b18
phase 3: iter 2
ProjectsByJackHe 5750131
bug fixes man
ProjectsByJackHe bbf64e0
E2E
ProjectsByJackHe c62d7b1
fix comment
ProjectsByJackHe 08ff04e
update xdp commit
ProjectsByJackHe b8285dc
get rid of tools
ProjectsByJackHe 4546dfe
get rid of preview features
ProjectsByJackHe ba539a7
add unit tests
ProjectsByJackHe d15f3a7
CLOG
ProjectsByJackHe 4fe8094
fix abi incompat
ProjectsByJackHe 47f162d
address code feedback
ProjectsByJackHe 0b83796
Merge branch 'main' into jackhe/xdp_map_dp_int_pr_2_core
ProjectsByJackHe f68a1d8
fix dp tests + update logging
ProjectsByJackHe d8fe2d2
infer xdp enablement if map mode is set
ProjectsByJackHe 96ded75
set xdpenabled to false explicitly in map mode returns error
ProjectsByJackHe 240f0d2
fix unit test, address comments
ProjectsByJackHe 1559660
generate clog, fix code pattern
ProjectsByJackHe 892304c
update comment
ProjectsByJackHe 1388f07
put map mode flag in rawdatapath
ProjectsByJackHe c472280
rename
ProjectsByJackHe acc113b
simplify the code
ProjectsByJackHe 2ae68ba
fix build
ProjectsByJackHe db73b3c
refactor map into RAW dp
ProjectsByJackHe 4eb0d70
CLOG
ProjectsByJackHe c9e7b1f
remove bogus test
ProjectsByJackHe c1fa28c
stash changes
ProjectsByJackHe 3b85f5e
improve tests
ProjectsByJackHe 815ff66
add cibir/qtip matrix
ProjectsByJackHe 39ce0e6
add CI to target map mode explicitly
ProjectsByJackHe 3d9d478
comment out map job
ProjectsByJackHe 9716cef
Merge branch 'main' into jackhe/xdp_map_dp_int_pr_2_core
ProjectsByJackHe d524666
move stuff out of the test harness
ProjectsByJackHe cb6db81
test harness refactors
ProjectsByJackHe 4c30ab2
more feedback refactors
ProjectsByJackHe d69d27d
add coverage for wildcard clients
ProjectsByJackHe 3e997ff
rename to not ref map mode
ProjectsByJackHe f658a8b
use -1
ProjectsByJackHe 3432322
add proper preview guards
ProjectsByJackHe File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -476,6 +476,18 @@ typedef struct CXPLAT_DATAPATH_INIT_CONFIG { | |||||
| // the Windows fast path causing a large performance regression. | ||||||
| // | ||||||
| BOOLEAN EnableDscpOnRecv; | ||||||
|
|
||||||
| // | ||||||
| // External XDP map configurations. When XdpMapConfigCount > 0 and | ||||||
| // XdpMapConfigs is non-NULL, the datapath operates in XDP map mode: the | ||||||
| // WinSock (normal) datapath is skipped, the raw (XDP) datapath is required | ||||||
| // to succeed, and XSK sockets are inserted into the provided XSKMAPs at | ||||||
| // init time. | ||||||
| // XdpMapConfigs is NULL if and only if XdpMapConfigCount == 0. | ||||||
| // The map configs must remain valid for the lifetime of the datapath. | ||||||
| // | ||||||
| const struct QUIC_XDP_MAP_CONFIG* XdpMapConfigs; | ||||||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. style: We don't repeat the "struct" keyword on use:
Suggested change
|
||||||
| uint32_t XdpMapConfigCount; | ||||||
| } CXPLAT_DATAPATH_INIT_CONFIG; | ||||||
|
|
||||||
| // | ||||||
|
|
||||||
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
nit: consider SAL to annotate the length.