[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: transmech editorial comments
Thanks for the careful read. I've applied all of them.
One question below.
> - The exit node of the tunnel (the decapsulator) receives the
> encapsulated packet, reassembles the packet if needed, removes
> the IPv4 header, updates the IPv6 header, and processes the
> received IPv6 packet.
>
> ==> why would the exit node update the IPv6 header? I assume that would be
> done for Hop Limit processing, but that's already covered by the further
> processing
Correct. fixed.
> ==> this seems a bit unclear forwarding for the case where the tunnel is not
> used for forwarding when the hop limit is not decremented. I tried to think
> of a better rewording, but perhaps the easiest fix would be to remove:
>
> That is, the
> IPv6 hop limit is decremented by 1 when an IPv6 packet traverses the
> tunnel.
How about replacing it with?
That is, forwarding
packets into a tunnel or packets received on a tunnel has the same
impact on the IPv6 hop limit as for other interfaces.
> - Determine when to fragment and when to report an ICMP "packet
> too big" error back to the source.
>
> ==> s/ICMP/ICMPv6/ or IPv6 ICMP
I also found text in several places with "IPvX ICMP" which I replaced
with "ICMPvX".
> Implementations MAY provide a mechanism, for example IP Tunnel MIB
> [RFC2667], to allow the administrator to configure the IPv4 TTL.
>
> [RFC3168] for issues relating to the ToS byte and
>
> ==> s/ToS/Type of Service/ (I wonder if we should call the field DSCP...)
DSCP is 6 bits out of the byte AFAIK, so keeping Type-of-service makes sense
to me.
> Any IPv6 options are preserved in the packet (after the IPv6 header).
>
> ==> this seems like an irrelevant sentence, could be removed?
Sure. (Especially since IPv6 doesn't have options after the base header - it
might have extension headers which contain options ...)
Erik