Skip to content

bpf: nodeport: optimize nat_46x64 in tail_nodeport_nat_egress_ipv6()#44745

Merged
julianwiedmann merged 1 commit intomainfrom
pr/jwi/main/bpf-nat46x64-dce
Mar 12, 2026
Merged

bpf: nodeport: optimize nat_46x64 in tail_nodeport_nat_egress_ipv6()#44745
julianwiedmann merged 1 commit intomainfrom
pr/jwi/main/bpf-nat46x64-dce

Conversation

@julianwiedmann
Copy link
Copy Markdown
Member

When neither of the NAT_46X64 features is enabled, we can trust that NAT46x64_MODE_XLATE is not set. And thus have the compiler dead-code-eliminate all the related pieces.

When neither of the NAT_46X64 features is enabled, we can trust that
NAT46x64_MODE_XLATE is not set. And thus have the compiler
dead-code-eliminate all the related pieces.

Signed-off-by: Julian Wiedmann <jwi@isovalent.com>
@julianwiedmann julianwiedmann added area/datapath Impacts bpf/ or low-level forwarding details, including map management and monitor messages. release-note/misc This PR makes changes that have no direct user impact. labels Mar 12, 2026
@julianwiedmann
Copy link
Copy Markdown
Member Author

This should be sufficient to eliminate all code that depends on this bool flag.

@julianwiedmann
Copy link
Copy Markdown
Member Author

/test

@julianwiedmann julianwiedmann marked this pull request as ready for review March 12, 2026 10:41
@julianwiedmann julianwiedmann requested a review from a team as a code owner March 12, 2026 10:41
@julianwiedmann julianwiedmann added this pull request to the merge queue Mar 12, 2026
Merged via the queue into main with commit a0d5dd6 Mar 12, 2026
508 of 510 checks passed
@julianwiedmann julianwiedmann deleted the pr/jwi/main/bpf-nat46x64-dce branch March 12, 2026 11:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/datapath Impacts bpf/ or low-level forwarding details, including map management and monitor messages. release-note/misc This PR makes changes that have no direct user impact.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants