From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from arroyo.ext.ti.com ([192.94.94.40]:41562 "EHLO arroyo.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755840Ab1ALNE2 (ORCPT ); Wed, 12 Jan 2011 08:04:28 -0500 Subject: Re: [PATCH] mac80211: use maximum number of a-msdu frames as default in BA RX From: Luciano Coelho To: Johannes Berg CC: "linux-wireless@vger.kernel.org" , "stable@kernel.org" In-Reply-To: <1294836844.3639.26.camel@jlt3.sipsolutions.net> References: <1294836681-21712-1-git-send-email-coelho@ti.com> <1294836844.3639.26.camel@jlt3.sipsolutions.net> Content-Type: text/plain; charset="UTF-8" Date: Wed, 12 Jan 2011 15:03:46 +0200 Message-ID: <1294837426.2097.32.camel@pimenta> MIME-Version: 1.0 Sender: linux-wireless-owner@vger.kernel.org List-ID: On Wed, 2011-01-12 at 13:54 +0100, Johannes Berg wrote: > On Wed, 2011-01-12 at 14:51 +0200, coelho@ti.com wrote: > > From: Luciano Coelho > > > > When the buffer size is set to zero in the block ack parameter set > > field, we should use the maximum supported number of subframes. The > > existing code was bogus and was doing some unnecessary calculations > > that lead to wrong values. > > > > Thanks Johannes for helping me figure this one out. > > > > Cc: stable@kernel.org > > Cc: Johannes Berg > > Reviewed-by: Johannes Berg > > It'd be easier for stable if you reordered with the other patch, I > think, but I guess it's easy enough to fix up. Will do. > Also some older stable versions might have this code in a different > file, but I'm not sure how to handle that. Dunno how to handle that either... If I need to do something, someone please just let me know. ;) -- Cheers, Luca.