[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[AVT] Comments on draft-chen-rtp-bv-00.txt



Hi,

Here are some comments on this draft. Before this I would like to say thank you to the authors for providing a very good initial version of the draft. I have very few comments.

1. There is need to tighten up the use of aggregation of consecutive frames. I would propose that in section 3.2 and 4.2 add a sentence stating that any frames in payload MUST be consecutive. If that is not intended there is need for having some mechanism for recovering the timestamp for each individual frame.

2. Section 3 and 4 are both missing specification for how to use the marker bit. I would suggest the normal definition that for packets with there first frame being an speech onset the marker bit is set to 1, otherwise 0. Please see draft-ietf-avt-profile-new-13.txt

3. I would also like to see some more text on how one recovers the RTP timestamp for an aggregated frame. I now that as long as you only intend to aggregate consecutive frames it will be RTP TS + TS_rate*0.005*(n-1) however this should be written out.

4. I would also suggest that you tighten the language in Section 5 regarding that BroadVoice 16 and 32 frames shall not be mixed in a file. Please use the suitable RFC 2119 language for this.

5. Section 6.1, the usage of maxptime. Are there any need to define a default value for maxptime and especially of selecting 200 ms?

6. Section 6.1, Public Specification. This needs to be a complete reference.

7. Section 7. I would suggest that you improve your text for how to map the parameters to SDP. I would suggest borrowing text from section 8.3 of RFC 3267.

8. The reference list needs to be split into normative and informative references.


Best Regards


Magnus Westerlund

Multimedia Technologies, Ericsson Research EAB/TVA/A
----------------------------------------------------------------------
Ericsson AB | Phone +46 8 4048287
Torshamsgatan 23 | Fax +46 8 7575550
S-164 80 Stockholm, Sweden | mailto: magnus.westerlund@ericsson.com



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