Re: [sfc] TTL field within the NSH base header

Jim,

This looks good to me.

Thanks,
Andy

On Thu, Apr 27, 2017 at 4:58 PM, James N Guichard <
james.n.guichard@huawei.com> wrote:

> Sorry from 8 to 5 .. geez
>
>
>
> *From:* Andrew G. Malis [mailto:agmalis@gmail.com]
> *Sent:* Thursday, April 27, 2017 4:56 PM
> *To:* Jim Guichard <jguichard1966@gmail.com>
> *Cc:* James N Guichard <james.n.guichard@huawei.com>; sfc@ietf.org
> *Subject:* Re: [sfc] TTL field within the NSH base header
>
>
>
> Jim,
>
>
>
> In that case, you need to remove the word “reserved” from the replacement
> text.
>
>
>
> Cheers,
>
> Andy
>
>
>
>
>
> On Thu, Apr 27, 2017 at 4:37 PM, Jim Guichard <jguichard1966@gmail.com>
> wrote:
>
> Yes but I also counted the other 3 bits that were reserved and are now
> allocated..
>
>
>
> On Thu, Apr 27, 2017 at 4:25 PM Andrew G. Malis <agmalis@gmail.com> wrote:
>
> Jim,
>
>
>
> The replacement text says:
>
>
>
>    There are eight reserved bits in the NSH Base Header. New bits
>
>    are assigned via Standards Action [RFC5226].
>
>
>
> There are now actually five reserved bits.
>
>
>
> Cheers,
>
> Andy
>
>
>
>
>
> On Thu, Apr 27, 2017 at 2:54 PM, James N Guichard <
> james.n.guichard@huawei.com> wrote:
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
> Dear WG:
>
>
>
>
>
>
>
> Having reviewed all of the email discussion on the mailing list it appears
> to the chairs that we have consensus to add a TTL field to the NSH base
> header. We would like to propose the following changes:
>
>
>
>
>
>
>
> Section 3.2:
>
>
>
> Update figure 2 as follows:
>
>
>
>
>
>
>
>      0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1
>
>
>
>     +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
>
>
>
>     |Ver|O|R|    TTL    |   Length  |R|R|R|R|MD Type| Next Protocol |
>
>
>
>     +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
>
>
>
>
>
>
>
> Add the following text after figure 2:
>
>
>
>
>
>
>
> TTL: Service plane time-to-live. An SFF MUST decrement the TTL by a value
> of 1 for all NSH packets it receives. Decrementing by a value of 1 from 0
> shall result in a TTL value of 63. The default for originating an NSH
> packet is a TTL value
>
> of 63. The decrement SHALL occur before testing for 0. After decrement, if
> the TTL is 0, the NSH packet MUST NOT be forwarded.
>
>
>
>
>
>
>
> Section 3.4:
>
>
>
> Update figure 4 to reflect the new base header format as per section 3.2
> base header.
>
>
>
>
>
>
>
> Section 3.5:
>
>
>
> Update figure 5 to reflect the new base header format as per section 3.2
> base header.
>
>
>
>
>
>
>
> Section 12.2.1:
>
>
>
> Current text is as follows:
>
>
>
>
>
>
>
>    There are ten bits at the beginning of the NSH Base Header.  New bits
>
>
>
>    are assigned via Standards Action [RFC5226].
>
>
>
>
>
>
>
>    Bits 0-1 - Version
>
>
>
>    Bit 2 - OAM (O bit)
>
>
>
>    Bit 3 - Critical TLV (C bit)
>
>
>
>    Bits 4-9 - Reserved
>
>
>
>
>
>
>
> Replace entire text as follows:
>
>
>
>
>
>
>
>    There are eight reserved bits in the NSH Base Header. New bits
>
>
>
>    are assigned via Standards Action [RFC5226].
>
>
>
>
>
>
>
>    Bits 0-1 - Version
>
>
>
>    Bit 2 - OAM (O bit)
>
>
>
>    Bit 3 - Reserved
>
>
>
>    Bits 16-19 - Reserved
>
>
>
>
>
>
>
> Section 12.2.3:
>
>
>
> Current text has the MD-type as 8-bit values.
>
>
>
>
>
>
>
> Update text for this section and table 1 to reflect 4-bit values *not*
> 8-bit values.
>
>
>
>
>
>
>
> *Please review carefully and indicate support for these changes (or any
> changes to the suggested text).*
>
>
>
>
>
>
>
> Thanks,
>
>
>
>
>
>
>
> Jim & Joel
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
> _______________________________________________
>
>
> sfc mailing list
>
>
> sfc@ietf.org
>
>
> https://www.ietf.org/mailman/listinfo/sfc
>
>
>
>
>
>
> _______________________________________________
>
> sfc mailing list
>
> sfc@ietf.org
>
> https://www.ietf.org/mailman/listinfo/sfc
>
>
>