-
-
Notifications
You must be signed in to change notification settings - Fork 9
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
Cake/TC need to be recompiled for AX86U #62
Comments
Hi @Adamm00 Looks like @odkrys is the member providing the builds and might be able to assist on this one. I would really like to see collab on this one. Here is the link to his repo: https://github.com/odkrys/entware-makefile-for-merlin |
I have some ideas on setting this up since I’ve been successfully building my own openwrt images, and Entware uses the openwrt buildroot. I am thinking this has to have 1) openwrt buildroot, 2) Entware repo, 3) asuswrt-merlin.ng repo, and 4) sched_cake_oot repo all available to build. Will test it out and let you know if I’m full of it (hint: probably). |
Yeah I tried briefly last week and gave up, I'm sure I was missing a step somewhere. In any case if you figure it out I can automate it for future updates/models. |
I’ve given up for now since I couldn’t build the Entware toolchain on WSL2 Ubuntu. Something wasn’t set right and couldn’t figure it out. Told you I was full of it. |
Thanks @dave14305 -- would really like to get @odkrys involved for his feedback/help with this. |
Does this version of cake support ack-filtering? |
Looks like he compiled new builds today. Will anyone test them? |
What is required to help out w/ testing? I looked at the repo, but not obvious (to me) what I would need to do from my end. Is the process documented? |
I've forked CakeQoS and made necessary changes, run the script and it works like a charm on my RT-AX58U. tc qdisc tc qdisc | grep root tc -s qdisc show dev eth4 tc -s qdisc show dev ifb9eth4 cake-qos status general CakeQOS-Merlin: > Upload Status: Bufferbloat Test Result: A+ A A+ http://www.dslreports.com/speedtest/67218653 (Connection 100DL/5UL, Settings 95DL/5UL) |
It doesn’t look like tc-adv is working for you. It shouldn’t be Unknown qdisc. |
That's the problem I see, no documentation anywhere, so left looking at the Makefile. You need a kernel to build against, and @odkrys uses Merlin's new builds. Second are these two includes:
sched-cake-oot: https://github.com/odkrys/entware-makefile-for-merlin/blob/main/sched-cake-oot/Makefile Seems @odkrys is building cake into the Merlin kernel natively...which is also pretty cool 💯 |
DSLReports' Speedtest scores went from B C B to A+ A A+, so there's definitely an improvement there 🤔 I am nowhere near a developer but is it possible that "Unknown qdisc" is a cosmetic error caused by an undefined string or something and the script just works? I've made several tests in the last 2 weeks on this unit with all the QoS options (Traditional, Adaptive, FlexQoS) turned on and off, and never got any A's on DSLReports, that's why I'm asking. |
Try running |
/opt/bin/tc -s qdisc
|
Can you document what you have done to get to the point of failure? I have a couple Ubuntu VMs hanging around that I wouldn't mind trying on.... |
@shemminger Sorry for the late reply. Yes it does support ack-filtering. Once you are ok with loosing a fair bit of your down cake on the router should be ok. My current config for example is:
|
@underd0se @tekneek and others here I've just uploaded some changes to the develop branch using the binaries from @odkrys Apologies on the massive delays. Day (and night) work sometimes gets in the way.
Please let me know your feedback, and we get it pushed to master. This mod is going to phase into an advanced addon for Merlin's firmware with it being rolled into the firmware, which is excellent as we don't have to worry about new builds. I'm very thankful for that as it will hopefully increase adoption of this awesome QoS alternative (with privacy). Testing in the new Alpha has been going good thus far. |
I've switched to your dev branch, installed cake-qos and:
Now I'm back on mine since me and wife are WFH and we need a stable connection. However, I can test it out on the weekend if you make changes. |
Ok thanks for the quick update. I'll check the diffs in your changes to see if anything was missed. |
Should be good now. I forgot to update the branch variable in the script. Apologies and thanks for testing again. |
Fixed and committed to version v1.0.8 |
Ideally with some instructions on how it was compiled in the first place so this task can be automated for future updates / models.
The text was updated successfully, but these errors were encountered: