Page MenuHomeFreeBSD

if_stf: document 6rd in the man page
ClosedPublic

Authored by kp on Nov 17 2021, 5:05 PM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 10 2024, 6:02 PM
Unknown Object (File)
Feb 8 2024, 6:32 PM
Unknown Object (File)
Feb 8 2024, 12:10 PM
Unknown Object (File)
Jan 9 2024, 11:50 PM
Unknown Object (File)
Dec 23 2023, 11:07 AM
Unknown Object (File)
Dec 13 2023, 10:08 AM
Unknown Object (File)
Dec 12 2023, 2:23 AM
Unknown Object (File)
Nov 17 2023, 8:55 PM

Details

Summary

PR: 253328
Sponsored by: Rubicon Communications, LLC ("Netgate")

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 42869
Build 39757: arc lint + arc unit

Event Timeline

kp requested review of this revision.Nov 17 2021, 5:05 PM

I can't vouch to the technical accuracy, but other than my 2 inline comments, the wording looks good to me. (Is 2002:8504 an IPv6 tunneling prefix? If not,, maybe uses of that should be changed to use something within the RFC 3849 IPv6 documentation prefix (2001:db8::/32) and the rest of the examples adjusted to use a longer prefix within that.)

share/man/man4/stf.4
60

While you're touching that page: should be "a site border router" or "site border routers".

310

Is it worth keeping that URL still? 6bone was completely phased out 15 years ago..

I can't vouch to the technical accuracy, but other than my 2 inline comments, the wording looks good to me. (Is 2002:8504 an IPv6 tunneling prefix? If not,, maybe uses of that should be changed to use something within the RFC 3849 IPv6 documentation prefix (2001:db8::/32) and the rest of the examples adjusted to use a longer prefix within that.)

2002::/16 is the prefix used for 6to4, so it is appropriate as an example in the 6to4 case. 6rd can use any IPv6 prefix, so the documentation prefix is appropriate there.

share/man/man4/stf.4
60

I'll post a separate review for this remark and the next one. I'd like to keep this review/commit for the 6rd changes only.

310

It's probably time to remove this yes. Like the other remark I'll do this in a separate commit.

emaste added inline comments.
share/man/man4/stf.4
64

Is there an expansion for CE?

This revision was not accepted when it landed; it landed in state Needs Review.Nov 20 2021, 7:05 PM
This revision was automatically updated to reflect the committed changes.