Date: Tue, 9 Jun 2009 13:30:59 -0700 From: Kip Macy <kmacy@freebsd.org> To: "Bjoern A. Zeeb" <bz@freebsd.org> Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r191255 - in head/sys: amd64/conf conf i386/conf net Message-ID: <3c1674c90906091330g3c1ed56es62223d8f8657b8f1@mail.gmail.com> In-Reply-To: <20090609143729.R22887@maildrop.int.zabbadoz.net> References: <200904190016.n3J0G4rA090911@svn.freebsd.org> <20090609143729.R22887@maildrop.int.zabbadoz.net>
next in thread | previous in thread | raw e-mail | index | archive | help
its opt-out now On Tue, Jun 9, 2009 at 8:01 AM, Bjoern A. Zeeb<bz@freebsd.org> wrote: > On Sun, 19 Apr 2009, Kip Macy wrote: > >> Author: kmacy >> Date: Sun Apr 19 00:16:04 2009 >> New Revision: 191255 >> URL: http://svn.freebsd.org/changeset/base/191255 >> >> Log: >> =A0- Import infrastructure for caching flows as a means of accelerating = L3 >> and L2 lookups >> =A0 as well as providing stateful load balancing when used with RADIX_MP= ATH. >> =A0- Currently compiled in to i386 and amd64 but disabled by default, it= can >> be enabled at >> =A0 runtime with 'sysctl net.inet.flowtable.enable=3D1'. >> >> =A0- Embedded users can remove it entirely from the kernel by adding >> 'nooption FLOWTABLE' to >> =A0 their kernel config files. >> >> =A0- A minimal hookup will be added to ip_output in a subsequent commit.= I >> would like to see >> =A0 more review before bringing in changes that require more churn. >> >> =A0Supported by: Bitgravity Inc. >> >> Added: >> =A0head/sys/net/flowtable.c =A0 (contents, props changed) >> =A0head/sys/net/flowtable.h =A0 (contents, props changed) >> Modified: >> =A0head/sys/amd64/conf/DEFAULTS > > =A0... >> >> =A0head/sys/i386/conf/DEFAULTS > > =A0... >> >> Modified: head/sys/amd64/conf/DEFAULTS >> >> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D >> --- head/sys/amd64/conf/DEFAULTS =A0 =A0 =A0 =A0Sat Apr 18 22:16:46 2009 >> =A0(r191254) >> +++ head/sys/amd64/conf/DEFAULTS =A0 =A0 =A0 =A0Sun Apr 19 00:16:04 2009 >> =A0(r191255) >> @@ -20,3 +20,5 @@ options =A0 =A0 =A0 GEOM_PART_BSD >> options =A0 =A0 =A0 =A0 GEOM_PART_EBR >> options =A0 =A0 =A0 =A0 GEOM_PART_EBR_COMPAT >> options =A0 =A0 =A0 =A0 GEOM_PART_MBR >> + >> +options =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0FLOWTABLE >> >> Modified: head/sys/i386/conf/DEFAULTS >> >> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D >> --- head/sys/i386/conf/DEFAULTS Sat Apr 18 22:16:46 2009 =A0 =A0 =A0 =A0= (r191254) >> +++ head/sys/i386/conf/DEFAULTS Sun Apr 19 00:16:04 2009 =A0 =A0 =A0 =A0= (r191255) >> @@ -28,3 +28,5 @@ options =A0 =A0 =A0 GEOM_PART_MBR >> # enable support for native hardware >> options =A0 =A0 =A0 =A0 NATIVE >> device =A0 =A0 =A0 =A0 =A0atpic >> + >> +options =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0FLOWTABLE > > ... > > > > I think FLOWTABLE does nto belong into DEFAULTS. =A0Really DEFAULTS was > meant for "You cannot boot without this" and if people start to weaken > it, DEFAULTS will soon be the new GENERIC. =A0That said I am not sure it > belongs to GENERIC either. > > -- > Bjoern A. Zeeb =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0The greatest ri= sk is not taking one. > --=20 When bad men combine, the good must associate; else they will fall one by one, an unpitied sacrifice in a contemptible struggle. Edmund Burke
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?3c1674c90906091330g3c1ed56es62223d8f8657b8f1>