Skip to contentSkip to conversation

pull request: iwlwifi 2014-02-23

PATCH New Emmanuel Grumbach 2 participants 3 messages Feb 23, 2014 – Feb 24, 2014 Networking drivers (wireless)

lore
Share & tools
lore
#0 lore raw

Hi John,

One single patch for this pull request intended for 3.14.
This should fix an issue raised in iwldvm when we have lots of association failures. There is a bugzilla for this bug - it hasn't been validated by the user, but I hope it will do the trick.

I hope I fixed my line wrapping issue this time. Let me know if you are still unhappy with the current format.

Thanks!
The following changes since commit 205e2210daa975d92ace485a65a31ccc4077fe1a:

iwlwifi: disable TX AMPDU by default for iwldvm (2014-02-13 15:24:35 +0200)

are available in the git repository at:

git://git.kernel.org/pub/scm/linux/kernel/git/iwlwifi/iwlwifi-fixes.git master

for you to fetch changes up to ec6f678c74dbdb06a6a775bbb00f1d26c17c404b:

iwlwifi: dvm: clear IWL_STA_UCODE_INPROGRESS when assoc fails (2014-02-20 19:16:27 +0200)

----------------------------------------------------------------
Emmanuel Grumbach (1):
iwlwifi: dvm: clear IWL_STA_UCODE_INPROGRESS when assoc fails

 drivers/net/wireless/iwlwifi/dvm/sta.c |    1 +
 1 file changed, 1 insertion(+)
#1 lore raw

PATCH 1/1 iwlwifi: dvm: clear IWL_STA_UCODE_INPROGRESS when assoc fails

From: Emmanuel Grumbach <[email protected]>

We set IWL_STA_UCODE_INPROGRESS flag when we add a station and clear it when we send the LQ command for it. But the LQ command is sent only when the association succeeds. If the association doesn't succeed, we would leave this flag set and that wouldn't indicate the station entry as vacant.

This probably fixes:
https://bugzilla.redhat.com/show_bug.cgi?id=1065663

---
 drivers/net/wireless/iwlwifi/dvm/sta.c |    1 +
 1 file changed, 1 insertion(+)
drivers/net/wireless/iwlwifi/dvm/sta.c +1 -0
index c0d070c..9cdd91c 100644
@@ -590,6 +590,7 @@ void iwl_deactivate_station(struct iwl_priv *priv, const u8 sta_id,
 			sizeof(priv->tid_data[sta_id][tid]));
 
 	priv->stations[sta_id].used &= ~IWL_STA_DRIVER_ACTIVE;
+	priv->stations[sta_id].used &= ~IWL_STA_UCODE_INPROGRESS;
 
 	priv->num_stations--;
 
Signature · 1 line
1.7.9.5
#2 lore raw
In reply to Emmanuel Grumbach
Quoted 13 lines · Emmanuel Grumbach

Hi John,

One single patch for this pull request intended for 3.14.
This should fix an issue raised in iwldvm when we have lots of association failures. There is a bugzilla for this bug - it hasn't been validated by the user, but I hope it will do the trick.

I hope I fixed my line wrapping issue this time. Let me know if you are still unhappy with the current format.

Thanks!
The following changes since commit 205e2210daa975d92ace485a65a31ccc4077fe1a:

iwlwifi: disable TX AMPDU by default for iwldvm (2014-02-13 15:24:35 +0200)

Quoted 7 lines

are available in the git repository at:

git://git.kernel.org/pub/scm/linux/kernel/git/iwlwifi/iwlwifi-fixes.git master

for you to fetch changes up to ec6f678c74dbdb06a6a775bbb00f1d26c17c404b:

iwlwifi: dvm: clear IWL_STA_UCODE_INPROGRESS when assoc fails (2014-02-20 19:16:27 +0200)

Pulling now...

Signature · 2 lines
John W. Linville		Someday the world will need a hero, and you
[email protected]			might be all we have.  Be ready.