Commit 5a99610c authored by Arik Nemtsov's avatar Arik Nemtsov Committed by Luciano Coelho

wlcore: free AP global links properly on recovery

Dont use free_sta() on AP global links. It would fail an internal check
within the function and various structures within the link struct would
not be reset.
Signed-off-by: default avatarArik Nemtsov <arik@wizery.com>
Signed-off-by: default avatarLuciano Coelho <coelho@ti.com>
parent f9ae0852
...@@ -2523,6 +2523,8 @@ static void __wl1271_op_remove_interface(struct wl1271 *wl, ...@@ -2523,6 +2523,8 @@ static void __wl1271_op_remove_interface(struct wl1271 *wl,
wl1271_ps_elp_sleep(wl); wl1271_ps_elp_sleep(wl);
} }
deinit: deinit:
wl12xx_tx_reset_wlvif(wl, wlvif);
/* clear all hlids (except system_hlid) */ /* clear all hlids (except system_hlid) */
wlvif->dev_hlid = WL12XX_INVALID_LINK_ID; wlvif->dev_hlid = WL12XX_INVALID_LINK_ID;
...@@ -2546,7 +2548,6 @@ static void __wl1271_op_remove_interface(struct wl1271 *wl, ...@@ -2546,7 +2548,6 @@ static void __wl1271_op_remove_interface(struct wl1271 *wl,
dev_kfree_skb(wlvif->probereq); dev_kfree_skb(wlvif->probereq);
wlvif->probereq = NULL; wlvif->probereq = NULL;
wl12xx_tx_reset_wlvif(wl, wlvif);
if (wl->last_wlvif == wlvif) if (wl->last_wlvif == wlvif)
wl->last_wlvif = NULL; wl->last_wlvif = NULL;
list_del(&wlvif->list); list_del(&wlvif->list);
......
...@@ -1047,7 +1047,8 @@ void wl12xx_tx_reset_wlvif(struct wl1271 *wl, struct wl12xx_vif *wlvif) ...@@ -1047,7 +1047,8 @@ void wl12xx_tx_reset_wlvif(struct wl1271 *wl, struct wl12xx_vif *wlvif)
/* TX failure */ /* TX failure */
for_each_set_bit(i, wlvif->links_map, WL12XX_MAX_LINKS) { for_each_set_bit(i, wlvif->links_map, WL12XX_MAX_LINKS) {
if (wlvif->bss_type == BSS_TYPE_AP_BSS) { if (wlvif->bss_type == BSS_TYPE_AP_BSS &&
i != wlvif->ap.bcast_hlid && i != wlvif->ap.global_hlid) {
/* this calls wl12xx_free_link */ /* this calls wl12xx_free_link */
wl1271_free_sta(wl, wlvif, i); wl1271_free_sta(wl, wlvif, i);
} else { } else {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment