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

qos: use -m state --state #9

Open
wants to merge 1 commit into
base: 374.43_2-update
Choose a base branch
from

Conversation

kjbracey2
Copy link

Avoid using -m conntrack --ctstate, as it's not implemented for IPv6 in kernel 2.6.22 or iptables 1.3.8. But -m state --state does exactly the same thing, and is implemented.

Avoid using -m conntrack --ctstate, as it's not implemented for IPv6 in
kernel 2.6.22 or iptables 1.3.8. But -m state --state does exactly the
same thing, and is implemented.
@kjbracey2
Copy link
Author

This is the simple solution for making IPv6 QoS work on MIPS, rather than #7. Tested applied to base code and seems to work.

I'm still interested in #7 as a potential path to QoS improvements and aligning with upstream, but this is far less risky.

@kjbracey2
Copy link
Author

I note that this was introduced in 92e88b8, and isn't upstream. Do you recall what the bug that was being worked around was?

@john9527
Copy link
Owner

Do you recall what the bug that was being worked around was?

At that time there was a lot of work going on across multiple distirbutions. I think it may have come as a port from Tomato.

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.

2 participants