On 2011-02-08 11:06 PM, Björn Smedman wrote: > On Tue, Feb 8, 2011 at 10:26 PM, Felix Fietkau <nbd@xxxxxxxxxxx> wrote: >> This patch fixes a long standing issue of pending packets in the queue being >> sent (and retransmitted many times) to sleeping stations. >> This was made worse by aggregation through driver-internal retransmitting >> of A-MDPU subframes. > > Nice. :) Just one thought, shouldn't there be one more call to > ieee80211_sta_set_tim() somewhere? I'm thinking there might be a race > condition otherwise where the tid queues are empty on sta_notify but > then later some failed/filtered A-MPDU subframes get queued... Good point, I'll fix that... > Also, just curious so feel free to answer or not, but how does the > hardware handle the ps filter thing? Does it have a ps bit for every > dst address? How many different dsts can it keep track of? The hardware has a bitfield of blocked destinations, using the destination index of the descriptor (which is also used as an index for the key cache). The key cache can hold 128 entries. Having a frame time out with excessive retries will cause the hardware to block frames using the same destination index, until a frame is sent with a bit that tells the hardware to clear that flag. Right now, all unencrypted traffic uses the same destination index (0), so unencrypted mode will have more corner cases compared to using WPA2, but I plan on fixing that soon. - Felix -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
