iwlagn: remove powersave warning
authorJohannes Berg <johannes.berg@intel.com>
Tue, 16 Nov 2010 19:51:04 +0000 (11:51 -0800)
committerWey-Yi Guy <wey-yi.w.guy@intel.com>
Thu, 25 Nov 2010 00:58:06 +0000 (16:58 -0800)
Through races, a packet may be enqueued for
transmission to a station while that station
is going to sleep, in which case the warning
here triggers. Instead of warning, check the
condition -- if this packet is not a PS-poll
response then we still enqueue it but it will
be rejected by the device since the station
is marked as asleep.

Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: Wey-Yi Guy <wey-yi.w.guy@intel.com>
drivers/net/wireless/iwlwifi/iwl-agn-tx.c

index 07bbc915529a34ac0945565b30d4126ef2ab559e..e8bd0b311946a7d75e6c6841d291484962f0acd4 100644 (file)
@@ -567,8 +567,8 @@ int iwlagn_tx_skb(struct iwl_priv *priv, struct sk_buff *skb)
        if (sta)
                sta_priv = (void *)sta->drv_priv;
 
-       if (sta_priv && sta_priv->asleep) {
-               WARN_ON(!(info->flags & IEEE80211_TX_CTL_PSPOLL_RESPONSE));
+       if (sta_priv && sta_priv->asleep &&
+           (info->flags & IEEE80211_TX_CTL_PSPOLL_RESPONSE)) {
                /*
                 * This sends an asynchronous command to the device,
                 * but we can rely on it being processed before the