diff mbox series

wifi: mac80211: add support for scanning in ap mode

Message ID 20230110110524.511258-1-vincent@systemli.org
State New
Headers show
Series wifi: mac80211: add support for scanning in ap mode | expand

Commit Message

Nick Hainke Jan. 10, 2023, 11:05 a.m. UTC
OpenWRT has shipped a patch since 2011 that allows it to perform a scan
in AP mode, whether it is supported by the driver or not. In certain
situations, it may be desirable to scan an interface that is currently
in AP mode regardless of whether frames are missed. The patch adds a
module parameter "allow_ap_scan" that, if set to true, allows the behavior
described above.

Tested-on: TP Link Archer C7 V2 (Qualcomm Atheros QCA9558,
           Qualcomm Atheros QCA9880-BR4A) with OpenWrt Linux 5.15.86

Signed-off-by: Nick Hainke <vincent@systemli.org>
---
 net/mac80211/cfg.c         | 3 +++
 net/mac80211/ieee80211_i.h | 3 +++
 net/mac80211/main.c        | 5 +++++
 3 files changed, 11 insertions(+)

Comments

Johannes Berg Jan. 10, 2023, 11:07 a.m. UTC | #1
On Tue, 2023-01-10 at 12:05 +0100, Nick Hainke wrote:
> OpenWRT has shipped a patch since 2011 that allows it to perform a scan
> in AP mode, whether it is supported by the driver or not. In certain
> situations, it may be desirable to scan an interface that is currently
> in AP mode regardless of whether frames are missed. The patch adds a
> module parameter "allow_ap_scan" that, if set to true, allows the behavior
> described above.
> 

I must say - not really in favour of throwing around random module
parameters like that :)

johannes
Johannes Berg Jan. 10, 2023, 11:19 a.m. UTC | #2
On Tue, 2023-01-10 at 12:18 +0100, Nick wrote:
> Thanks for your feedback. Can you suggest a better way to do this?
> 

Well there already is NL80211_SCAN_FLAG_AP?

johannes
diff mbox series

Patch

diff --git a/net/mac80211/cfg.c b/net/mac80211/cfg.c
index 8f9a2ab502b3..04730fb0f621 100644
--- a/net/mac80211/cfg.c
+++ b/net/mac80211/cfg.c
@@ -2723,6 +2723,9 @@  static int ieee80211_scan(struct wiphy *wiphy,
 		 */
 		fallthrough;
 	case NL80211_IFTYPE_AP:
+		/* Support scanning in AP mode regardless of driver support. */
+		if (allow_ap_scan)
+			break;
 		/*
 		 * If the scan has been forced (and the driver supports
 		 * forcing), don't care about being beaconing already.
diff --git a/net/mac80211/ieee80211_i.h b/net/mac80211/ieee80211_i.h
index 63ff0d2524b6..d48c7dd00dd4 100644
--- a/net/mac80211/ieee80211_i.h
+++ b/net/mac80211/ieee80211_i.h
@@ -2566,4 +2566,7 @@  ieee80211_eht_cap_ie_to_sta_eht_cap(struct ieee80211_sub_if_data *sdata,
 				    const struct ieee80211_eht_cap_elem *eht_cap_ie_elem,
 				    u8 eht_cap_len,
 				    struct link_sta_info *link_sta);
+
+extern bool allow_ap_scan;
+
 #endif /* IEEE80211_I_H */
diff --git a/net/mac80211/main.c b/net/mac80211/main.c
index 846528850612..c33d99717cd0 100644
--- a/net/mac80211/main.c
+++ b/net/mac80211/main.c
@@ -33,6 +33,11 @@ 
 #include "led.h"
 #include "debugfs.h"
 
+bool allow_ap_scan;
+module_param(allow_ap_scan, bool, 0644);
+MODULE_PARM_DESC(allow_ap_scan,
+		 "Support scanning in AP mode regardless of driver support.");
+
 void ieee80211_configure_filter(struct ieee80211_local *local)
 {
 	u64 mc;