[Gen-art] Gen-ART review of draft-ietf-payload-rtp-ancillary-10

Christer Holmberg <christer.holmberg@ericsson.com> Fri, 25 August 2017 14:01 UTC

Return-Path: <christer.holmberg@ericsson.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 85CBF13294F; Fri, 25 Aug 2017 07:01:57 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.219
X-Spam-Level:
X-Spam-Status: No, score=-4.219 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_MED=-2.3, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, SPF_PASS=-0.001, URIBL_BLOCKED=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 k5CN3wldJFhS; Fri, 25 Aug 2017 07:01:55 -0700 (PDT)
Received: from sessmg23.ericsson.net (sessmg23.ericsson.net [193.180.251.45]) (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 D168A13238E; Fri, 25 Aug 2017 07:01:51 -0700 (PDT)
X-AuditID: c1b4fb2d-4e93a9c0000057a4-fb-59a02dcd395d
Received: from ESESSHC020.ericsson.se (Unknown_Domain [153.88.183.78]) by sessmg23.ericsson.net (Symantec Mail Security) with SMTP id 10.80.22436.DCD20A95; Fri, 25 Aug 2017 16:01:50 +0200 (CEST)
Received: from ESESSMB109.ericsson.se ([169.254.9.194]) by ESESSHC020.ericsson.se ([153.88.183.78]) with mapi id 14.03.0352.000; Fri, 25 Aug 2017 16:01:48 +0200
From: Christer Holmberg <christer.holmberg@ericsson.com>
To: "gen-art@ietf.org" <gen-art@ietf.org>
CC: "draft-ietf-payload-rtp-ancillary.all@ietf.org" <draft-ietf-payload-rtp-ancillary.all@ietf.org>
Thread-Topic: Gen-ART review of draft-ietf-payload-rtp-ancillary-10
Thread-Index: AQHTHaqxEo34UfG1SUuQcI7eQ4U2ew==
Date: Fri, 25 Aug 2017 14:01:47 +0000
Message-ID: <D5C6097B.204C3%christer.holmberg@ericsson.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
user-agent: Microsoft-MacOutlook/14.7.4.170508
x-originating-ip: [153.88.183.16]
Content-Type: multipart/alternative; boundary="_000_D5C6097B204C3christerholmbergericssoncom_"
MIME-Version: 1.0
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFuplkeLIzCtJLcpLzFFi42KZGbHdT/ec7oJIgwsvmSymPPvAYnH11WcW ByaPJUt+MgUwRnHZpKTmZJalFunbJXBl/FnxlrVgcmLFq/u9TA2MPcFdjJwcEgImEg/7zzB2 MXJxCAkcYZS4s3c9K4SzhFFi3qNNQBkODjYBC4nuf9ogDSICmhJzV7xlArGZBVIlvu+awQ5i CwvYSbyb3s8IUeMssezRaihbT+LR0XVgNSwCqhLNvxtZQWxeAWuJS4fuMYPYjAJiEt9PrYGa KS5x68l8JojjBCSW7DnPDGGLSrx8/I8V5BxRoJnv9ntChBUldp5tZ4ZoTZDY92IHG8R4QYmT M5+wTGAUnoVk6iwkZbOQlEHEDSTen5vPDGFrSyxb+BrK1pfY+OUs4yygzcxAV1+8noKsZAEj xypG0eLU4uLcdCNjvdSizOTi4vw8vbzUkk2MwEg6uOW37g7G1a8dDzEKcDAq8fBe1V4QKcSa WFZcmXuIUYKDWUmE9wpIiDclsbIqtSg/vqg0J7X4EKM0B4uSOK/DvgsRQgLpiSWp2ampBalF MFkmDk6pBsbCR3HGC78tL2cvmdTzc+t6qYlTpsTz1ta98F1ZYeq79Jv4+7o7IUc0Z0cu1Jrz +lv4nB/3jqj78J85yluR8/2VfIaBr2h4mMjDCOZKs7xf2QdXbN0sm73JP7DAupn1ffVepqxd EjYnYu/yVh7bLXkp4oDto2UX3t7yLwpgVNy1KtqiqPWi4SclluKMREMt5qLiRAAHq278oAIA AA==
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/7SoKDiTMY8bsG04saPJ9OSk-zUU>
Subject: [Gen-art] Gen-ART review of draft-ietf-payload-rtp-ancillary-10
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.22
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: <https://mailarchive.ietf.org/arch/browse/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: Fri, 25 Aug 2017 14:01:57 -0000

I am the assigned Gen-ART reviewer for this draft. For background on Gen-ART, please see the FAQ at <http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>



Document:                       draft-ietf-payload-rtp-ancillary-10.txt

Reviewer:                         Christer Holmberg

Review Date:                   25 August 2017

IETF LC End Date:           25 July 2017

IETF Telechat Date:        N/A



Summary: The document is well written. However, I have a few issues that I would like the authors to address.


Major Issues: None



Minor Issues:

Q1: In section 5, I think you should say that, in subsequent offers/answers, the used DID/SDID values must be included in the SDP, even if the values haven’t changed.

Q2: In section 4, I think it would be good to indicate whether there are any default values for the parameters, or whether omitting them means that they are not specified.

Q3: In section 5, I don’t think you need to say that the answerer MAY reject the offer. That is basic offer/answer. Instead, the text could say that the answerer, if it accepts the offer, MUST respond with all or subset, and that the answerer MUST NOT include things that weren’t in the offer…

Q4: Related to Q3, it would be good to indicate if there are specific criteria for rejecting the offer. For example if the answerer is not able to provide a subset of the offered values. Or, is it allowed for the answerer to not return anything?

Q5: Do we really need Section 5.2? My suggestion would be to remove the declarative considerations – unless it is known that someone is actually going to use it.

Q6: Section 4.1 talks about associating ANC RTP streams with other streams. What if ANC RTP streams are multiplexed with other RTP streams (using the SDP BUNDLE mechanism)? I assume that does not automatically mean they are associated? It would be good to have a “Multiplexing Considerations” section and describe that.



Editorial Issues:

Q1: In the Introduction, please add a reference to RFC 3550 on the first occurrence of “RTP”.

Q2: In some places, the text says “RFC 3550 [RFC3550]”, "RTP RFC 3550 [RFC3550]”etc. I suggest to simply say "[RFC3550]”.