[mpls] Comments on draft-ietf-mpls-sfc-01

["Andrew G. Malis" <[e-mail]>]

Adrian et al,

I just had a chance to review draft-ietf-mpls-sfc-01 in detail, and have a
few comments.

First of all, thanks for including the section 4.5 text.

That said, I’m having a hard time distinguishing between using MPLS-SR for
SFC and the “micro chains” discussed in section 4.4. I agree that the
“micro chains" are a logical extreme of section 4.2, but that said, section
4.2 does include swapping as well as popping, while section 4.4 is pure
popping. It would be greatly appreciated if you could add text that
differentiates “micro chains” from MPLS-SR, otherwise it’s just a different
name for the same functionality. If there is actually is no difference,
then section 4.4 should be removed.

I also have a comment in section 12. The text currently says "It should be
noted that the MPLS encoding is slightly less functional than the direct
use of the NSH.” In this case, “slightly” is a judgement for the reader to
make, and I see a huge loss of functionality because there’s no support for
per-packet metadata. Please remove the word “slightly” in this text.
Without the word “slightly”, the text changes from an opinion to a fact.

Thanks,
Andy