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.

Sign up for GitHub

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

Jump to bottom

bpf: use LB addr as srcIP for outer hdr in DSR/IPIP #14260

Merged
merged 1 commit into from
Dec 4, 2020
Merged

bpf: use LB addr as srcIP for outer hdr in DSR/IPIP #14260

merged 1 commit into from
Dec 4, 2020

Conversation

borkmann
Copy link
Member

@borkmann borkmann commented Dec 3, 2020 •
edited

see commit msg, will add rss-friendly option as an extension.

@borkmann
bpf: use LB addr as srcIP for outer hdr in DSR/IPIP
69be319 
This is currently less optimial in terms of RSS, but it will have known
addresses for the network wrt policy etc. We're going to extend this for
other modes where we have a special srcIP prefix to better interoperate
with RSS on backend nodes.

Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
@borkmann borkmann added release-note/misc This PR makes changes that have no direct user impact. feature/lb-only Impacts cilium running in lb-only datapath mode labels Dec 3, 2020
@borkmann borkmann requested a review from brb December 3, 2020 16:24
@maintainer-s-little-helper maintainer-s-little-helper bot added this to In progress in 1.10.0 Dec 3, 2020
@borkmann borkmann marked this pull request as ready for review December 3, 2020 21:58
@borkmann borkmann requested a review from a team December 3, 2020 21:58
@borkmann
Copy link
Member Author

borkmann commented Dec 3, 2020

test-me-please

@borkmann
Copy link
Member Author

borkmann commented Dec 3, 2020

retest-4.9

@borkmann
Copy link
Member Author

borkmann commented Dec 3, 2020

retest-runtime

@borkmann borkmann merged commit 64e342c into master Dec 4, 2020
@borkmann borkmann deleted the pr/ipip-lb-ip branch December 4, 2020 00:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@christarazi christarazi christarazi approved these changes

@brb brb Awaiting requested review from brb

Assignees

@brb brb

@christarazi christarazi

Labels
feature/lb-only Impacts cilium running in lb-only datapath mode release-note/misc This PR makes changes that have no direct user impact.
Projects
No open projects
1.10.0
Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@borkmann @christarazi @brb