docs: NodePort XDP on AWS #12156
Merged
docs: NodePort XDP on AWS #12156
Conversation
This file contains 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
borkmann
added
pending-review
area/documentation
|
Please set the appropriate release note label. |
3 similar comments
|
Please set the appropriate release note label. |
|
Please set the appropriate release note label. |
|
Please set the appropriate release note label. |
borkmann
added
the
release-note/misc
borkmann
force-pushed
the
pr/xdp-eks-doc
branch
4 times, most recently
from
ba27208
to
0bc8fdc
Compare
Write up a small getting started guide based on Tobias' notes on EKS (https://gist.github.com/tklauser/268641aea4fced9e8c3b4a2f7536661b). Signed-off-by: Tobias Klauser <tklauser@distanz.ch> Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
jrfastab
reviewed
Jun 17, 2020
| The default MTU is set to 9001 on the ena driver. Given XDP buffers are linear, they | ||
| operate on a single page. A driver typically reserves some headroom for XDP as well | ||
| (e.g. for encapsulation purpose), therefore, the highest possible MTU for XDP would | ||
| be 3818. |
There was a problem hiding this comment.
Limitation is a bit unfortunate. I wonder if we should add an option to configure mtu for 1.9?
There was a problem hiding this comment.
Yes, the agent could take care of it for 1.9.
jrfastab
reviewed
Jun 17, 2020
| Combined: 4 | ||
|
|
||
| In order to use XDP the channels must be set to at most 1/2 of the value from | ||
| ``Combined`` above. Both, MTU and channel changes are applied as follows: |
There was a problem hiding this comment.
iirc some drivers will automagically reconfigure to meet the rx/tx queue requirements perhaps we should file a feature request against ena to do this.
There was a problem hiding this comment.
Agree, either that or implement it directly upstream. E.g. on Azure no additional knobs are needed.
jrfastab
approved these changes
Jun 17, 2020
20 tasks
maintainer-s-little-helper
bot
moved this from Needs backport from master
to Backport pending to v1.8
in 1.8.0
maintainer-s-little-helper
bot
moved this from Backport pending to v1.8
to Backport done to v1.8
in 1.8.0
maintainer-s-little-helper
bot
moved this from Backport pending to v1.8
to Backport done to v1.8
in 1.8.0
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Write up a small getting started guide based on Tobias' notes on
EKS (https://gist.github.com/tklauser/268641aea4fced9e8c3b4a2f7536661b).
Signed-off-by: Tobias Klauser tklauser@distanz.ch
Signed-off-by: Daniel Borkmann daniel@iogearbox.net