Re: [Cerowrt-devel] some kernel updates

[Jesper Dangaard Brouer <jbrouer@redhat.com>]

On Fri, 23 Aug 2013 21:56:02 +0200
Sebastian Moeller <moeller0@gmx.de> wrote:

> Hi Dave,
> 
> I guess I found the culprit:
> 
> once I added $ADSLL to the ingress() in simple.qos:

Yes, exactly!

> ingress() {
[...]
> 
> $TC qdisc del dev $DEV root  2> /dev/null
> $TC qdisc add dev $DEV root handle 1: ${STABSTRING} htb default 12
> $TC class add dev $DEV parent 1: classid 1:1 htb $LQ rate ${CEIL}kbit ceil ${CEIL}kbit $ADSLL
> $TC class add dev $DEV parent 1:1 classid 1:10 htb $LQ rate ${CEIL}kbit ceil ${CEIL}kbit prio 0 $ADSLL
> $TC class add dev $DEV parent 1:1 classid 1:11 htb $LQ rate 32kbit ceil ${PRIO_RATE}kbit prio 1 $ADSLL
> $TC class add dev $DEV parent 1:1 classid 1:12 htb $LQ rate ${BE_RATE}kbit ceil ${BE_CEIL}kbit prio 2 $ADSLL
> $TC class add dev $DEV parent 1:1 classid 1:13 htb $LQ rate ${BK_RATE}kbit ceil ${BE_CEIL}kbit prio 3 $ADSLL
[...]
> 
>
> I get basically the same RRUL ping RTTs for htb_private as for tc_stab.
> So Jesper was right the patch seems to fix the issue. I guess I should
> send out my current version of yours and Toke's AQM scripts soon.

Great news, so my patch did work as expected! :-)

-- 
Best regards,
  Jesper Dangaard Brouer
  MSc.CS, Sr. Network Kernel Developer at Red Hat
  Author of http://www.iptv-analyzer.org
  LinkedIn: http://www.linkedin.com/in/brouer