From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <jbrouer@redhat.com>
Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28])
	by huchra.bufferbloat.net (Postfix) with ESMTP id CB11321F150
	for <cerowrt-devel@lists.bufferbloat.net>;
	Tue, 27 Aug 2013 03:45:33 -0700 (PDT)
Received: from int-mx01.intmail.prod.int.phx2.redhat.com
	(int-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.11])
	by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id r7RAjS9t001649
	(version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK);
	Tue, 27 Aug 2013 06:45:28 -0400
Received: from localhost (ovpn-116-39.ams2.redhat.com [10.36.116.39])
	by int-mx01.intmail.prod.int.phx2.redhat.com (8.13.8/8.13.8) with ESMTP
	id r7RAjQAL004231; Tue, 27 Aug 2013 06:45:26 -0400
Date: Tue, 27 Aug 2013 12:45:23 +0200
From: Jesper Dangaard Brouer <jbrouer@redhat.com>
To: Sebastian Moeller <moeller0@gmx.de>
Message-ID: <20130827124523.0c52391b@redhat.com>
In-Reply-To: <5BEF0C7C-C2F4-45A9-9FF2-E32A05B8D67B@gmx.de>
References: <CAA93jw7rankvo0B7+HzbASYY2gBcQfa2B-_dC2z6CHhqfWSkJQ@mail.gmail.com>
	<56B261F1-2277-457C-9A38-FAB89818288F@gmx.de>
	<CAA93jw6ku0OOXzNcAUtK4UL5uc7R2zVAOKo1+Fwzmr7gCH1pzA@mail.gmail.com>
	<2148E2EF-A119-4499-BAC1-7E647C53F077@gmx.de>
	<03951E31-8F11-4FB8-9558-29EAAE3DAE4D@gmx.de>
	<CAA93jw6PS1rQqxPTfroW9J1tOrp8gWE1G98mYkKCNTAvPoYrAg@mail.gmail.com>
	<9A9B094D-CA07-48B0-85FE-FA7C759FEDE3@gmx.de>
	<CAA93jw5-Dv7NTh-bU6BK=huWtpo6DKCBfEG7nFDjHEOnqi0wJw@mail.gmail.com>
	<5BEF0C7C-C2F4-45A9-9FF2-E32A05B8D67B@gmx.de>
Organization: Red Hat Inc.
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
X-Scanned-By: MIMEDefang 2.67 on 10.5.11.11
Cc: "cerowrt-devel@lists.bufferbloat.net" <cerowrt-devel@lists.bufferbloat.net>
Subject: Re: [Cerowrt-devel] some kernel updates
X-BeenThere: cerowrt-devel@lists.bufferbloat.net
X-Mailman-Version: 2.1.13
Precedence: list
List-Id: Development issues regarding the cerowrt test router project
	<cerowrt-devel.lists.bufferbloat.net>
List-Unsubscribe: <https://lists.bufferbloat.net/options/cerowrt-devel>,
	<mailto:cerowrt-devel-request@lists.bufferbloat.net?subject=unsubscribe>
List-Archive: <https://lists.bufferbloat.net/pipermail/cerowrt-devel>
List-Post: <mailto:cerowrt-devel@lists.bufferbloat.net>
List-Help: <mailto:cerowrt-devel-request@lists.bufferbloat.net?subject=help>
List-Subscribe: <https://lists.bufferbloat.net/listinfo/cerowrt-devel>,
	<mailto:cerowrt-devel-request@lists.bufferbloat.net?subject=subscribe>
X-List-Received-Date: Tue, 27 Aug 2013 10:45:34 -0000

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