[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
RE: [AVT] RE: I-D ACTION:draft-ash-avt-hc-over-mpls-protocol-00.txt
Colin,
Thanks very much for your comments, please see responses below.
Jerry
> The definition of terms and reference to RFC 2119 needs to be moved
> within the body of the text, rather than being a separate unnumbered
> section. I'd suggest placing the paragraph at the start of Section 2.
>
> Section 1, 2nd paragraph: "the compressed packet routed on an MPLS
> LSP using MPLS labels" --> "the compressed packet _is_then_ routed..."
>
> Section 2, 1st paragraph: I found the phrasing about pseudowires
> transporting the "HC context and other control messages" a little
> confusing, implying that the context and control messages are
> transported separately to the compressed data packets. It might be
> clearer to say "MPLS pseudowires (PWs) are used to transport the
> header compressed packets between the ingress and egress...".
>
> Section 2, 3rd paragraph: The sentence "Compressed packets and HC
> control packets are routed on a separate MPLS LSP/PW..." could be
> interpreted as a requirement to setup two PWs, one for the HC
> compressed data packets and one for the HC control packets. It might
> be clearer to state "Compressed data is routed on a separated MPLS
> LSP/PW from compressor to decompressor"?
>
> Section 2.1, top of page 7: I agree that there is no problem with
> duplicate CIDs since they're received on separate sessions. However,
> the text can be read to imply that something other than normal CID
> processing is required. This confusion comes from the statement
> following ("...the HD can distinguish these this it knows the
> source by means of the PW label"), which implies that there is a
> single HD for all PWs. It would be clearer if the draft made it
> explicit that each PW had a logically separate HD instance,
> independent of all other PWs.
Agree with all of the above comments/suggestions; will change
accordingly.
> Section 2.1, last paragraph: in order to assign a new PW type, the
> IANA must be given explicit guidance on what to assign. This
> requires a statement much like those in section 2.1 of the
> pwe3-iana-allocation draft, listing a numeric type to assign along
> with a description for that type.
This can be clarified by changing Section 6 'IANA Considerations' in the
draft to read:
"As discussed in Section 2.1, a new PW type as defined in [IANA] needs
to be assigned as follows:
PW type Description
0x00?? Header-compressed packets"
> Section 2.2: how are the reserved values to be allocated? The draft
> needs to either define an allocation policy (and IANA registry) or
> mandate that they MUST NOT be assigned.
You have a good point. We can change the "RESERVED" codepoint labels to
"NOT USED", making it clearer that they're not reserved for future
assignment, but rather will never be used.
_______________________________________________
Audio/Video Transport Working Group
avt at ietf.org
https://www1.ietf.org/mailman/listinfo/avt