iwlwifi: audit single frames from AGG queue in RS
authorEmmanuel Grumbach <emmanuel.grumbach@intel.com>
Wed, 9 Jan 2013 08:20:36 +0000 (10:20 +0200)
committerJohannes Berg <johannes.berg@intel.com>
Wed, 16 Jan 2013 13:20:20 +0000 (14:20 +0100)
commit7d3590dfa7109166b9f8d7b17380031e42b6d15d
tree11dd58c711361b7b1503e3446f49bc146e383eca
parent811665bdbf436aba2bf8b5271b9d8a3fb3bf35e8
iwlwifi: audit single frames from AGG queue in RS

The rate scaling won't treat the information in a frame
with IEEE80211_TX_CTL_AMPDU set if IEEE80211_TX_STAT_AMPDU
is cleared. But all the frames coming from an AGG tx queue
have IEEE80211_TX_CTL_AMPDU set, and IEEE80211_TX_STAT_AMPDU
is set only if the frame was sent in an AMPDU.
This means that all the data in frames in AGG tx queues that
aren't sent as an AMPDU is thrown away.
This is even more harmful when in bad link conditions, the
frames are sent in an AMPDU and then finally sent as single
frame. So a lot of failures weren't reported and the rate
scaling got stuck in high rates leading to very poor
connectivity.

Fix that by clearing IEEE80211_TX_CTL_AMPDU when the frame
isn't part of an AMPDU.

This bug was introduced by

39aa4160bea0ffc6e809be8a0a962b3cac2bc544
iwlwifi: don't clear CTL_AMPDU on frame status

This fix basically reverts the aforementioned commit.

Signed-off-by: Emmanuel Grumbach <emmanuel.grumbach@intel.com>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
drivers/net/wireless/iwlwifi/dvm/tx.c