Commit cf104c2a authored by Kalle Valo's avatar Kalle Valo

ath6kl: fix atomicity in ath6kl_cfg80211_scan_node()

ath6kl_cfg80211_scan_node() was calling cfg80211_inform_bss_frame()
with CFP_KERNEL but the function is executed with a spin lock taken.
This is wrong and the function must use GFP_ATOMIC instead.
Signed-off-by: default avatarKalle Valo <kvalo@qca.qualcomm.com>
parent b2c76bbe
...@@ -720,6 +720,7 @@ static inline bool is_ch_11a(u16 ch) ...@@ -720,6 +720,7 @@ static inline bool is_ch_11a(u16 ch)
return (!((ch >= 2412) && (ch <= 2484))); return (!((ch >= 2412) && (ch <= 2484)));
} }
/* struct ath6kl_node_table::nt_nodelock is locked when calling this */
static void ath6kl_cfg80211_scan_node(void *arg, struct bss *ni) static void ath6kl_cfg80211_scan_node(void *arg, struct bss *ni)
{ {
struct wiphy *wiphy = (struct wiphy *)arg; struct wiphy *wiphy = (struct wiphy *)arg;
...@@ -769,7 +770,7 @@ static void ath6kl_cfg80211_scan_node(void *arg, struct bss *ni) ...@@ -769,7 +770,7 @@ static void ath6kl_cfg80211_scan_node(void *arg, struct bss *ni)
"%s: bssid %pM ch %d freq %d size %d\n", __func__, "%s: bssid %pM ch %d freq %d size %d\n", __func__,
mgmt->bssid, channel->hw_value, freq, size); mgmt->bssid, channel->hw_value, freq, size);
cfg80211_inform_bss_frame(wiphy, channel, mgmt, cfg80211_inform_bss_frame(wiphy, channel, mgmt,
size, signal, GFP_KERNEL); size, signal, GFP_ATOMIC);
kfree(ieeemgmtbuf); kfree(ieeemgmtbuf);
} }
......
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