Re: [MMUSIC] Additional editorial change suggestions to draft-ietf-mmusic-data-channel-sdpneg

"Roni Even (A)" <roni.even@huawei.com> Sun, 24 June 2018 05:49 UTC

Return-Path: <roni.even@huawei.com>
X-Original-To: mmusic@ietfa.amsl.com
Delivered-To: mmusic@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id BF2A712F1AC; Sat, 23 Jun 2018 22:49:20 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.19
X-Spam-Level:
X-Spam-Status: No, score=-4.19 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_MED=-2.3, SPF_PASS=-0.001, T_KAM_HTML_FONT_INVALID=0.01] 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 eK7liS4x7bby; Sat, 23 Jun 2018 22:49:17 -0700 (PDT)
Received: from huawei.com (lhrrgout.huawei.com [194.213.3.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 6922E128CF3; Sat, 23 Jun 2018 22:49:17 -0700 (PDT)
Received: from lhreml707-cah.china.huawei.com (unknown [172.18.7.106]) by Forcepoint Email with ESMTP id C72D69F3C91AD; Sun, 24 Jun 2018 06:49:10 +0100 (IST)
Received: from DGGEMM404-HUB.china.huawei.com (10.3.20.212) by lhreml707-cah.china.huawei.com (10.201.108.48) with Microsoft SMTP Server (TLS) id 14.3.382.0; Sun, 24 Jun 2018 06:49:12 +0100
Received: from DGGEMM506-MBX.china.huawei.com ([169.254.3.222]) by DGGEMM404-HUB.china.huawei.com ([10.3.20.212]) with mapi id 14.03.0382.000; Sun, 24 Jun 2018 13:49:07 +0800
From: "Roni Even (A)" <roni.even@huawei.com>
To: Christer Holmberg <christer.holmberg@ericsson.com>, "mmusic@ietf.org" <mmusic@ietf.org>
CC: "mmusic-chairs@ietf.org" <mmusic-chairs@ietf.org>
Thread-Topic: Additional editorial change suggestions to draft-ietf-mmusic-data-channel-sdpneg
Thread-Index: AdQK4xMejienA30iT7uESrzuitKBjQAm3AwQ
Date: Sun, 24 Jun 2018 05:49:06 +0000
Message-ID: <6E58094ECC8D8344914996DAD28F1CCD89C424@DGGEMM506-MBX.china.huawei.com>
References: <90db57fa68fe451eb328104436eb7a43@ericsson.com>
In-Reply-To: <90db57fa68fe451eb328104436eb7a43@ericsson.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [10.200.202.67]
Content-Type: multipart/alternative; boundary="_000_6E58094ECC8D8344914996DAD28F1CCD89C424DGGEMM506MBXchina_"
MIME-Version: 1.0
X-CFilter-Loop: Reflected
Archived-At: <https://mailarchive.ietf.org/arch/msg/mmusic/uQGX3-K1C82YntnlHVoMCq6p6fM>
Subject: Re: [MMUSIC] Additional editorial change suggestions to draft-ietf-mmusic-data-channel-sdpneg
X-BeenThere: mmusic@ietf.org
X-Mailman-Version: 2.1.26
Precedence: list
List-Id: Multiparty Multimedia Session Control Working Group <mmusic.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/mmusic>, <mailto:mmusic-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/mmusic/>
List-Post: <mailto:mmusic@ietf.org>
List-Help: <mailto:mmusic-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/mmusic>, <mailto:mmusic-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sun, 24 Jun 2018 05:49:21 -0000

HI Christer,
Thanks to you and Bo for the comments. I added them to the -20 version.
As for comment 9 I added the text to the end of section 6. There is text in section 5.2.1 but I think it is good to have your suggested text in section 6 that talks about offer/answer

Roni

From: Christer Holmberg [mailto:christer.holmberg@ericsson.com]
Sent: Saturday, June 23, 2018 2:13 PM
To: mmusic@ietf.org
Cc: Roni Even (A); mmusic-chairs@ietf.org
Subject: Additional editorial change suggestions to draft-ietf-mmusic-data-channel-sdpneg

Hi,

I got some off-line comments from Bo, on the text modifications I suggested for draft-ietf-mmusic-data-channel-sdpneg, and the impact it had on some other text. So, below are some changes that I still think need to be done:

---


Q1 (section 6):



Old text:



   "This section defines how data channels can be negotiated using the

   SDP offer/answer mechanism.  The procedures apply for a given data

   channel."



It was commented that the text is not clear enough that multiple data channels can be negotiated separately and independently of each other, even if they are associated with the same media description (m- line). I suggest to clarify that.



Suggested new text:



       "This section defines how data channels can be negotiated using the SDP

       offer/answer mechanism. A given media description can describe multiple

       data channels (each represented by a separate SDP dcmap attribute) that

       can be created, modified and closed using different offer/answer exchanges.

       The procedures in this section apply for a given data channel."



---



Q2 (section 6):



Old text:

   "The generic offer/answer procedures for negotiating the SCTP
   association used to realize are defined in
   [I-D.ietf-mmusic-sctp-sdp<https://datatracker.ietf.org/doc/html/draft-ietf-mmusic-data-channel-sdpneg-19#ref-I-D.ietf-mmusic-sctp-sdp>]. This section only defines the data
   channel specific procedures."



There is text missing between 'realize' and 'are'.



Suggested new text:


   "The generic offer/answer procedures for negotiating the SCTP
   association used to realize data channels are defined in
   [I-D.ietf-mmusic-sctp-sdp<https://datatracker.ietf.org/doc/html/draft-ietf-mmusic-data-channel-sdpneg-19#ref-I-D.ietf-mmusic-sctp-sdp>]. This section only defines the data
   channel specific procedures."



---



Q3 (section 6.1):



Old text:



   "As described in [I-D.ietf-rtcweb-data-protocol<https://datatracker.ietf.org/doc/html/draft-ietf-mmusic-data-channel-sdpneg-19#ref-I-D.ietf-rtcweb-data-protocol>], in order to avoid

   SCTP Stream identifier collisions, the endpoint acting as DTLS client

   (for the SCTP association used to realize data channels) will use

   even identifier values, and the endpoint acting as DTLS server will

   use odd identifier values.  Those rules also apply when SCTP streams

   for data channels are negotiated using the offer/answer mechanism.

   SCTP stream identifiers associated with data channels that have been

   negotiated using DCEP MUST NOT be included in SDP offers and answers."



It was commented that the text makes normative references to draft-ietf-rtcweb-data-protocol, which is an informative reference (you don't need to support DCEP if you use SDP O/A). However, the DCEP rules regarding even/odd identifier values are also used for SDP.



Suggested new text:



      "In order to avoid SCTP Stream identifier collisions, in alignment with

[I-D.ietf-rtcweb-data-protocol<https://datatracker.ietf.org/doc/html/draft-ietf-mmusic-data-channel-sdpneg-19#ref-I-D.ietf-rtcweb-data-protocol>], the endpoint acting as DTLS client (for

the SCTP association used to realize data channels) MUST use even identifier values,

and the endpoint acting as DTLS server MUST use odd identifier values. SCTP stream

identifiers associated with data channels that have been negotiated using DCEP MUST NOT

be included in SDP offers and answers."



----



Q4 (section 6.2):



Old text:



   "The data channel types defined in [I-D.ietf-rtcweb-data-protocol<https://datatracker.ietf.org/doc/html/draft-ietf-mmusic-data-channel-sdpneg-19#ref-I-D.ietf-rtcweb-data-protocol>] are

   mapped to the dcmap SDP media description in the following manner

   where "ordered=true" is the default and may be omitted:"



The formulation "the dcmap SDP media description" is a bit awkward. A "dcmap" attribute does not make up a media description. Suggest to replace with "the dcmap attribute parameters".



Suggested new text:



   "The data channel types defined in [I-D.ietf-rtcweb-data-protocol<https://datatracker.ietf.org/doc/html/draft-ietf-mmusic-data-channel-sdpneg-19#ref-I-D.ietf-rtcweb-data-protocol>] are

   mapped to the dcmap attribute parameters in the following manner

   where "ordered=true" is the default and may be omitted:"



---



Q5 (section 6.2):



Old text:



   "The SDP answer SHALL echo the same subprotocol, max-retr, max-time,

   and ordered parameters, form the offer, and MAY include a label

   parameter.  They MAY appear in any order, which could be different

   from the SDP offer, in the SDP answer."



It was commented that the text does not belong to section 6.2, but to section 6.4.



I suggest to remove the text from section 6.2, and modify the first bullet in Section 6.4:



"o SHALL include an SDP dcmap attribute (Section 5 and Section 6.2) associated

with the data channel in the "m=" section representing the SCTP association used

to realize the data channel. The value of the dcmap-stream-id, max-retr and max-time

values of the dcmap attribute SHALL be identical to the value used for the data channel

in the offer; and"



---



Q6 (section 6.2):



Old text:



   "When sending a subsequent offer or an answer, and for as long as the

   data channel is still open, the sender MUST replicate the same

   information."



The text does not belong in Section 6.2, and also already exists in section 6.6 (where it belongs).



I suggested to remove the paragraph.



---



Q7 (section 6.3):



Old text:



   "o  MAY include one or more SDP dcsa attributes (Section 6.2<https://datatracker.ietf.org/doc/html/draft-ietf-mmusic-data-channel-sdpneg-19#section-6.2>)

      associated with the data channel.  The value of the stream-id part

      of each attribute SHALL match the dcmap-stream-id value of the

      dcmap attribute."



The reference to Section 6.2 is wrong. It shall be Section 5.2.



Suggested new text:



   "o  MAY include one or more SDP dcsa attributes (Section 5.2<https://datatracker.ietf.org/doc/html/draft-ietf-mmusic-data-channel-sdpneg-19#section-6.2>)

      associated with the data channel.  The value of the stream-id part

      of each attribute SHALL match the dcmap-stream-id value of the

      dcmap attribute."



---



Q8 (section 6.4):



Old text:



   "o  MAY include one or more SDP dcsa attributes (Section 6.2<https://datatracker.ietf.org/doc/html/draft-ietf-mmusic-data-channel-sdpneg-19#section-6.2>)

      associated with the data channel."



The reference to Section 6.2 is wrong. It shall be Section 5.2.



Suggested text:



   "o  MAY include one or more SDP dcsa attributes (Section 6.2<https://datatracker.ietf.org/doc/html/draft-ietf-mmusic-data-channel-sdpneg-19#section-6.2>)

      associated with the data channel."



---



Q9 (section 6.4):



   o  MAY include one or more SDP dcsa attributes (Section 6.2<https://datatracker.ietf.org/doc/html/draft-ietf-mmusic-data-channel-sdpneg-19#section-6.2>)

      associated with the data channel.



It was questioned whether there are no generic offer/answer considerations for dcsa attributes.



In my opinion each new sub-protocol that uses dcsa attributes must define the sub-protocol specific offer/answer procedures for the attributes. In order to make that clear, I suggest to add the following new text:



"The detailed offer/answer procedures for the dcsa attribute are dependent on the associated sub-protocol.

A sub-protocol specification MUST define the offer/answer procedures for the dsca attribute (if applicable)

associated with the sub-protocol."



---



Q10 (section 6.6):



Old text:



"When an offer sends a subsequent offer, that includes information for a previously

negotiated SCTP stream for a data channel, unless the offerer intends to close the

data channel (Section 6.6.1), the offerer SHALL include the previously negotiated

SDP attributes and attribute values associated with the data channel."



It was commented that the sections contains a single, long sentence that is difficult to parse. I suggest to remove the SCTP part, because it doesn't really belong to this spec.



Suggested new text:



"When an offer sends a subsequent offer that includes information for a previously

negotiated data channel, unless the offerer intends to close the data channel

(Section 6.6.1), the offerer SHALL include the previously negotiated SDP attributes

and attribute values associated with the data channel."



---



Q11 (section 6.7):



Old text:



      "An SDP offer or answer has no "a=dcmap:" attributes but has

      "a=dcsa:" attributes.



      *  This is considered an error case.  In this case the receiver of

         such an SDP offer or answer SHOULD ignore the "a=dcsa:"

         attributes and SHOULD process the SDP offer or answer as per

         above case 'SDP offer has no "a=dcmap:" attributes' or case

         'SDP answer has no "a=dcmap:" attributes'."



Due to some other changes, the reference (to section 6.7) is wrong.



I suggested to remove this bullet, and have a generic statement, e.g., in section 5.2, saying that DCSA attributes without an associated DCMAP attribute MUST be discarded.



Alternatively, somehow remove the "as per above" reference.



---


Regards,

Christer