Re: [Gen-art] Gen-ART review of draft-ietf-mmusic-ice-tcp-15
"Vijay K. Gurbani" <vkg@bell-labs.com> Tue, 01 November 2011 14:26 UTC
Return-Path: <vkg@bell-labs.com>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 902D121F9850 for <gen-art@ietfa.amsl.com>; Tue, 1 Nov 2011 07:26:14 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -106.419
X-Spam-Level:
X-Spam-Status: No, score=-106.419 tagged_above=-999 required=5 tests=[AWL=-0.120, BAYES_00=-2.599, MIME_8BIT_HEADER=0.3, RCVD_IN_DNSWL_MED=-4, USER_IN_WHITELIST=-100]
Received: from mail.ietf.org ([12.22.58.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id s-80+kbWr8ci for <gen-art@ietfa.amsl.com>; Tue, 1 Nov 2011 07:26:13 -0700 (PDT)
Received: from ihemail4.lucent.com (ihemail4.lucent.com [135.245.0.39]) by ietfa.amsl.com (Postfix) with ESMTP id B09C421F9852 for <gen-art@ietf.org>; Tue, 1 Nov 2011 07:26:13 -0700 (PDT)
Received: from usnavsmail2.ndc.alcatel-lucent.com (usnavsmail2.ndc.alcatel-lucent.com [135.3.39.10]) by ihemail4.lucent.com (8.13.8/IER-o) with ESMTP id pA1EPvj8010894 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK); Tue, 1 Nov 2011 09:25:57 -0500 (CDT)
Received: from umail.lucent.com (umail-ce2.ndc.lucent.com [135.3.40.63]) by usnavsmail2.ndc.alcatel-lucent.com (8.14.3/8.14.3/GMO) with ESMTP id pA1EPuFW026987 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NOT); Tue, 1 Nov 2011 09:25:57 -0500
Received: from shoonya.ih.lucent.com (shoonya.ih.lucent.com [135.185.238.235]) by umail.lucent.com (8.13.8/TPES) with ESMTP id pA1EPuVc029690; Tue, 1 Nov 2011 09:25:56 -0500 (CDT)
Message-ID: <4EB00227.8060307@bell-labs.com>
Date: Tue, 01 Nov 2011 09:28:55 -0500
From: "Vijay K. Gurbani" <vkg@bell-labs.com>
Organization: Bell Laboratories, Alcatel-Lucent
User-Agent: Mozilla/5.0 (X11; Linux i686; rv:7.0) Gecko/20110927 Thunderbird/7.0
MIME-Version: 1.0
To: Ari Keränen <ari.keranen@ericsson.com>
References: <4EAF17BC.6020409@bell-labs.com> <4EAF2AF5.9010003@ericsson.com>
In-Reply-To: <4EAF2AF5.9010003@ericsson.com>
Content-Type: text/plain; charset="ISO-8859-1"; format="flowed"
Content-Transfer-Encoding: 8bit
X-Scanned-By: MIMEDefang 2.57 on 135.245.2.39
X-Scanned-By: MIMEDefang 2.64 on 135.3.39.10
Cc: "fandreas@cisco.com" <fandreas@cisco.com>, General Area Review Team <gen-art@ietf.org>, Gonzalo Camarillo <gonzalo.camarillo@ericsson.com>, "draft-ietf-mmusic-ice-tcp@tools.ietf.org" <draft-ietf-mmusic-ice-tcp@tools.ietf.org>
Subject: Re: [Gen-art] Gen-ART review of draft-ietf-mmusic-ice-tcp-15
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/gen-art>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 01 Nov 2011 14:26:14 -0000
On 10/31/2011 06:10 PM, Ari Keränen wrote: >> - S1: The draft says, "This specification does so by following the >> outline of ICE itself, and calling out the additions and changes >> necessary in each section of ICE to support TCP candidates." >> >> Does this imply that this specification normatively updates >> rfc5425 (the ICE RFC)? If so, this is not reflected in the >> masthead for the document as "Updates: RFC 5425". > > Actually no. These additions are specific to TCP so the (UDP) ICE RFC is > not updated. In an effort to keep this draft reasonably compact, only > the delta from the RFC5245 is defined. So, when something is not defined > here, the behavior defined in RFC5245 is assumed. Ari: Thanks for the clarification. In that case, I would think that inserting some text to the document fashioned along what you write above will help those folks who would read your document and assume normative changes to rfc5425. A small change such as this will suffice (please feel free to edit): OLD: This specification does so by following the outline of ICE itself, and calling out the additions and changes necessary in each section of ICE to support TCP candidates. NEW: This specification does so by following the outline of ICE itself, and calling out the additions and changes necessary to support TCP candidates in ICE. The base behaviour of ICE [RFC5245] remains unchanged except for the extensions in this document that define the usage of ICE with TCP candidates. One more comments; please see below. >> - S4.1, last paragraph. It says that, "TCP-based STUN transactions >> are paced out at one every Ta seconds." However, rfc5245 Section >> 16 says that, "These transactions are paced at a rate of one >> every Ta millisecond, ..." >> >> I suspect that in your draft, Ta refers to the same Ta as in >> rfc5245, so it seems to me that they should be paced one every >> Ta milliseconds to conform to rfc5245, no? > > This one is a bit tricky since even RFC5245 is not consistent with this > (e.g., section 5.8. of RFC5245 says "Ta seconds later", and section > 16.1. is even more vague). We chose to use "seconds" so that the > formulas in 16.1. of 5245 would have right units, but actually in the > text 5245 does use "milliseconds" more often. I'm fine either way but > since you found this confusing, probably the milliseconds is a better > choice. I believe that ms is what is intended (but the author of rfc5245 can always be asked for a clarification). Here's my reasoning: The formulas of Section 16.1 are stated in terms of ms; i.e., Ta = MAX(20ms, 1/sum_{i=1}^k (1/Ta_i)). To get an authoritative answer from the MAX() function would require both parameters to be the same units. That is, MAX(20, 3) is rather ambiguous if the first parameter is in terms of ms and the second in terms of second. On the other hand, MAX(20, 3000) is straightforward to evaluate since both parameters are in terms of ms. Thanks, - vijay -- Vijay K. Gurbani, Bell Laboratories, Alcatel-Lucent 1960 Lucent Lane, Rm. 9C-533, Naperville, Illinois 60566 (USA) Email: vkg@{bell-labs.com,acm.org} / vijay.gurbani@alcatel-lucent.com Web: http://ect.bell-labs.com/who/vkg/
- [Gen-art] Gen-ART review of draft-ietf-mmusic-ice… Vijay K. Gurbani
- Re: [Gen-art] Gen-ART review of draft-ietf-mmusic… Ari Keränen
- Re: [Gen-art] Gen-ART review of draft-ietf-mmusic… Vijay K. Gurbani
- Re: [Gen-art] Gen-ART review of draft-ietf-mmusic… Ari Keränen