iwlwifi: mvm: don't override the rate with the AMSDU len
authorEmmanuel Grumbach <emmanuel.grumbach@intel.com>
Tue, 3 May 2016 09:08:43 +0000 (12:08 +0300)
committerEmmanuel Grumbach <emmanuel.grumbach@intel.com>
Wed, 4 May 2016 17:59:55 +0000 (20:59 +0300)
commit5c08b0f5026fcc13efb947c4d1f2ca3558145f68
tree17820fa70707a2078dc4605c8f075e9fbb2fe137
parentf742aaf36edf0390c54d0614bc4d20fd4cd3762a
iwlwifi: mvm: don't override the rate with the AMSDU len

The TSO code creates A-MSDUs from a single large send. Each
A-MSDU is an skb and skb->len doesn't include the number of
bytes which need to be added for the headers being added
(subframe header, TCP header, IP header, SNAP, padding).

To be able to set the right value in the Tx command, we
put the number of bytes added by those headers in
driver_data in iwl_mvm_tx_tso and use this value in
iwl_mvm_set_tx_cmd.

The problem by setting this value in driver_data is that
it overrides the ieee80211_tx_info. The bug manifested
itself when we send P2P related frames in CCK since the
rate in ieee80211_tx_info is zero-ed. This of course is
a violation of the P2P specification.

To fix this, copy the original ieee80211_tx_info to the
stack and pass it to the functions which need it.
Assign the number of bytes added by the headers to the
driver_data inside the skb itself.

Fixes: a6d5e32f247c ("iwlwifi: mvm: send large SKBs to the transport")
Signed-off-by: Emmanuel Grumbach <emmanuel.grumbach@intel.com>
drivers/net/wireless/intel/iwlwifi/mvm/tx.c