Re: Possible BUG in IPv4 TCP window handling, all recent 2.4.x/2.6.x kernels

[Guillaume Autran <gautran@mrv.com>]

Hi!

Sorry Alexey for keeping it quiet but I got pulled away to some other duties for 
the past 3 weeks.

Anyway, the similar problem I was reporting has not been seen since that last 
incident a month ago. We did change, on our application side, some of the 
parameters (aka SO_RCVBUF) that did not need to be set in the first place (bug 
on our side).

This plus your patch seem to have resolve the issues we were having. So, it's 
all good !

Thanks again..
Guillaume.


Alexey Kuznetsov wrote:
> Hello!
>
>
> > > Anyway, ignoring this puzzle, the following patch for 2.4 should help.
> > >
> > >
> > > --- net/ipv4/tcp_input.c.orig	2003-02-20 20:38:39.000000000 +0300
> > > +++ net/ipv4/tcp_input.c	2005-09-02 22:28:00.845952888 +0400
> > > @@ -343,8 +343,6 @@
> > > 			app_win -= tp->ack.rcv_mss;
> > > 		app_win = max(app_win, 2U*tp->advmss);
> > >
> > > -		if (!ofo_win)
> > > -			tp->window_clamp = min(tp->window_clamp, app_win);
> > > 		tp->rcv_ssthresh = min(tp->window_clamp, 2U*tp->advmss);
> > > 	}
> > > }
> >
> > I'm very happy to report that the above patch, applied to 2.6.12.6, seems 
> > to have cured the TCP window problem we were experiencing.
>
>
> Good. I think the patch is to be applied to all mainstream kernels.
>
> The only obstacle is the second report by Guillaume Autran <gautran@mrv.com>,
> which has some allied characteristics, but after analysis it is something
> impossible, read, cryptic and severe bug. :-( I did not get a responce
> to the last query, so the investigation stalled.
>
> Alexey

--
=======================================
Guillaume Autran
Senior Software Engineer
MRV Communications, Inc.
Tel: (978) 952-4932 office
=======================================

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/