On 01/29/16 at 01:58P, hiren panchasara wrote: > On 01/29/16 at 01:51P, hiren panchasara wrote: > > On 01/29/16 at 01:38P, Yongmin Cho wrote: > > > Hi, all. > > > > > > I have an opinion about net.inet.tcp.initcwnd_segments. > > > You know, snd_cwnd is restarted transmission after a long idle > > > period(Current RTO). > > > And, All of congestion control algorithm is using newreno_after_idle > > > function after a long idle period. > > > But, The newreno_after_idle function is not using initcwnd_segments. > > > I think, The initcwnd_segments should be used in newreno_after_idle > > > function, If the newreno_after_idle is called. > > > I referred to rfc6928. > > > > > > Please check my opinion. > > > > You are absolutely right. We (FreeBSD) adopted initcwnd to be 10 > > segments but never bothered to update newreno_after_idle() function to > > reflect that in calculating cwnd after idle period. Though the comments > > in that function clearly says: > > "The restart window is the initial window or the last CWND, whichever is > > smaller." > > > > I think we should make following change to accommodate it: [snip] https://reviews.freebsd.org/D5124
Cheers, Hiren
pgpGsZE8uAthJ.pgp
Description: PGP signature
