Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 15 Sep 2010 21:51:02 +1000
From:      Lawrence Stewart <lstewart@freebsd.org>
To:        Andre Oppermann <andre@FreeBSD.org>
Cc:        svn-src-head@FreeBSD.org, svn-src-all@FreeBSD.org, src-committers@FreeBSD.org
Subject:   Re: svn commit: r212653 - head/sys/netinet
Message-ID:  <4C90B326.4000208@freebsd.org>
In-Reply-To: <201009151039.o8FAdU4H030416@svn.freebsd.org>
References:  <201009151039.o8FAdU4H030416@svn.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On 09/15/10 20:39, Andre Oppermann wrote:
> Author: andre
> Date: Wed Sep 15 10:39:30 2010
> New Revision: 212653
> URL: http://svn.freebsd.org/changeset/base/212653
> 
> Log:
>   Change the default MSS for IPv4 and IPv6 TCP connections from an
>   artificial power-of-2 rounded number to their real values specified
>   in RFC879 and RFC2460.
>   
>   From the history and existing comments it appears that the rounded
>   numbers were intended to be advantageous for the kernel and mbuf
>   system.  However this hasn't been the case at for at least a long
>   time.  The mbuf clusters used in tcp_output() have enough space
>   to hold the larger real value for the default MSS for both IPv4 and
>   IPv6.  Note that the default MSS is only used when path MTU discovery
>   is disabled.
>   
>   Update and expand related comments.
>   
>   Reviewed by:	lsteward (including some word-smithing)

For the record, I reviewed and fully support the functional changes made
by this patch, but explicitly objected to and offered an alternate for
the proposed comment wording changes.

Andre, given that we had a disagreement about the comment wording, I
would have preferred it if you had noted in your commit log that I had
raised an objection to or at least not reviewed/endorsed the comment
changes.

It's not important enough an issue to spend any more time on, but I'm a
bit upset to see this committed with an acknowledgement to my review and
word-smithing, much of which ended up being ignored (which is fine, but
then don't put my name to it).

Cheers,
Lawrence



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?4C90B326.4000208>