Return-path: Received: from yw-out-2324.google.com ([74.125.46.29]:7598 "EHLO yw-out-2324.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751906AbYJWUSr (ORCPT ); Thu, 23 Oct 2008 16:18:47 -0400 Received: by yw-out-2324.google.com with SMTP id 9so175015ywe.1 for ; Thu, 23 Oct 2008 13:18:45 -0700 (PDT) Message-ID: <43e72e890810231318t2ef200e4wed96b48efefa62bb@mail.gmail.com> (sfid-20081023_221851_979205_757ED539) Date: Thu, 23 Oct 2008 13:18:44 -0700 From: "Luis R. Rodriguez" To: "Tomas Winkler" Subject: Re: [RFC] mac80211: Re-enable aggregation Cc: "Johannes Berg" , Sujith , "linville@tuxdriver.com" , "linux-wireless@vger.kernel.org" , "Luis Rodriguez" In-Reply-To: <1ba2fa240810231131l9c9597au2acf166a13a8c03b@mail.gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 References: <18684.16351.638713.791015@gargle.gargle.HOWL> <1224495531.18024.55.camel@johannes.berg> <18684.24323.743610.871307@gargle.gargle.HOWL> <1224505349.27899.17.camel@johannes.berg> <18684.51206.771543.514682@localhost.localdomain> <1224669612.28639.49.camel@johannes.berg> <18688.17545.136327.991699@gargle.gargle.HOWL> <1224773124.6002.32.camel@johannes.berg> <43e72e890810231023q5650184r8ce50cb51b63e706@mail.gmail.com> <1ba2fa240810231131l9c9597au2acf166a13a8c03b@mail.gmail.com> Sender: linux-wireless-owner@vger.kernel.org List-ID: On Thu, Oct 23, 2008 at 11:31 AM, Tomas Winkler wrote: > On Thu, Oct 23, 2008 at 7:23 PM, Luis R. Rodriguez wrote: >> I checked internally to verify where you would decide when to AMPDU >> and to try to get different reviews and opinions, and it seems that >> the path we take right now is correct as there is not much overhead so >> we always use AMPDU with whoever supports it with data frames. If >> you're a STA you do it all the time with the AP for data frames. >> >> I noticed iwlagn had some more logic within the RC but I gave up >> trying to follow the logic. I suspect they do the same though, Tomas? Thanks for the feedback! > I got different output from our system engineering. APMDU has overhead Is that because of the interactions required in firmware for scheduling BTW? > and if there is not requirement > for high throughput we don't initiate one. This makes sense to me regardless. > What is for sure we shounld > not have aggregation enabled for VO AC. Any particular reason? I'll review this too, but my guess would be the buffering required before TXing. I'll see how we perform with aggr on VO AC. > I've payed attention that there are different APs do that differently > some opens BA session on BE AC immediately upon association some do it > on high throughput I haven't seen in the second case it will be opened > on simple ping. > Anyhow if we would implement the triggering on throughput simply > setting starting and stopping thresholds to small values will have > same effect as your solution and setting it to 0 and maximum will > respectively will have effect on opening BA session upon association. Seems reasonable. If this is set then as a hw configurable option in mac80211 then we can even allow RC control this, should this ever be needed (we don't). Luis