Re: [payload] New version of draft-ietf-payload-vp8
Colin Perkins <csp@csperkins.org> Tue, 22 September 2015 21:00 UTC
Return-Path: <csp@csperkins.org>
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 822641B2E53 for <payload@ietfa.amsl.com>; Tue, 22 Sep 2015 14:00:59 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.899
X-Spam-Level:
X-Spam-Status: No, score=-1.899 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HTML_MESSAGE=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 A-wloLYMNhfd for <payload@ietfa.amsl.com>; Tue, 22 Sep 2015 14:00:56 -0700 (PDT)
Received: from haggis.mythic-beasts.com (haggis.mythic-beasts.com [IPv6:2a00:1098:0:86:1000:0:2:1]) (using TLSv1.2 with cipher DHE-RSA-AES128-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 907581B2E3E for <payload@ietf.org>; Tue, 22 Sep 2015 14:00:56 -0700 (PDT)
Received: from [81.187.2.149] (port=44225 helo=[192.168.0.21]) by haggis.mythic-beasts.com with esmtpsa (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.80) (envelope-from <csp@csperkins.org>) id 1ZeUgG-0000gK-G7; Tue, 22 Sep 2015 22:00:54 +0100
Content-Type: multipart/alternative; boundary="Apple-Mail=_CB6F1A3F-3F42-4FD3-93E6-B98732CD44F4"
Mime-Version: 1.0 (Mac OS X Mail 8.2 \(2104\))
From: Colin Perkins <csp@csperkins.org>
In-Reply-To: <71734E21-F414-42AD-995F-384A8A719D4C@alvestrand.no>
Date: Tue, 22 Sep 2015 22:00:42 +0100
Message-Id: <23B4B22E-DC89-4BC0-9F9D-404379C8CA28@csperkins.org>
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> <88518539-9321-4616-A84F-736A1C9D2256@csperkins.org> <71734E21-F414-42AD-995F-384A8A719D4C@alvestrand.no>
To: Harald Alvestrand <harald@alvestrand.no>
X-Mailer: Apple Mail (2.2104)
X-BlackCat-Spam-Score: -28
X-Mythic-Debug: Threshold = On =
Archived-At: <http://mailarchive.ietf.org/arch/msg/payload/QXI9_dwTsDNymP7ij-Tefz2m13M>
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 21:00:59 -0000
Remind me? > On 22 Sep 2015, at 21:57, Harald Alvestrand <harald@alvestrand.no> wrote: > > Would you be OK with the shorter version I suggested? > > Den 22. september 2015 22:51:55 CEST, skrev Colin Perkins <csp@csperkins.org>: > On 22 Sep 2015, at 18:32, Harald Alvestrand <harald@alvestrand.no> 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) > > 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. > > I thought the question was for others in the working group. My opinion is that we do need text here, and I proposed something I thought appropriate, although I’m open to other suggestions. The current text does need to be changed, however. > > Colin > > > > > > > -- 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. > > 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 <https://www.ietf.org/mailman/listinfo/payload> > > > -- > Surveillance is pervasive. Go Dark. > > > payload mailing list > payload@ietf.org > https://www.ietf.org/mailman/listinfo/payload <https://www.ietf.org/mailman/listinfo/payload> > > > > -- > Sent from my Android device with K-9 Mail. Please excuse my brevity. -- Colin Perkins https://csperkins.org/
- [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