Skip to content

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

Merged
julianwiedmann merged 1 commit into
mainfrom
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 into
mainfrom
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 <[email protected]>
@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