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
bpf: clean up some revalidate_data() users #25337
bpf: clean up some revalidate_data() users #25337
Conversation
/test |
f793f5b
to
aed38b1
Compare
/test |
aed38b1
to
1df740d
Compare
/test |
1df740d
to
8a6594b
Compare
/ci-verifier |
Commit 0664e3f4010b275282fa1ef16b45d7c7f59b20fd does not contain "Signed-off-by". Please follow instructions provided in https://docs.cilium.io/en/stable/contributing/development/contributing_guide/#developer-s-certificate-of-origin |
/ci-verifier |
0664e3f
to
213da4d
Compare
/ci-verifier |
213da4d
to
dd6a8f8
Compare
/ci-verifier |
dd6a8f8
to
58b9183
Compare
/ci-l4lb |
d897c5f
to
eb33923
Compare
Don't bother validating an IPv4 header when the packet is actually IPv6. Signed-off-by: Julian Wiedmann <jwi@isovalent.com>
Nothing uses the L3 header after this revalidation. Signed-off-by: Julian Wiedmann <jwi@isovalent.com>
When called in from-overlay or from-netdev, we already validated the IPv6 header and can skip the validation inside nodeport_lb6(). Signed-off-by: Julian Wiedmann <jwi@isovalent.com>
eb33923
to
0632883
Compare
/test |
/test-1.24-5.4 |
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.
Left some comments. Comments are minor, so not requesting changes
GH ginkgo fails are #25964 |
Some completely innocent cleanups to our
revalidate_data()
usage.