[aqm] [Technical Errata Reported] RFC8033 (5095)

RFC Errata System <rfc-editor@rfc-editor.org> Thu, 24 August 2017 17:14 UTC

Return-Path: <wwwrun@rfc-editor.org>
X-Original-To: aqm@ietfa.amsl.com
Delivered-To: aqm@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 0E8D713202D for <aqm@ietfa.amsl.com>; Thu, 24 Aug 2017 10:14:23 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.201
X-Spam-Level:
X-Spam-Status: No, score=-4.201 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_MED=-2.3, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
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 L5vKqBXNc0Wu for <aqm@ietfa.amsl.com>; Thu, 24 Aug 2017 10:14:21 -0700 (PDT)
Received: from rfc-editor.org (rfc-editor.org [4.31.198.49]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id A1C8D126BF0 for <aqm@ietf.org>; Thu, 24 Aug 2017 10:14:21 -0700 (PDT)
Received: by rfc-editor.org (Postfix, from userid 30) id 45501B814EF; Thu, 24 Aug 2017 10:13:51 -0700 (PDT)
To: ropan@cisco.com, prenatar@cisco.com, fredbaker.ietf@gmail.com, g.white@cablelabs.com, spencerdawkins.ietf@gmail.com, ietf@kuehlewind.net, rs.ietf@gmx.at, wes@mti-systems.com
X-PHP-Originating-Script: 30:errata_mail_lib.php
From: RFC Errata System <rfc-editor@rfc-editor.org>
Cc: liang_tian@affirmednetworks.com, aqm@ietf.org, rfc-editor@rfc-editor.org
Content-Type: text/plain; charset=UTF-8
Message-Id: <20170824171351.45501B814EF@rfc-editor.org>
Date: Thu, 24 Aug 2017 10:13:51 -0700 (PDT)
Archived-At: <https://mailarchive.ietf.org/arch/msg/aqm/UmDWJltUEyqbE4O0Rh76KzyG0s8>
X-Mailman-Approved-At: Thu, 24 Aug 2017 11:09:00 -0700
Subject: [aqm] [Technical Errata Reported] RFC8033 (5095)
X-BeenThere: aqm@ietf.org
X-Mailman-Version: 2.1.22
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: <https://mailarchive.ietf.org/arch/browse/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: Thu, 24 Aug 2017 17:14:23 -0000

The following errata report has been submitted for RFC8033,
"Proportional Integral Controller Enhanced (PIE): A Lightweight Control Scheme to Address the Bufferbloat Problem".

--------------------------------------
You may review the report below and at:
http://www.rfc-editor.org/errata/eid5095

--------------------------------------
Type: Technical
Reported by: Liang Tian <liang_tian@affirmednetworks.com>

Section: 5.2

Original Text
-------------
      if PIE->in_measurement_ == TRUE:
         PIE->dq_count_ = PIE->dq_count_ + deque_pkt_size;
         if PIE->dq_count_ >= DQ_THRESHOLD then
            weight = DQ_THRESHOLD/2^16
            PIE->avg_dq_time_ = (now - PIE->measurement_start_) *
                                weight + PIE->avg_dq_time_ *
                                (1 - weight);
            PIE->dq_count_ = 0;
            PIE->measurement_start_ = now
         else
            PIE->in_measurement_ = FALSE;

Corrected Text
--------------
      if PIE->in_measurement_ == TRUE:
         PIE->dq_count_ = PIE->dq_count_ + deque_pkt_size;
         if PIE->dq_count_ >= DQ_THRESHOLD then
            weight = DQ_THRESHOLD/2^16
            PIE->avg_dq_time_ = (now - PIE->measurement_start_) *
                                weight + PIE->avg_dq_time_ *
                                (1 - weight);
            PIE->in_measurement_ = FALSE;

Notes
-----
There should not be an "else" because if PIE->dq_count_ >= DQ_THRESHOLD, this measurement is over: avg_dq_time is calculated and in_measurement is set to FALSE; otherwise dq_count has been increased before this "if" and now we wait for next packet. Resetting dq_count and measurement_start is not necessary because they will be set again when a new measurement begins.

Instructions:
-------------
This erratum is currently posted as "Reported". If necessary, please
use "Reply All" to discuss whether it should be verified or
rejected. When a decision is reached, the verifying party  
can log in to change the status and edit the report, if necessary. 

--------------------------------------
RFC8033 (draft-ietf-aqm-pie-10)
--------------------------------------
Title               : Proportional Integral Controller Enhanced (PIE): A Lightweight Control Scheme to Address the Bufferbloat Problem
Publication Date    : February 2017
Author(s)           : R. Pan, P. Natarajan, F. Baker, G. White
Category            : EXPERIMENTAL
Source              : Active Queue Management and Packet Scheduling
Area                : Transport
Stream              : IETF
Verifying Party     : IESG