802.1Q vlan performance. [PATCH]

John Baldwin jhb at freebsd.org
Fri Aug 25 14:48:36 UTC 2006


On Friday 25 August 2006 06:11, Gleb Smirnoff wrote:
> On Fri, Aug 25, 2006 at 12:04:25PM +0200, Ian FREISLICH wrote:
> I> > Alternatively is there an objection to making this a kernel
> I> > configuration file option as opposed to leaving it as an undocumented
> I> > buried treasure?
> I> 
> I> I see it is an option, just undocumented.
> I> 
> I> Whitespace mangled:
> I> 
> I> RCS file: /home/ncvs/src/sys/conf/NOTES,v
> I> retrieving revision 1.1378
> I> diff -u -d -r1.1378 NOTES
> I> --- /usr/src/sys/conf/NOTES       17 Aug 2006 00:37:03 -0000      1.1378
> I> +++ /usr/src/sys/conf/NOTES       25 Aug 2006 10:03:18 -0000
> I> @@ -646,6 +646,7 @@
> I>  #
> I>  device         ether                   #Generic Ethernet
> I>  device         vlan                    #VLAN support (needs miibus)
> I> +options                VLAN_ARRAY              #Lookup table for trunk interface
> I>  device         wlan                    #802.11 support
> I>  device         wlan_wep                #802.11 WEP support
> I>  device         wlan_ccmp               #802.11 CCMP support
> 
> I'd prefer not to put it into NOTES. When enabled, less code will
> be covered by LINT.

Bah, GENERIC will cover it then.  All options should be in NOTES.  If
an option removes code and it's enabled in GENERIC, then you should put
it in NOTES but comment it out.

-- 
John Baldwin


More information about the freebsd-current mailing list