* [ath9k-devel] About A-MPDU aggregation. ATH_AMPDU_SUBFRAME_DEFAULT flag.
@ 2010-04-05 23:11 Rakesh Kumar
2010-04-05 23:40 ` Felix Fietkau
0 siblings, 1 reply; 3+ messages in thread
From: Rakesh Kumar @ 2010-04-05 23:11 UTC (permalink / raw)
To: ath9k-devel
Hi,
I notice this parameter ATH_AMPDU_SUBFRAME_DEFAULT and in the code in
xmit.c, function ath_tx_form_aggr limits the number of sub-frames that can
be included in a A-MPDU to half the total size. The accompanying comments
say:
/* do not exceed subframe limit */
But why halve the baw_size for that? I looked around the standard documents,
couldn't find any reference to halving the station-specfic baw_size
(negotiated during association) to half.
--
Rakesh
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.ath9k.org/pipermail/ath9k-devel/attachments/20100405/62ea2339/attachment.htm
^ permalink raw reply [flat|nested] 3+ messages in thread* [ath9k-devel] About A-MPDU aggregation. ATH_AMPDU_SUBFRAME_DEFAULT flag.
2010-04-05 23:11 [ath9k-devel] About A-MPDU aggregation. ATH_AMPDU_SUBFRAME_DEFAULT flag Rakesh Kumar
@ 2010-04-05 23:40 ` Felix Fietkau
2010-04-06 3:10 ` Rakesh Kumar
0 siblings, 1 reply; 3+ messages in thread
From: Felix Fietkau @ 2010-04-05 23:40 UTC (permalink / raw)
To: ath9k-devel
On 2010-04-05 4:11 PM, Rakesh Kumar wrote:
> Hi,
>
> I notice this parameter ATH_AMPDU_SUBFRAME_DEFAULT and in the code in
> xmit.c, function ath_tx_form_aggr limits the number of sub-frames that
> can be included in a A-MPDU to half the total size. The accompanying
> comments say:
>
> /* do not exceed subframe limit */
>
> But why halve the baw_size for that? I looked around the standard
> documents, couldn't find any reference to halving the station-specfic
> baw_size (negotiated during association) to half.
Maybe because ath9k tries to keep two A-MPDU aggregates in the xmit
queue and has to handle retransmission of failed subframes in software.
IMHO it makes sense to use half of the BAW for each of the queue entries
under those conditions.
- Felix
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2010-04-06 3:10 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-04-05 23:11 [ath9k-devel] About A-MPDU aggregation. ATH_AMPDU_SUBFRAME_DEFAULT flag Rakesh Kumar
2010-04-05 23:40 ` Felix Fietkau
2010-04-06 3:10 ` Rakesh Kumar
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.