iwlwifi: WARN only once when we have trouble in reclaim
authorEmmanuel Grumbach <emmanuel.grumbach@intel.com>
Tue, 12 Jun 2012 15:31:21 +0000 (18:31 +0300)
committerJohannes Berg <johannes.berg@intel.com>
Wed, 13 Jun 2012 07:02:09 +0000 (09:02 +0200)
This flow can actually happen due to a corner case in
mac80211: the station is deleted before we get a chance
to reclaim all the packets in flight in AGG queue.
The tid_data for this station is zeroed, and we lose
the match with the Tx queue.

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

index 1f9457fdb933cb1571cc2bef208213d4d274649e..5971a23aa47d1218317460404ef5401f38d43547 100644 (file)
@@ -1183,7 +1183,8 @@ int iwlagn_rx_reply_tx(struct iwl_priv *priv, struct iwl_rx_cmd_buffer *rxb,
                }
 
                /*we can free until ssn % q.n_bd not inclusive */
-               WARN_ON(iwl_reclaim(priv, sta_id, tid, txq_id, ssn, &skbs));
+               WARN_ON_ONCE(iwl_reclaim(priv, sta_id, tid,
+                                        txq_id, ssn, &skbs));
                iwlagn_check_ratid_empty(priv, sta_id, tid);
                freed = 0;