Re: draft-ietf-l2tpext-tdm-05

Alexander Vainshtein <[email protected]> Sun, 3 Aug 2008 16:23:53 +0300
Newsgroups gmane.ietf.l2tpext
Message-ID <A3C5DF08D38B6049839A6F553B331C7680265CEA76@ILPTMAIL02.ecitele.com>
Ignacio,
Lots of thanks for a very detailed review.

I agree with you that the document is ready for the WG LC, since most of the comments are editorial and can be done after the LC.

I would appreciate  your advice regarding split of the IANA Considerations section.

Regards,
        Sasha

> -----Original Message-----
> From: Ignacio Goyret [mailto:[email protected]]
> Sent: Friday, August 01, 2008 10:00 PM
> To: Alexander Vainshtein
> Cc: [email protected]
> Subject: draft-ietf-l2tpext-tdm-05
>
> Hi Sasha,
> This last rev has been a big improvement. Thanks for your work!
>
> I have just a few more editorial changes:
>
> * Section 2, last paragraph (page 3)
>
> OLD: There are two new AVPs for the Session Connection Messages.
>                                             ^^^^^^^^^^ ^
> NEW: There are two new AVPs for the Session Management messages.
>                                             ^^^^^^^^^^ ^
>
> * Section 2.1, page 4:
>
> OLD:
>      a) The specified value MUST be an integer multiple of
>         number of DS0 channels in the corresponding attachment
>         circuit.
> NEW:
>      a) The specified value MUST be an integer multiple of the
>                                                            ^^^
>         number of DS0 channels in the corresponding attachment
>         circuit.
>
> * Section 2.2, page 5:
> OLD:  PT is the payload type expected in the RTP header.
> Value of zero
>                                                           ^
> NEW:  PT is the payload type expected in the RTP header.  A
> value of zero
>                                                           ^^^
>
> * Section 3, 2nd bullet point under "CESoPSN basic" (page 7):
>
>    s/teh/the/
>
> * IANA Considerations:
>   Just to make it clear to everyone involved, you may want to separate
>   the lonely return code from the list of return/error codes and make
>   two lists: a return code list (one item) and an error code
> list (with
>   the 5 items).
>
> The text looks reasonably solid to me, so I believe we are ready
> for WGLC.
>
> Thanks again for the effort that you have put into this work.
>
> Cheers,
> -Ignacio
>
>