Return-path: Received: from he.sipsolutions.net ([78.46.109.217]:54721 "EHLO sipsolutions.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756898Ab1K2WgC (ORCPT ); Tue, 29 Nov 2011 17:36:02 -0500 Subject: Re: [PATCH] mac80211: reset addba retries after timeout From: Johannes Berg To: Nikolay Martynov Cc: linville@tuxdriver.com, linux-wireless@vger.kernel.org In-Reply-To: (sfid-20111129_232917_739824_BB56AB1A) References: <1322533625-24641-1-git-send-email-mar.kolya@gmail.com> <1322555084.4110.2.camel@jlt3.sipsolutions.net> <1322579973.4110.22.camel@jlt3.sipsolutions.net> <1322604877.11282.1.camel@jlt3.sipsolutions.net> (sfid-20111129_232917_739824_BB56AB1A) Content-Type: text/plain; charset="UTF-8" Date: Tue, 29 Nov 2011 23:35:59 +0100 Message-ID: <1322606159.11282.2.camel@jlt3.sipsolutions.net> (sfid-20111129_233607_526916_75337404) Mime-Version: 1.0 Sender: linux-wireless-owner@vger.kernel.org List-ID: On Tue, 2011-11-29 at 17:28 -0500, Nikolay Martynov wrote: > Hi. > > 2011/11/29 Johannes Berg : > > On Tue, 2011-11-29 at 17:01 -0500, Nikolay Martynov wrote: > > > >> >> Do APs broken is such way exist? I.e. APs which declare aggregation > >> >> support but do not respond to addba. > >> > > >> > I seem to remember something ... not really sure. > >> > >> If such APs exist I think it might be possible to extend my patch to > >> do something like the following. For the first time make 10 attempts > >> to establish addba. If this fails - make no more attempts for the > >> duration of the connection. If this succeeds - use logic I've added in > >> the patch. This should handle broken APs and won't allow agg to be > >> disable because of some random blackout. > > > > Wouldn't that be equivalent to just bumping the number of tries? > > Well, sort of if you are willing to bump it to 10-15, I guess it > would be kind of same thing. > I've seen couple of times when 3 wasn't enough and agg got disabled, > but 10-15 seems reasonably large to give up completely. Maybe the better approach would be to bump it up, but also only allow an attempt once every say 15 seconds or so? It seems kinda useless to try 3 (or 10/15) times in quick succession and then give up, vs. just trying more spaced out? johannes