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

["Toke Høiland-Jørgensen" <toke@toke.dk>]

Sebastian Moeller <moeller0@gmx.de> writes:

> 	Not that I have shown great taste in the past, but I think it
> would be somewhat cleaner to put the logic into the hot plug script
> and keep run.sh “simple” (in the past I had introduced a large number
> of leakage, especially of IFBs by not properly removing/stopping old
> instances and was quite happy to have the take all active interfaces
> down loop as a last defense against accidental leaks).

Well, the biggest issue I can see with not having any logic in run.sh is
that in that case, *all* interfaces will be reconfigured when the
hotplug event happens. However, I'm not sure exactly how common it is to
have more than one interface configured for SQM, and if so, whether or
not reconfiguring everything on every hotplug event (well, only for for
SQM-enabled interfaces I suppose) is an issue.

The modifications to run.sh should keep it functioning the way it does
currently if run 'manually' the shell or LUCI. Unless the $DEVICE
env-var is set for some other reason...


> 	But I am now also running pppoe directly from cerowrt and see
> 	the same issue, sqm is confused when the pppoe interface
> 	temporarily goes away, so at least I can now test this issue ;)

Well, a first pass could be to see if the modified run.sh I sent last
time around actually works... ;)

-Toke