Re: [Dart] I-D Action: draft-ietf-dart-dscp-rtp-07.txt

"Black, David" <david.black@emc.com> Wed, 01 October 2014 22:54 UTC

Return-Path: <david.black@emc.com>
X-Original-To: dart@ietfa.amsl.com
Delivered-To: dart@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 18D8C1A87D0; Wed, 1 Oct 2014 15:54:14 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -5.087
X-Spam-Level:
X-Spam-Status: No, score=-5.087 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, RCVD_IN_DNSWL_MED=-2.3, RP_MATCHES_RCVD=-0.786, SPF_PASS=-0.001] 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 vUv0c5k4r_3d; Wed, 1 Oct 2014 15:54:11 -0700 (PDT)
Received: from mailuogwhop.emc.com (mailuogwhop.emc.com [168.159.213.141]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 079A01A87D5; Wed, 1 Oct 2014 15:54:10 -0700 (PDT)
Received: from maildlpprd06.lss.emc.com (maildlpprd06.lss.emc.com [10.253.24.38]) by mailuogwprd02.lss.emc.com (Sentrion-MTA-4.3.0/Sentrion-MTA-4.3.0) with ESMTP id s91Ms9Y9025965 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Wed, 1 Oct 2014 18:54:09 -0400
X-DKIM: OpenDKIM Filter v2.4.3 mailuogwprd02.lss.emc.com s91Ms9Y9025965
DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d=emc.com; s=jan2013; t=1412204049; bh=LhpUtWSAmbPpKHdeCeFX8DxPfH4=; h=From:To:CC:Subject:Date:Message-ID:Content-Type: Content-Transfer-Encoding:MIME-Version; b=TzRNcBX0sPPGAjEeNcrJqWQDAz/qcGLCNA4fUpl3uU8SnEQu5qmu6z4aeE6k+uLjt 7n9/MFK1UJ/WTHYDvkKCmC1/kmE+PSNrVw8Vp+lwJDzPARhcVKP4hStKhd1jH4A53W iJnD3g3PUsUzNgD1ttKSCrsLMraN1S6+cA54JiVw=
X-DKIM: OpenDKIM Filter v2.4.3 mailuogwprd02.lss.emc.com s91Ms9Y9025965
Received: from mailusrhubprd54.lss.emc.com (mailusrhubprd54.lss.emc.com [10.106.48.19]) by maildlpprd06.lss.emc.com (RSA Interceptor); Wed, 1 Oct 2014 18:53:38 -0400
Received: from mxhub09.corp.emc.com (mxhub09.corp.emc.com [10.254.92.104]) by mailusrhubprd54.lss.emc.com (Sentrion-MTA-4.3.0/Sentrion-MTA-4.3.0) with ESMTP id s91MrwCK023069 (version=TLSv1/SSLv3 cipher=AES128-SHA bits=128 verify=FAIL); Wed, 1 Oct 2014 18:53:59 -0400
Received: from MXHUB105.corp.emc.com (10.253.50.22) by mxhub09.corp.emc.com (10.254.92.104) with Microsoft SMTP Server (TLS) id 8.3.327.1; Wed, 1 Oct 2014 18:53:58 -0400
Received: from MX104CL02.corp.emc.com ([169.254.8.131]) by MXHUB105.corp.emc.com ([10.253.50.22]) with mapi id 14.03.0195.001; Wed, 1 Oct 2014 18:53:58 -0400
From: "Black, David" <david.black@emc.com>
To: "Ruediger.Geib@telekom.de" <Ruediger.Geib@telekom.de>
Thread-Topic: [Dart] I-D Action: draft-ietf-dart-dscp-rtp-07.txt
Thread-Index: Ac/dypUYBYDu3jbzQ1+xP9yfrKj/gA==
Date: Wed, 1 Oct 2014 22:53:57 +0000
Message-ID: <CE03DB3D7B45C245BCA0D24327794936046458@MX104CL02.corp.emc.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [10.238.45.98]
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-Sentrion-Hostname: mailusrhubprd54.lss.emc.com
X-RSA-Classifications: public
Archived-At: http://mailarchive.ietf.org/arch/msg/dart/AAa7c_zM0aNrsC5xo3EQy1a1CkY
Cc: "Black, David" <david.black@emc.com>, "dart@ietf.org" <dart@ietf.org>, "ietf@ietf.org" <ietf@ietf.org>
Subject: Re: [Dart] I-D Action: draft-ietf-dart-dscp-rtp-07.txt
X-BeenThere: dart@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: "\"DiffServ Applied to RTP Transports discussion list\"" <dart.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/dart>, <mailto:dart-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/dart/>
List-Post: <mailto:dart@ietf.org>
List-Help: <mailto:dart-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/dart>, <mailto:dart-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 01 Oct 2014 22:54:14 -0000

Hi Ruediger,

> Sorry for being late with my review.

No apology necessary - you have the honor of submitting the first IETF Last Call
comments on this draft.

> There's one change I'd like to suggest:
> 
> Section 3.1, list number 2. briefly explains AF. The section doesn't
> explicitly mention that traffic carried by one of the three PHBs of an AF
> class are not reordered. This feature, avoiding reordering packets belonging
> to different PHBs of an AF class, is however a kind of punch line whenever AF
> is referred to in later sections. So I'd welcome mentioning it in section 3.1,
> list point 2.

Sure, that's a good suggestion - I'll do that.  That absence of reordering
concept is covered elsewhere in the draft, but it first shows up in Section 5.1,
so mentioning it earlier in Section 3.1 will help.

On the minor comments, I'd just as soon not tinker with the example text in
Section 4, our AD suggested emphasizing that point in Section 5.1 (but if
other reviewers dislike "very", that word can be bit-bucketed), and the ECMP
implementation described in your comments is one that cannot claim to support
AF, as it does not meet the requirements of RFC 2597.  It's a network operator
responsibility to not run AF across such an implementation; I think that sort
of DiffServ operational concern is best covered elsewhere.

My current inclination is not to make any changes for these minor comments.

> And one editorial comment:
> 
> Section 3., last bullet point and following section: the bullet point is
> focused on Lower Effort PHB marked by CS 1 and the following section continues
> by discussing CS 1 issues more general. This discussion may be added to the
> bullet point (to me the discussion doesn't seem to be related to the first
> bullet point).

That's a good catch - the paragraph after that CS1 bullet point mixes two
concepts that ought to be teased apart and merged into the two foregoing
bullets to improve clarity:

- Expanding on the non-requirements for different class selector code points
	to provide service differentiation.  This text should be merged into
	the first bullet.

- Expanding on the problems with assuming that CS1 selects a Lower Effort
	service.  This text should be merged into the second (CS1) bullet.

Thanks,
--David

> -----Original Message-----
> From: Ruediger.Geib@telekom.de [mailto:Ruediger.Geib@telekom.de]
> Sent: Wednesday, October 01, 2014 3:14 AM
> To: Black, David
> Cc: dart@ietf.org; ietf@ietf.org
> Subject: Re: [Dart] I-D Action: draft-ietf-dart-dscp-rtp-07.txt,
> 
> Hi David,
> 
> The draft is well written and I regard it as very helpful.
> 
> There's one change I'd like to suggest:
> 
> Section 3.1, list number 2. briefly explains AF. The section doesn't
> explicitly mention that traffic carried by one of the three PHBs of an AF
> class are not reordered. This feature, avoiding reordering packets belonging
> to different PHBs of an AF class, is however a kind of punch line whenever AF
> is referred to in later sections. So I'd welcome mentioning it in section 3.1,
> list point 2.
> 
> ################# minor comments (no objection if ignored)
> #############################
> 
> Section 4. , second block. Why are I-frames marked AF41 and P-frames AF43 in
> the example? To me, AF42 for P makes more sense (or make it AF42 for I
> frames). AF43 could be used for B-frames (which aren't mentioned in the text).
> 
> Section 5.1 second block after the bullet pointed list
> 
> ".making reordering very likely."
> 
> Would "making reordering likely" be sufficient?
> 
> Section 5.1
> 
> There's at least one ECMP implementation I'm aware of which includes QoS bits
> to calculate the load balancing hash value. It reorders packets of a flow
> using multiple classes, if they are spaced less than the resulting delay
> difference. This is an exceptional implementation, the other solutions I'm
> aware of do ignore QoS bits when calculating ECMP hash values. ECMP is a
> proprietary feature, I think.
> 
> And one editorial comment:
> 
> Section 3., last bullet point and following section: the bullet point is
> focused on Lower Effort PHB marked by CS 1 and the following section continues
> by discussing CS 1 issues more general. This discussion may be added to the
> bullet point (to me the discussion doesn't seem to be related to the first
> bullet point).
> 
> ########################################################
> 
> Sorry for being late with my review.
> 
> Regards,
> 
> Ruediger
> 
> 
>