Page MenuHomeFreeBSD
Differential D33042

if_stf: document 6rd in the man page
ClosedPublic
Actions

Authored by kp on Nov 17 2021, 5:05 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Jun 27, 9:02 AM
Unknown Object (File)
Tue, Jun 16, 10:56 PM
Unknown Object (File)
Sun, Jun 14, 6:18 PM
Unknown Object (File)
Sat, Jun 13, 9:07 PM
Unknown Object (File)
May 26 2026, 5:49 AM
Unknown Object (File)
May 6 2026, 12:26 PM
Unknown Object (File)
May 6 2026, 8:18 AM
Unknown Object (File)
May 6 2026, 7:29 AM
View All Files
Subscribers
emaste
imp
pauamma_chezdork.com

Details

Reviewers
None
Group Reviewers
network
pfsense
manpages
Commits
rG3f7b9525ea48: if_stf: document 6rd in the man page
Summary

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

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kp created this revision.Nov 17 2021, 5:05 PM
Herald added a reviewer: manpages. · View Herald TranscriptNov 17 2021, 5:05 PM
Herald added a subscriber: imp. · View Herald Transcript
kp requested review of this revision.Nov 17 2021, 5:05 PM
Harbormaster completed remote builds in B42845: Diff 98651.Nov 17 2021, 5:05 PM
kp updated this revision to Diff 98652.Nov 17 2021, 5:05 PM
kp added a parent revision: D33041: net tests: fix if_stf:6to4.

Fix typo

Harbormaster completed remote builds in B42846: Diff 98652.Nov 17 2021, 5:06 PM
pauamma_chezdork.com added a subscriber: pauamma_chezdork.com.Nov 17 2021, 6:52 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..

kp added a comment.Nov 17 2021, 10:10 PM
In D33042#746144, @pauamma_gundo.com wrote:

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.

kp added a child revision: D33046: if_stf: minor man page improvements.Nov 17 2021, 10:22 PM
emaste added a subscriber: emaste.Nov 18 2021, 2:29 AM
emaste added inline comments.
share/man/man4/stf.4
64

Is there an expansion for CE?

kp updated this revision to Diff 98699.Nov 18 2021, 4:33 PM

Expand CE

Harbormaster completed remote builds in B42869: Diff 98699.Nov 18 2021, 4:33 PM
kp mentioned this in D33037: if_stf: add 6rd support.Nov 19 2021, 6:19 PM
This revision was not accepted when it landed; it landed in state Needs Review.Nov 20 2021, 7:05 PM
Closed by commit rG3f7b9525ea48: if_stf: document 6rd in the man page (authored by kp). · Explain Why
This revision was automatically updated to reflect the committed changes.
kp added a commit: rG3f7b9525ea48: if_stf: document 6rd in the man page.

Revision Contents
Changeset List

PathSize
share/
man/
man4/
66 lines

Diff 98784

View Options

share/man/man4/stf.4

Loading...