Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1762913AbXJMSSv (ORCPT ); Sat, 13 Oct 2007 14:18:51 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1762609AbXJMSSS (ORCPT ); Sat, 13 Oct 2007 14:18:18 -0400 Received: from 1wt.eu ([62.212.114.60]:3101 "EHLO 1wt.eu" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1762588AbXJMSSR (ORCPT ); Sat, 13 Oct 2007 14:18:17 -0400 Date: Sat, 13 Oct 2007 20:10:45 +0200 From: Willy Tarreau To: Adrian Bunk Cc: Ilpo =?iso-8859-1?Q?J=E4rvinen?= , LKML , stable@kernel.org, "David S. Miller" , Greg Kroah-Hartman Subject: Re: [2.6.20.21 review 12/35] TCP: Fix TCP handling of SACK in bidirectional flows. Message-ID: <20071013181045.GB18155@1wt.eu> References: <20071013142822.%N@1wt.eu> <20071013143452.%N@1wt.eu> <20071013172214.GA18155@1wt.eu> <20071013175036.GD4211@stusta.de> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20071013175036.GD4211@stusta.de> User-Agent: Mutt/1.5.11 Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1567 Lines: 39 Hi Adrian, On Sat, Oct 13, 2007 at 07:50:36PM +0200, Adrian Bunk wrote: > On Sat, Oct 13, 2007 at 07:22:14PM +0200, Willy Tarreau wrote: > >... > > Thanks for your help, I really appreciate it. In fact, I've reviewed them > > four, but two of them did not apply and the code looked somewhat different, > > so I considered them irrelevant to 2.6.20. I didn't understand that they > > were all related, so maybe I checked them in a wrong order. > > > > I'll recheck all that in the right sequence and will merge them four, or > > get back to you if something still puzzles me. > > I discussed this issue with Ilpo just yesterday regarding 2.6.16, and > the result of our discussion was that I reverted it. OK. > TCP being in some situations a bit more conservative than it should be > isn't a big issue and not worth backporting with a risk of introducing > a regression. I agree with this. The impression I got from the description of the two patches I merged was that the problems they fix were quite annoying. But maybe I should take that with a grain of salt. > I'd recommend you simply drop the two patches for 2.6.20. That sounds OK to me. If 2.6.16 is fine without the patches, 2.6.20 certainly is, particularly if we keep in mind that it's a last version. Thanks very much for your insights Adrian, Willy - 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/