On 24 February 2012 12:23, Doug Barton <do...@freebsd.org> wrote: > First, I'm always glad to see devs updating the documentation, so please > don't take this as in any way being critical, just trying to help. :)
n/p, I like feedback. > On 02/23/2012 13:56, Andrew Thompson wrote: >> Modified: head/share/man/man4/lagg.4 >> ============================================================================== >> --- head/share/man/man4/lagg.4 Thu Feb 23 21:54:04 2012 >> (r232079) >> +++ head/share/man/man4/lagg.4 Thu Feb 23 21:56:53 2012 >> (r232080) >> @@ -142,10 +142,12 @@ and >> modes will use the RSS hash from the network card if available to avoid >> computing one, this may give poor traffic distribution if the hash is >> invalid >> or uses less of the protocol header information. >> -Local hash computation can be forced by setting the >> +Local hash computation can be forced per interface by setting the > > This is a great example of why we use short lines in the man page > sources. It allows people to come along later and add text without > having to make changes to multiple lines. I can see your point here. >> .Va net.link.lagg.X.use_flowid >> .Xr sysctl 8 >> -variable to zero where X is the interface number. >> +variable to zero where X is the interface number, the default for >> +new interfaces is set via >> +.Va net.link.lagg.default_use_flowid . > > If adding a comma there was the right change the new clause should still > have started on the next line, for the reason described above. However, > what you added is actually a sentence that can/should stand on its own. Fixed. Andrew _______________________________________________ svn-src-head@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/svn-src-head To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"