-
Notifications
You must be signed in to change notification settings - Fork 63
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
SQM does not always detect cake as available qdisc at boot #122
Comments
Hannu Nyman <[email protected]> writes:
I have lately noticed that SQM does not always properly recognize cake
at OpenWrt startup. The list of available qdiscs in LuCI contains then
only fq_codel, but no trace of cake. Manually running
`/usr/lib/sqm/update-available-qdiscs` brings cake to the qdisc list.
Hmm, that's odd... Do you get anything in the log? Could you try enable
debugging and see if anything shows up in the logs?
Try putting this:
SQM_VERBOSITY_MAX=10
SQM_DEBUG=1
in /etc/sqm.conf and look in the syslog or in the logfiles created in
/var/run/sqm
|
The system log shows nothing interesting by default:
I added verbosity to the config, so maybe something interesting surfaces on the next times. |
There wouldn't be, by default. I'm interested in the debug output from the qdisc checking script at boot, which is why the debug config should go into /etc/sqm.conf
|
I tried normal reboot, and both qdiscs got detected ok. I suspect that the trouble is somehow related to the first boot after flashing, as the discs & kept settings are initialised, so the process timings may be a bit different from a normal boot. |
Silly question,
is fq_codel by chance baked into the kernel while cake is a module?
Best Regards
Sebastian
… On May 26, 2020, at 20:56, Hannu Nyman ***@***.***> wrote:
I tried normal reboot, and both qdiscs got detected ok.
I suspect that the trouble is somehow related to the first boot after flashing, as the discs & kept settings are initialised, so the process timings may be a bit different from a normal boot.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
Ah, right. Are you baking sqm-scripts and kmod-sched-cake into the image?
|
Sure. They are included in the image. I include luci-app-sqm and it pulls the dependencies. I tried now re-flashing the router, but this time the detection worked ok at boot. The failure does not happen always. (But there is no trace of cake detection in SQM start log, although the marker file is now present in available_qdiscs folder.)
Hmmm. Ps. I added the verbosity to the uci config file, not /etc/sqm.conf, but that should make no difference, I think. |
Interesting enough I do not have them compiled into the image —both are modules— and I am experiencing same behaviour. Difference is that I am on 19.07.3 with a few cherry-picked patches from master. |
Hmm, since the available qdiscs list is only used for populating the GUI, I wonder if we could just run the detection the first time we need it?
|
Sounds reasonable. |
I can confirm I have this bug On my wrt3200acm after a fresh flash. |
possibly related... edit: false alarm... issue was on my end... @ any script error or hang at .qos does not cleanup /tmp/run/sqm/sqm-run.lock/ |
I have lately noticed that SQM does not always properly recognize cake at OpenWrt startup. The list of available qdiscs in LuCI contains then only fq_codel, but no trace of cake. Manually running
/usr/lib/sqm/update-available-qdiscs
brings cake to the qdisc list.Note the time difference below. fq_codel detected at boot, cake later on manual run:
I have noticed this several times after the reboot after the flashing of new OpenWrt master firmware into my ipq806x/R7800. (I am usually using simple/fq_codel, but I have lately used also cake. So it is hard to say if this is really new, or already something old.)
I suspect that there is some race condition or similar in the SQM startup process, so that it misses detecting cake.
I haven't tried to debug it further, but thought to mention it case somebody else runs into that. Could be that a small delay in the detection script would help, or something similar.
The text was updated successfully, but these errors were encountered: