> introduced a TX BA session timeout timer. However the
> timer was reset for each outgoing transmission instead
> what 802.11-2007 11.5.3 specifies:
You should probably reference 802.11-2012 now since some things were
renumbered.
> "The inactivity timer at the originator is reset when a
> BlockAck frame corresponding to the TID for which the
> Block Ack policy is set is received."
> This patch was only compile-tested and the design has
> some rather big problems:
Maybe then we should just revert Nikolay's patch?
> - The spec says we should test for BlockAcks, however
> that is not feasible because it is a control frame
> (so FIF_CONTROL might filter it on some hardware).
In some way, all devices are going to have to report these frames. Maybe
not as the frame itself, but as some other notification, to allow
cleaning up the TX queues accordingly, I think? There's a BA
notification in iwlwifi for example.
> +++ b/net/mac80211/status.c
> @@ -439,6 +439,29 @@ void ieee80211_tx_status(struct ieee80211_hw *hw, struct sk_buff *skb)
> }
> }
>
> + if (ieee80211_is_data_qos(fc) && acked &&
> + info->flags & IEEE80211_TX_CTL_AMPDU &&
> + local->hw.flags & IEEE80211_HW_HAS_RATE_CONTROL) {
I'm confused -- what does this have to do with HW_HAS_RATE_CONTROL?
johannes
--
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
