• Emmanuel Grumbach's avatar
    iwlwifi: mvm: don't override the rate with the AMSDU len · 5c08b0f5
    Emmanuel Grumbach authored
    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: a6d5e32f ("iwlwifi: mvm: send large SKBs to the transport")
    Signed-off-by: default avatarEmmanuel Grumbach <emmanuel.grumbach@intel.com>
    5c08b0f5
tx.c 43.1 KB