ath10k: Ignore SWBA event for a vif if its marked for no beacon

https://patchwork.kernel.org/project/linux-wireless/patch/1474359727-6854-1-git-send-email-mohammed@qca.qualcomm.com/

Message ID 1474359727-6854-1-git-send-email-mohammed@qca.qualcomm.com
State Not Applicable
Delegated to: Kalle Valo
Headers show
Commit Message
Mohammed Shafi ShajakhanSept. 20, 2016, 8:22 a.m. UTC
From: Mohammed Shafi Shajakhan mohammed@qti.qualcomm.com

Ignore processing further in SWBA event scheduled for a vif, if mac80211
has marked the particular vif for stop beaconing and brought the vdev
down in ‘ath10k_control_beaconing’. This should potentially avoid ath10k
warning/error messages while running continuous wifi down/up with max
number of vaps configured. Found this change during code walk through
and going through other beacon configuration related functions in ath10k

Signed-off-by: Mohammed Shafi Shajakhan mohammed@qti.qualcomm.com

drivers/net/wireless/ath/ath10k/wmi.c | 6 ++++++
1 file changed, 6 insertions(+)
Comments
Kalle ValoSept. 28, 2016, 9:48 a.m. UTC | #1
Mohammed Shafi Shajakhan mohammed@qti.qualcomm.com wrote:

From: Mohammed Shafi Shajakhan mohammed@qti.qualcomm.com

Ignore processing further in SWBA event scheduled for a vif, if mac80211
has marked the particular vif for stop beaconing and brought the vdev
down in ‘ath10k_control_beaconing’. This should potentially avoid ath10k
warning/error messages while running continuous wifi down/up with max
number of vaps configured. Found this change during code walk through
and going through other beacon configuration related functions in ath10k

Signed-off-by: Mohammed Shafi Shajakhan mohammed@qti.qualcomm.com

Patch applied to ath-next branch of ath.git, thanks.

b63b33ecafa5 ath10k: Ignore SWBA event for a vif if its marked for no beacon
Patch
diff --git a/drivers/net/wireless/ath/ath10k/wmi.c b/drivers/net/wireless/ath/ath10k/wmi.c
index eb4ab6f…f37f178 100644
— a/drivers/net/wireless/ath/ath10k/wmi.c
+++ b/drivers/net/wireless/ath/ath10k/wmi.c
@@ -3514,6 +3514,12 @@ void ath10k_wmi_event_host_swba(struct ath10k *ar, struct sk_buff *skb)
continue;
}

•   /* mac80211 would have already asked us to stop beaconing and
  

•    * bring the vdev down, so continue in that case
  

•    */
  

•   if (!arvif->is_up)
  

•   	continue;
  

•   /* There are no completions for beacons so wait for next SWBA
     * before telling mac80211 to decrement CSA counter
     *