Re: [payload] New version of draft-ietf-payload-vp8
"Ben Campbell" <ben@nostrum.com> Tue, 22 September 2015 17:43 UTC
Return-Path: <ben@nostrum.com>
X-Original-To: payload@ietfa.amsl.com
Delivered-To: payload@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 2D47A1A911B for <payload@ietfa.amsl.com>; Tue, 22 Sep 2015 10:43:43 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.91
X-Spam-Level:
X-Spam-Status: No, score=-1.91 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, T_RP_MATCHES_RCVD=-0.01] 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 LtxT8HbpTU8h for <payload@ietfa.amsl.com>; Tue, 22 Sep 2015 10:43:41 -0700 (PDT)
Received: from nostrum.com (raven-v6.nostrum.com [IPv6:2001:470:d:1130::1]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 192BD1A9114 for <payload@ietf.org>; Tue, 22 Sep 2015 10:43:41 -0700 (PDT)
Received: from [172.20.10.2] (mobile-166-173-187-116.mycingular.net [166.173.187.116]) (authenticated bits=0) by nostrum.com (8.15.2/8.14.9) with ESMTPSA id t8MHhWQK077498 (version=TLSv1 cipher=DHE-RSA-AES128-SHA bits=128 verify=NO); Tue, 22 Sep 2015 12:43:37 -0500 (CDT) (envelope-from ben@nostrum.com)
X-Authentication-Warning: raven.nostrum.com: Host mobile-166-173-187-116.mycingular.net [166.173.187.116] claimed to be [172.20.10.2]
From: Ben Campbell <ben@nostrum.com>
To: Harald Alvestrand <harald@alvestrand.no>
Date: Tue, 22 Sep 2015 12:43:26 -0500
Message-ID: <C760EE04-FB6D-4B9C-A555-F7BC1DA4DB17@nostrum.com>
In-Reply-To: <560190B1.3040400@alvestrand.no>
References: <CAOhzyf=qB9MNQO6=AjC8OArrxcC8zDwsOa_RdF2aV-3jwmHLCw@mail.gmail.com> <05DD07CB-26A5-4D07-9605-86C7D321B093@nostrum.com> <658F1EF8-8933-43E7-ADDF-173904FE60A2@csperkins.org> <55F1AEC9.6050408@alvestrand.no> <4D9392D8-4790-41EB-9FC6-B7CC98895E10@nostrum.com> <560190B1.3040400@alvestrand.no>
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"; format="flowed"
Content-Transfer-Encoding: 8bit
X-Mailer: MailMate (1.9.2r5107)
Archived-At: <http://mailarchive.ietf.org/arch/msg/payload/ruxY0kX-hZV8wa2gOzQG3PRMpMU>
Cc: payload@ietf.org
Subject: Re: [payload] New version of draft-ietf-payload-vp8
X-BeenThere: payload@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: Audio/Video Transport Payloads working group discussion list <payload.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/payload>, <mailto:payload-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/payload/>
List-Post: <mailto:payload@ietf.org>
List-Help: <mailto:payload-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/payload>, <mailto:payload-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 22 Sep 2015 17:43:43 -0000
inline: On 22 Sep 2015, at 12:32, Harald Alvestrand wrote: > On 09/22/2015 04:40 PM, Ben Campbell wrote: >> Hi Harald, >> >> I will check with Alissa to see if her concerns have been addressed. >> >> Has there been closure on Elwyn's Gen-ART telechat review and on >> Colin's comments on the payload list? I was under the impression that >> at least Colin's comments might result in an update (although we can >> handle minor changes with RFC editor notes) I didn't see a response to the above. IIUC, these are distinct from Colin's and Elwyn's other points that I had copied below. >> >> Also, I had some comments a while back to which I do not recall >> seeing >> a response: >> >>> -- section 3, Payload Type: "... the VP8 RTP payload profile MUST >>> assign a dynamic payload type number ..." >>> >>> If I read correctly, Colin Perkins objected to this change. Did you >>> consider his objection? > I asked in response if we can delete the whole thing about payload > type > assignment, since it's not a proper concern of the payload type. I did > not get a response, so I thought it couldn't be important. It's probably worth poking Colin to make sure he's paying attention (I will do so.) But if we don't see a quick response, then I will agree with your assessment. >>> >>> -- 4.2 >>> >>> I didn't see anything addressing Elwyn's Gen-ART (last call) review >>> question about: “What happens if L=1 but both T=0 and K=0 so that >>> there is no TID value present? Or indeed if T=0 but K=1 so that the >>> TID field is there but 'MUST be ignored by the receiver' >>> (definition >>> of TID field)?” Did I miss something? > > On this matter, we thought the text was clear that those streams would > be illegal, and we don't specify the processing of illegal streams. > But > if it isn't clear that it's illegal, we may want to reconsider. I think that's a reasonable response, and a change is probably not needed. >> >> Thanks! >> >> Ben. >> >> >> On 10 Sep 2015, at 11:24, Harald Alvestrand wrote: >> >>> On 09/10/2015 03:28 AM, Colin Perkins wrote: >>>>> On 9 Sep 2015, at 23:12, Ben Campbell <ben@nostrum.com> wrote: >>>>> >>>>> On 9 Sep 2015, at 16:01, Henrik Lundin wrote: >>>>> >>>>>> We have made no technical changes to the document, but have added >>>>>> a number of grammar updates and clarifications. Some of the >>>>>> important clarifications are as follows: >>>>> Payload participants: Please note that there are in fact changes >>>>> to >>>>> 2119 language in this version. I won't quibble about whether or >>>>> not >>>>> those count as "technical" changes, but please take a moment to >>>>> review them. If people object to any of them please say so asap , >>>>> as this draft will likely be on the agenda for the IESG telechat >>>>> next week. >>>> The new text about the Payload Type in Section 4.1 says: >>>> >>>> Payload type (PT): In line with the policy in Section 3 of >>>> [RFC3551], applications using the VP8 RTP payload profile MUST >>>> assign a dynamic payload type number to be used in each RTP >>>> session and provide a mechanism to indicate the mapping. See >>>> Section 6.2 for the mechanism to be used with the Session >>>> Description Protocol (SDP) [RFC4566]. >>>> >>>> This would be okay, except that RTP profiles that don’t derive >>>> from >>>> RFC 3551 could be defined. A better wording might be: >>>> >>>> Payload Type (PT): The assignment of an RTP payload type for this >>>> packet format is outside the scope of this document and will not be >>>> specified here. It is expected that the RTP profile for a >>>> particular >>>> class of applications will assign a payload type for this encoding, >>>> or if that is not done, then a payload type in the dynamic range >>>> SHALL be chosen by means of an out-of-band signaling protocol. See >>>> Section 6.2 for the mechanism to be used with the Session >>>> Description >>>> Protocol (SDP) [RFC4566]. >>>> >>>> >>>> In Section 4.2, rather than add a note that “this X bit is not to >>>> be >>>> confused with the X bit in the RTP header”, would it not make >>>> sense >>>> to rename this X bit? >>> >>> We considered this option, but decided that having a large existing >>> body >>> of documentation outside of the IETF that refers to the "X bit" and >>> having a payload type registration that called it something else, it >>> ws >>> better to keep the name and just make a note that it's different. >>> >>>> Similarly for the M bit in the extension data fields, and the P bit >>>> in the payload header (which isn’t mentioned). Or, at least, >>>> annotate each mention of these fields to say which X, M, or P bit >>>> it >>>> required (e.g., “the P bit in the payload header” or “the P >>>> bit in >>>> the RTP header” rather than “the P bit”). >>> >>> This can be done, and probably should be done for any reference that >>> occurs outside of the section describing the VP8 payload descriptor >>> (section 4.2) - I couldn't find any such references. >>> >>> There is no P bit in the VP8 payload descriptor, but there is one in >>> the >>> VP8 payload header, which is why we missed warning about the name >>> collision there. It's never referred to in text. >>> >>> >>>> >>>> Colin >>>> >>>> >>>> >>>> >>>> >>>> >>>> >>> >>> >>> -- >>> Surveillance is pervasive. Go Dark. >>> >>> >>> _______________________________________________ >>> payload mailing list >>> payload@ietf.org >>> https://www.ietf.org/mailman/listinfo/payload > > > -- > Surveillance is pervasive. Go Dark.
- [payload] New version of draft-ietf-payload-vp8 Henrik Lundin
- Re: [payload] New version of draft-ietf-payload-v… Ben Campbell
- Re: [payload] New version of draft-ietf-payload-v… Ben Campbell
- Re: [payload] New version of draft-ietf-payload-v… Colin Perkins
- Re: [payload] New version of draft-ietf-payload-v… Harald Alvestrand
- Re: [payload] New version of draft-ietf-payload-v… Colin Perkins
- Re: [payload] New version of draft-ietf-payload-v… Harald Alvestrand
- Re: [payload] New version of draft-ietf-payload-v… Ben Campbell
- Re: [payload] New version of draft-ietf-payload-v… Harald Alvestrand
- Re: [payload] New version of draft-ietf-payload-v… Ben Campbell
- Re: [payload] New version of draft-ietf-payload-v… Ben Campbell
- Re: [payload] New version of draft-ietf-payload-v… Ben Campbell
- Re: [payload] New version of draft-ietf-payload-v… Colin Perkins
- Re: [payload] New version of draft-ietf-payload-v… Harald Alvestrand
- Re: [payload] New version of draft-ietf-payload-v… Colin Perkins
- Re: [payload] New version of draft-ietf-payload-v… Harald Alvestrand
- Re: [payload] New version of draft-ietf-payload-v… Colin Perkins
- Re: [payload] New version of draft-ietf-payload-v… Harald Alvestrand
- Re: [payload] New version of draft-ietf-payload-v… Ben Campbell
- Re: [payload] New version of draft-ietf-payload-v… Harald Alvestrand