[AVT] Re: Status of draft-kerr-avt-vorbis-rtp-01

Colin Perkins <csp@csperkins.org> Tue, 06 May 2003 22:30 UTC

Received: from www1.ietf.org (ietf.org [132.151.1.19] (may be forged)) by ietf.org (8.9.1a/8.9.1a) with ESMTP id SAA12478 for <avt-archive@odin.ietf.org>; Tue, 6 May 2003 18:30:57 -0400 (EDT)
Received: (from mailnull@localhost) by www1.ietf.org (8.11.6/8.11.6) id h46Mdc904451 for avt-archive@odin.ietf.org; Tue, 6 May 2003 18:39:38 -0400
Received: from www1.ietf.org (localhost.localdomain [127.0.0.1]) by www1.ietf.org (8.11.6/8.11.6) with ESMTP id h46McC804279; Tue, 6 May 2003 18:38:12 -0400
Received: from ietf.org (odin.ietf.org [132.151.1.176]) by www1.ietf.org (8.11.6/8.11.6) with ESMTP id h46MZl803305 for <avt@optimus.ietf.org>; Tue, 6 May 2003 18:35:47 -0400
Received: from ietf-mx (ietf-mx.ietf.org [132.151.6.1]) by ietf.org (8.9.1a/8.9.1a) with ESMTP id SAA12326 for <avt@ietf.org>; Tue, 6 May 2003 18:26:36 -0400 (EDT)
Received: from ietf-mx ([132.151.6.1]) by ietf-mx with esmtp (Exim 4.12) id 19DAvR-00033n-00 for avt@ietf.org; Tue, 06 May 2003 18:28:41 -0400
Received: from wireless249.east.isi.edu ([65.123.202.249] helo=purple.nge.isi.edu) by ietf-mx with esmtp (Exim 4.12) id 19DAvP-00033k-00 for avt@ietf.org; Tue, 06 May 2003 18:28:41 -0400
Received: from purple.nge.isi.edu (localhost [127.0.0.1]) by purple.nge.isi.edu (8.12.9/8.12.9) with ESMTP id h46MTPIs070328; Tue, 6 May 2003 18:29:25 -0400 (EDT) (envelope-from csp@purple.nge.isi.edu)
To: philkerr@elec.gla.ac.uk
cc: avt@ietf.org
In-Reply-To: Your message of "Thu, 01 May 2003 14:08:02 BST." <1051794482.3eb11c325c7ba@dlana.elec.gla.ac.uk>
Date: Tue, 06 May 2003 18:29:25 -0400
Message-ID: <70327.1052260165@purple.nge.isi.edu>
From: Colin Perkins <csp@csperkins.org>
Subject: [AVT] Re: Status of draft-kerr-avt-vorbis-rtp-01
Sender: avt-admin@ietf.org
Errors-To: avt-admin@ietf.org
X-BeenThere: avt@ietf.org
X-Mailman-Version: 2.0.12
Precedence: bulk
List-Unsubscribe: <https://www1.ietf.org/mailman/listinfo/avt>, <mailto:avt-request@ietf.org?subject=unsubscribe>
List-Id: Audio/Video Transport Working Group <avt.ietf.org>
List-Post: <mailto:avt@ietf.org>
List-Help: <mailto:avt-request@ietf.org?subject=help>
List-Subscribe: <https://www1.ietf.org/mailman/listinfo/avt>, <mailto:avt-request@ietf.org?subject=subscribe>

Hi Phil,

--> philkerr@elec.gla.ac.uk writes:
>I'm checking on the status of draft-kerr-avt-vorbis-rtp-01 and how things can be
>moved forward with it.  The update was submitted just before the cutoff for the
>last AVT meeting and there seems to have been no action on it since.

I took the liberty of cc'ing the AVT mailing list, to encourage feedback.

>There are a few small changes I may wish to make to the draft, which will be
>discussed at a Vorbis meeting tomorrow, but I wanted to check with you first on
>if the 01 draft is good enough to move forward.

I think it's in good shape, although I have a couple of issues:

 - Section 2.1 notes that the P, X and CC fields of the RTP header are set
   to 0. I'm not sure it's appropriate for a payload format to specify this: 
   I can imagine valid scenarios where each of these can be used with Vorbis.

 - The discussion in section 3 can make use of normative language to be
   clear on how frames are packetized. I suggest the following changes:

       Any Vorbis packet that is larger than 256 octets and less than the
       path-MTU should be placed in a RTP packet by itself.
                ^^^^^^ MUST

       Any Vorbis packet that is 256 bytes or less should be bundled in the
                                                   ^^^^^^ SHOULD
       RTP packet with as many Vorbis packets as will fit, up to a maximum
       of 32.

       If a Vorbis packet will not fit into the RTP packet, it must be 
                within the network MTU ^^^^^^^^^^^^^^^^^^^     ^^^^ SHOULD
       fragmented.  A fragmented packet has a zero in the last five bits 
       of the payload header.  Each fragment after the first will also set 
       the Continued (C) bit to one in the payload header.  The RTP packet 
       containing the last fragment of the Vorbis packet will have the 
       Marker (F) bit set to one.
       ^^^^^^ Final Fragment 
                  (to avoid confusion with the RTP Marker bit)

 - The IANA considerations section needs to be expanded. Section 4 of RFC
   3047 is a good example, to illustrate the format.

 - Regarding the configuration headers, is there a need to send updates
   during a session? If not, it might be appropriate to define some SDP
   parameters to convey the configuration data at session initiation time,
   rather than relying on RTCP. If RTCP is to be used, it's necessary to
   discuss reliability, and how a receiver reacts if the information is
   lost.

I also have a few editorial comments:

 - The interpretation of key words and reference to RFC 2119 should be
   moved into the Introduction rather than being in the Status of this
   Memo section.

 - I suggest moving the last three paragraphs of the Introduction into
   section 2.3, where the packing of the payload data is discussed. It
   may also be appropriate to include a slightly longer description of
   the Vorbis codec and when it might be useful in the Introduction.

 - In section 3.1, it might be useful to include the RTP packet header 
   details, to show how the RTP sequence number and timestamp are used
   (sequence number increases by one for each packet, timestamp stays 
   the same for each fragment).

 - Section 7 might reference the discussion of congestion control in 
   the RTP spec and/or profile

 - References should be split into Normative and Informative sections.


Cheers,
Colin
_______________________________________________
Audio/Video Transport Working Group
avt@ietf.org
https://www1.ietf.org/mailman/listinfo/avt