bce(4) man page updates

Sean Bruno seanbru at yahoo-inc.com
Thu May 31 17:53:46 UTC 2012


On Thu, 2012-05-31 at 10:15 -0700, Sean Bruno wrote:
> On Thu, 2012-05-31 at 05:46 -0700, Warren Block wrote:
> > On Wed, 30 May 2012, Sean Bruno wrote:
> > 
> > > Was trolling around inside of bce(4) and the Broadcom docs today and
> > > made the following update to the man page.  Thoughts?
> > >
> > > Sean
> > >
> > > http://people.freebsd.org/~sbruno/bce_man.txt
> > 
> > Nice!
> > 
> > Just minor suggestions.
> > 
> > The two "whether or not" sentences ought to be just "Enable/Disable ..." 
> > like the rest.
> > 
> > A couple of "Cannot be assigned the value 0" sentences can be simpler as 
> > "Cannot be set to 0" if that doesn't change the meaning.
> > 
> > "Valid values are in the range from 0-256." can be simpler as
> > "Values from 0-256 are valid."
> > 
> > Many "The default value is" which could be simplified to "The default 
> > is", or added to the end of a previous sentence as an aside:
> > "Values from 0-100 are valid (default 18)."
> 
> 
> Indeed.  Much more gooder wordingness.  Updated!
> 
> http://people.freebsd.org/~sbruno/bce_man.txt
> 
> Sean
> 

Updated again, looks like one of the tuneables has changed in -head:

hw.bce.loose_rx_mtu --> hw.bce.strict_rx_mtu

Sean




More information about the freebsd-net mailing list