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.

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

Reduce memory accesses when using sets #13

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

brada4
Copy link

@brada4 brada4 commented Nov 15, 2024

Use more efficient (smaller) nftables conntrack id in place of iptables-translate 4-tupple

Consequentially avoid extracting large piles of data from packet payload

Unify ipv4 and ipv6 meters as now they are same type and certainly unique to eachother

And enable ack rate-limiting for ipv6 using same meter at no extra processing cost.

Roughly equals to hudra0/qosmate#7 , most impactful part of qosmate 0.5.32-> 0.5.34

Signed-off-by: Andris PE [email protected]

Andris PE added 2 commits November 15, 2024 20:50
Use more efficient (smaller) nftables conntrack id in place of
iptables-translate 4-tupple

Consequentially avoid extracting large piles of data from packet payload

Unify ipv4 and ipv6 meters as now they are same type and certainly
unique to eachother

And enable ack rate-limiting for ipv6 using same meter at no extra processing
cost.
Precisely reflecting previous limit behaviour and not accidentally
halving intended rate.

Signed-off-by:
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant