cvs commit: src/sys/netinet tcp_subr.c
ru at FreeBSD.org
Mon Sep 11 23:51:29 PDT 2006
On Tue, Sep 12, 2006 at 02:38:13AM +0400, Yar Tikhiy wrote:
> On Mon, Sep 11, 2006 at 06:28:26PM +0400, Gleb Smirnoff wrote:
> > On Mon, Sep 11, 2006 at 12:36:57AM -0500, Mike Silbersack wrote:
> > M> >It stops creating any state (nor socket neither compressed tcptw) for the
> > M> >TCP connection, where both endpoints were local. This save a lot of
> > M> >resources on servers running HTTP accelerators, or database
> > M> >servers+clients.
> > M>
> > M> I think that you should change the description to say "Do not create TCP
> > M> TIME_WAIT entries for local <-> local connections" (or something similar)
> > M> to remove the ambiguity, and turn it on by default for 7. This seems like
> > M> a good idea, I'm jealous that I didn't think of it. :)
> > OOh, the sysctl name and its description were discussed with ru@, who
> > is our famous nitpicker. If you want to change name or description -
> > mail to him. /me hides
> (It was me who suggested a different name for the sysctl in question.)
> IMHO it's time to define a general style for new sysctl names.
> The first two points could be a) "foo" vs. "nofoo" for boolean
> knobs, and b) underscore usage.
> Presently there seem to be few "nofoo" knobs in the system, e.g.:
> # sysctl -aN | fgrep .no
> kern.nodump_coredump -- should be coredump_nodump
> net.inet6.icmp6.nodeinfo -- false match
> I don't think new ones should be introduced; their logic would be
> unnecessarily vague.
> OTOH, there are enough sysctl names containing an underscore to
> encourage using it in complex names. An exception could be made
> for really short names, such as consisting of not more than 2 words,
> each being not longer than 4 letters.
> What do you think about that, folks?
Fine with me, if *you* want to change that. Also fine with me if
the default is changed to "don't create" (if there's a consensus
on this). We wanted to be on the safe side with this commit, so
the default was to preserve the old behavior for POLA reasons.
ru at FreeBSD.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 187 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/cvs-src/attachments/20060912/489584a3/attachment.pgp
More information about the cvs-src