netdev
[Top] [All Lists]

Re: [TCP] Fixed mss in tcp_init_cwnd

To: "David S. Miller" <davem@xxxxxxxxxxxxx>
Subject: Re: [TCP] Fixed mss in tcp_init_cwnd
From: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx>
Date: Tue, 28 Sep 2004 09:09:38 +1000
Cc: netdev@xxxxxxxxxxx
In-reply-to: <20040927160454.67cd1e11.davem@davemloft.net>
References: <20040927080828.GA12056@gondor.apana.org.au> <20040927120031.55fb4a49.davem@davemloft.net> <20040927220240.GA7633@gondor.apana.org.au> <20040927160454.67cd1e11.davem@davemloft.net>
Sender: netdev-bounce@xxxxxxxxxxx
User-agent: Mutt/1.5.6+20040722i
On Mon, Sep 27, 2004 at 04:04:54PM -0700, David S. Miller wrote:
>
> > IMHO we should still fix it though at least for consistency.
> 
> That early on in the connection, it should be setting both
> values, right?  If so your patch still needs a tweak. :)

I'm not saying that this patch makes any difference in terms of
run-time results.  At that point both mss_cache and mss_cache_std
should contain the same values.

However, since the value that's intended here is the physical MSS,
we should use mss_cache_std for the sake of consistency.

The only reason I spotted this at all is because I grepped -w for
mss_cache and found this function as one of the very few users.

Cheers,
-- 
Visit Openswan at http://www.openswan.org/
Email: Herbert Xu ~{PmV>HI~} <herbert@xxxxxxxxxxxxxxxxxxx>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt

<Prev in Thread] Current Thread [Next in Thread>