Return-path: Received: from bear.ext.ti.com ([192.94.94.41]:52131 "EHLO bear.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756198Ab1ALNEa (ORCPT ); Wed, 12 Jan 2011 08:04:30 -0500 Subject: Re: [PATCH 1/2] mac80211: add hw configuration for max ampdu buffer size From: Luciano Coelho To: Johannes Berg CC: "linux-wireless@vger.kernel.org" , "juuso.oikarinen@nokia.com" In-Reply-To: <1294836406.3639.25.camel@jlt3.sipsolutions.net> References: <1294836052-16828-1-git-send-email-coelho@ti.com> <1294836406.3639.25.camel@jlt3.sipsolutions.net> Content-Type: text/plain; charset="UTF-8" Date: Wed, 12 Jan 2011 15:04:18 +0200 Message-ID: <1294837458.2097.33.camel@pimenta> MIME-Version: 1.0 Sender: linux-wireless-owner@vger.kernel.org List-ID: On Wed, 2011-01-12 at 13:46 +0100, Johannes Berg wrote: > On Wed, 2011-01-12 at 14:40 +0200, coelho@ti.com wrote: > > From: Luciano Coelho > > > > Some devices don't support the maximum AMDPU buffer size of 64, so we > > need to add an option to configure this in the hardware configuration. > > This value will be used in the ADDBA response instead of the value > > suggested in the request, if the latter is greater than the max > > supported. > > > > Signed-off-by: Luciano Coelho > > --- > > include/net/mac80211.h | 4 ++++ > > net/mac80211/agg-rx.c | 3 +++ > > net/mac80211/main.c | 1 + > > 3 files changed, 8 insertions(+), 0 deletions(-) > > > > diff --git a/include/net/mac80211.h b/include/net/mac80211.h > > index 5b3fd5a..1341d82 100644 > > --- a/include/net/mac80211.h > > +++ b/include/net/mac80211.h > > @@ -1143,6 +1143,9 @@ enum ieee80211_hw_flags { > > * @napi_weight: weight used for NAPI polling. You must specify an > > * appropriate value here if a napi_poll operation is provided > > * by your driver. > > + * @max_rx_aggregation_subframes: maximum buffer size (number of > > + * sub-frames) to be used for A-MPDU block ack receiver > > + * aggregation. > > ... "This is only relevant if the device has restrictions on the number > of subframes, if it relies on mac80211 to do reordering it shouldn't be > set." maybe? Yes, this makes it clearer. Thanks! I'll add it. -- Cheers, Luca.