Re: [Cerowrt-devel] [sqm-scripts] not started at boot?

[Sebastian Moeller <moeller0@gmx.de>]

[-- Attachment #1: Type: text/plain, Size: 465 bytes --]

Hi Alan,

I would be delighted if you could try the etc/hotplug.d/iface/11-sqm and the updated usr/lib/sqm/run.sh from the attached files (I am not sure about the root to the files in the attached folder, but you will figure it out, I am sure ;) ). This should fix the reconnect when pop interface transiently go away issue. I am not sure whether this fixes the startup after reboot issue, so please feel free to test this as well.

Best Regards
	Sebastian


[-- Attachment #2: sqm-scripts.zip --]
[-- Type: application/zip, Size: 27418 bytes --]

[-- Attachment #3: Type: text/plain, Size: 1382 bytes --]


On Feb 15, 2015, at 17:33 , Alan Jenkins <alan.christopher.jenkins@gmail.com> wrote:

> 
> On 15/02/15 15:56, Toke Høiland-Jørgensen wrote:
>> Sebastian Moeller <moeller0@gmx.de> writes:
>> 
>>> 	I am not sure that this works as intended. The first thing
>>> 	run.sh does is take down all running SQM instances:
>> Ah yes, seems I was a bit too trigger-happy there ;)
> I was just about to explain the same, after testing it :).  (I have more interfaces coming up after pppoe-wan, so SQM stops immediately after it's started)
> 
>> Here's a version of run.sh that should also short-circuit the 'down'
>> part if called from hotplug.
> Yep. that version works ok.
> 
>> The alternative is, of course, to have logic in the hotplug script to
>> only call run.sh for interfaces that are enabled, but that would require
>> parsing /etc/config/sqm from there. My initial thought was that
>> short-circuiting the logic in run.sh was 'cleaner'; but I'm not entirely
>> sure about that... Thoughts?
>> 
>> -Toke
> 
> It's a small tweak, but maybe the argument to run.sh could be made explicit.
> 
> run.sh start $DEVICE # specific device
> run.sh # all devices
> 
> Regards
> Alan
> _______________________________________________
> Cerowrt-devel mailing list
> Cerowrt-devel@lists.bufferbloat.net
> https://lists.bufferbloat.net/listinfo/cerowrt-devel