[aqm] review of CoDel -00 draft
Wesley Eddy <wes@mti-systems.com> Tue, 17 March 2015 15:45 UTC
Return-Path: <wes@mti-systems.com>
X-Original-To: aqm@ietfa.amsl.com
Delivered-To: aqm@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 4B7C21A701E for <aqm@ietfa.amsl.com>; Tue, 17 Mar 2015 08:45:56 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.9
X-Spam-Level:
X-Spam-Status: No, score=-1.9 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_NONE=-0.0001] autolearn=ham
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id XTBB_iGNAVcV for <aqm@ietfa.amsl.com>; Tue, 17 Mar 2015 08:45:54 -0700 (PDT)
Received: from atl4mhob04.myregisteredsite.com (atl4mhob04.myregisteredsite.com [209.17.115.42]) by ietfa.amsl.com (Postfix) with ESMTP id C95AD1A701A for <aqm@ietf.org>; Tue, 17 Mar 2015 08:45:54 -0700 (PDT)
Received: from mailpod.hostingplatform.com ([10.30.71.207]) by atl4mhob04.myregisteredsite.com (8.14.4/8.14.4) with ESMTP id t2HFjqOj032500 for <aqm@ietf.org>; Tue, 17 Mar 2015 11:45:52 -0400
Received: (qmail 25391 invoked by uid 0); 17 Mar 2015 15:45:52 -0000
X-TCPREMOTEIP: 207.54.183.210
X-Authenticated-UID: wes@mti-systems.com
Received: from unknown (HELO ?192.168.1.112?) (wes@mti-systems.com@207.54.183.210) by 0 with ESMTPA; 17 Mar 2015 15:45:52 -0000
Message-ID: <55084C23.4040108@mti-systems.com>
Date: Tue, 17 Mar 2015 11:45:39 -0400
From: Wesley Eddy <wes@mti-systems.com>
Organization: MTI Systems
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:31.0) Gecko/20100101 Thunderbird/31.5.0
MIME-Version: 1.0
To: "aqm@ietf.org" <aqm@ietf.org>
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
Archived-At: <http://mailarchive.ietf.org/arch/msg/aqm/dltjFMi29b6a-cLENTu-eIkNBh4>
Subject: [aqm] review of CoDel -00 draft
X-BeenThere: aqm@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: "Discussion list for active queue management and flow isolation." <aqm.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/aqm>, <mailto:aqm-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/aqm/>
List-Post: <mailto:aqm@ietf.org>
List-Help: <mailto:aqm-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/aqm>, <mailto:aqm-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 17 Mar 2015 15:45:56 -0000
I reviewed and have some comments on the CoDel draft: http://tools.ietf.org/html/draft-ietf-aqm-codel-00 1) I believe it would be a good idea to tie the goals listed in section 1 (in the bullet list on page 4) to the AQM guidelines from the RFC-to-be of draft-ietf-aqm-recommendation. Largely, the CoDel goals were brought into that rather than being pulled from it, but it will be good to provide a sentence or so that ties the working group documents and material together. 2) In the discussion of sojourn time as a metric (section 3.1) and using the minimum time to separate good queue from bad queue, it struck me that there is some parallel between this and the way that LEDBAT works in using the delta from the minimum as indication of queues (and congestion) building. It may be worth noting or expanding on delay-based CC in endpoints and in-network. 3) Since the algorithm and pseudocode has been published a few places before, it would be good to draw attention to a section that notes any changes or differences between what will be published in this RFC and any other revisions of the pseudocode floating around the net (or clearly note if there aren't any). In generally, it's very readable and I think it would serve as a clear basis for implementing the algorithm, so would be happy to see it go forward quickly through this working group to become an RFC. Small / editorial comments: - The abstract has [TSVBB2011] which doesn't seem to actually exist as a reference - The abstract could really be trimmed to just the 2nd paragraph, as the first is just background that's in the Introduction anyways - The "(covered in another draft)" at the end of section 1 can be replaced with a real reference (there's already one later in the draft in 4.6) -- Wes Eddy MTI Systems
- [aqm] review of CoDel -00 draft Wesley Eddy