iwlwifi: don't mess up QoS counters with non-QoS frames
authorEmmanuel Grumbach <emmanuel.grumbach@intel.com>
Wed, 1 Feb 2012 15:01:32 +0000 (07:01 -0800)
committerJohn W. Linville <linville@tuxdriver.com>
Wed, 1 Feb 2012 20:26:00 +0000 (15:26 -0500)
commit3d29dd9b5b160ba4542a9b8f869a220559e633a0
tree300ef96f8681c3b853686b11b04062f3045860f2
parenta6c84622b7fa3ea5417a9d1d0ce0bc3e7fbe3be1
iwlwifi: don't mess up QoS counters with non-QoS frames

In my AMPDU rework, I rely on the sequence numbers of frames. But
I didn't check that the frame has a valid tid before updating the
tracking counters. As a result, the Tx queues were stalled. People
who hit this bug saw that we simply didn't let any data out.

This bug was introduced in 3.3.

This patch fixes that and checks that the frame is a QoS frame before
looking at its tid and changing the counters.

Signed-off-by: Emmanuel Grumbach <emmanuel.grumbach@intel.com>
Signed-off-by: Wey-Yi Guy <wey-yi.w.guy@intel.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/iwlwifi/iwl-agn-tx.c
drivers/net/wireless/iwlwifi/iwl-commands.h
drivers/net/wireless/iwlwifi/iwl-trans-pcie.c