linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* Re: [PATCH v5] cfg80211: fix dfs channel state after stopping AP
@ 2013-12-18  9:19 Marek Puzyniak
  2013-12-18 11:11 ` Simon Wunderlich
  0 siblings, 1 reply; 6+ messages in thread
From: Marek Puzyniak @ 2013-12-18  9:19 UTC (permalink / raw)
  To: Simon Wunderlich; +Cc: linux-wireless, Johannes Berg

On 17 December 2013 15:44, Simon Wunderlich <sw@simonwunderlich.de> wrote:
> Sorry for answering after so many revisions, but ... why do you think this is
> required? Many APs first do a CAC on all available channels, but then pick one
> channel and use that, while being able to switch to another channel
> immediately.
>
> WIth this patch (if I understand that correctly), this is not possible
> anymore. Why do you think it is required? I can not find this requirement in
> ETSI, at least.
>
> To quote from ETSI 301.893:
>
> Once the RLAN has started operations on an Available Channel, then that
> channel becomes an Operating Channel. During normal operation, the master
> device shall monitor all Operating Channels (In-Service Monitoring) to ensure
> that there is no radar operating within these channel(s). If no radar was
> detected on an Operating Channel but the RLAN stops operating on that channel,
> then the channel becomes an Available Channel.
>
> I think this patch is wrong and should not be applied.
>
> Thanks,
>     Simon

I also have doubts when reading this specification:

ETSI 301.893 v1.7.0:

"A master device shall only start operations on Available Channels. At
installation (or reinstallation) of the equipment, the RLAN is assumed
to have no Available Channels within the band 5 250 MHz to 5 350 MHz
and/or 5 470 MHz to 5 725 MHz. In such a case, before starting
operations on one or more of these channels, the master device shall
perform either a Channel Availability Check or an Off-Channel CAC to
ensure that there are no radars operating on any selected channel. If
no radar has been detected, the channel(s) becomes an Available
Channel(s) and remains as such until a radar signal is detected during
the In-Service Monitoring. The Channel Availability Check or the
Off-Channel CAC may be performed over a wider bandwidth such that all
channels within the tested bandwidth become Available Channels."

First thing: how to interpret words installation and reinstallation?
If we include stating hostapd in installation process then we should
have no available channels when starting hostapd.

Second thing: How to interpret this sentence: "If no radar has been
detected, the channel(s) becomes an Available Channel(s) and remains
as such until a radar signal is detected during the In-Service
Monitoring."
As I understand this sentence channel can be available as long as
radar detection works, so called In-Service Monitoring. When in
service monitoring is off then channel can not be available. Of course
this is my interpretation. So if there are APs than can monitor one
channel after switch to new channel then I agree that both channels
can be available. In hostapd when last AP interface is stopped then it
is no longer monitoring current channel, so starting AP after some
time can not happened without CAC. Simple case: we have our AP on
mobile device, it was stopped in one place where was no radars and
after moving to new place we start AP again. But this new place is in
radar range. So we should start beaconig or start CAC first?


Marek

^ permalink raw reply	[flat|nested] 6+ messages in thread
* [PATCH v5] cfg80211: fix dfs channel state after stopping AP
@ 2013-12-16 15:50 Marek Puzyniak
  2013-12-16 20:31 ` Johannes Berg
  2013-12-17 14:44 ` Simon Wunderlich
  0 siblings, 2 replies; 6+ messages in thread
From: Marek Puzyniak @ 2013-12-16 15:50 UTC (permalink / raw)
  To: linux-wireless; +Cc: johannes, Marek Puzyniak

In AP mode DFS channel state is changed to DFS_AVAILABLE
after successful CAC and remains as such until a radar
signal is detected during the In-Service Monitoring.
When AP is stopped it is no longer monitoring current channel
for radar signals. DFS channel state should be changed
to DFS_USABLE when last AP interface is stopped. Starting AP
again on that channel will start CAC instead of starting radiation.

Signed-off-by: Marek Puzyniak <marek.puzyniak@tieto.com>
---

Patch v1:
*Initial version. Partially in mac80211 and cfg80211.

Patch v2:
*Moved all implementation to cfg80211.

Patch v3:
*Rephrase patch description.

Patch v4:
*Fix coding style,
*Do not check if RADAR flag is set, when dfs channel state was AVAILABLE
 change it to USABLE, 
*Handle multiBSSID APs - Change DFS channel state when last AP was stoped. 

Patch v5:
*Fix coding style,
*Remove unnecessary and nested wdev_lock.

 net/wireless/ap.c   | 36 ++++++++++++++++++++++++++++++++++++
 net/wireless/chan.c | 38 ++++++++++++++++++++++++++++++++++++++
 net/wireless/core.h | 11 +++++++++++
 3 files changed, 85 insertions(+)

diff --git a/net/wireless/ap.c b/net/wireless/ap.c
index 324e8d8..28d1a3c 100644
--- a/net/wireless/ap.c
+++ b/net/wireless/ap.c
@@ -6,6 +6,39 @@
 #include "rdev-ops.h"
 
 
+static bool cfg80211_is_last_ap_running(struct cfg80211_registered_device *rdev,
+					struct net_device *dev)
+{
+	struct wireless_dev *wdev = dev->ieee80211_ptr;
+	struct wireless_dev *wdev_iter;
+
+	ASSERT_WDEV_LOCK(wdev);
+
+	if (rdev->num_running_ifaces == 1) {
+		/* Last running AP iface */
+		return true;
+	}
+
+	/* More running interfaces */
+	list_for_each_entry(wdev_iter, &rdev->wdev_list, list) {
+		if (wdev_iter == wdev)
+			continue;
+
+		if (!wdev_iter->netdev)
+			continue;
+
+		if (!netif_running(wdev_iter->netdev)) {
+			continue;
+		}
+
+		if (wdev_iter->iftype == NL80211_IFTYPE_AP) {
+			return false;
+		}
+	}
+
+	return true;
+}
+
 static int __cfg80211_stop_ap(struct cfg80211_registered_device *rdev,
 			      struct net_device *dev)
 {
@@ -31,6 +64,9 @@ static int __cfg80211_stop_ap(struct cfg80211_registered_device *rdev,
 		wdev->ssid_len = 0;
 	}
 
+	if (cfg80211_is_last_ap_running(rdev, dev))
+		cfg80211_leave_dfs_chandef(wdev->wiphy, &wdev->preset_chandef);
+
 	return err;
 }
 
diff --git a/net/wireless/chan.c b/net/wireless/chan.c
index 78559b5..54c217a 100644
--- a/net/wireless/chan.c
+++ b/net/wireless/chan.c
@@ -490,6 +490,44 @@ static bool cfg80211_chandef_dfs_available(struct wiphy *wiphy,
 	return r;
 }
 
+static void cfg80211_leave_dfs_chans(struct wiphy *wiphy,
+				     u32 center_freq,
+				     u32 bandwidth)
+{
+	struct ieee80211_channel *c;
+	u32 freq, start_freq, end_freq;
+
+	start_freq = cfg80211_get_start_freq(center_freq, bandwidth);
+	end_freq = cfg80211_get_end_freq(center_freq, bandwidth);
+
+	for (freq = start_freq; freq <= end_freq; freq += 20) {
+		c = ieee80211_get_channel(wiphy, freq);
+		if (!c)
+			continue;
+		if (c->dfs_state != NL80211_DFS_AVAILABLE)
+			continue;
+
+		c->dfs_state = NL80211_DFS_USABLE;
+		c->dfs_state_entered = jiffies;
+	}
+}
+
+void cfg80211_leave_dfs_chandef(struct wiphy *wiphy,
+				struct cfg80211_chan_def *chandef)
+{
+	int width;
+
+	if (WARN_ON(!cfg80211_chandef_valid(chandef)))
+		return;
+
+	width = cfg80211_chandef_get_width(chandef);
+	cfg80211_leave_dfs_chans(wiphy, chandef->center_freq1, width);
+
+	if (!chandef->center_freq2)
+		return;
+
+	cfg80211_leave_dfs_chans(wiphy, chandef->center_freq2, width);
+}
 
 static bool cfg80211_secondary_chans_ok(struct wiphy *wiphy,
 					u32 center_freq, u32 bandwidth,
diff --git a/net/wireless/core.h b/net/wireless/core.h
index 37ec16d..4ed2c81 100644
--- a/net/wireless/core.h
+++ b/net/wireless/core.h
@@ -398,6 +398,17 @@ void cfg80211_set_dfs_state(struct wiphy *wiphy,
 
 void cfg80211_dfs_channels_update_work(struct work_struct *work);
 
+/**
+ * cfg80211_leave_dfs_chandef - Leaving dfs chandef
+ * @wiphy: the wiphy
+ * @chandef: chandef for the current channel
+ *
+ * This function is called when dfs chandef is being not used for different
+ * reasons. Change channels DFS_AVAILABLE to DFS_USABLE again. Leave channels
+ * DFS_UNAVAILABLE untouched.
+ */
+void cfg80211_leave_dfs_chandef(struct wiphy *wiphy,
+				struct cfg80211_chan_def *chandef);
 
 static inline int
 cfg80211_can_change_interface(struct cfg80211_registered_device *rdev,
-- 
1.8.1.2


^ permalink raw reply related	[flat|nested] 6+ messages in thread
end of thread, other threads:[~2013-12-18 11:11 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-12-18  9:19 [PATCH v5] cfg80211: fix dfs channel state after stopping AP Marek Puzyniak
2013-12-18 11:11 ` Simon Wunderlich
  -- strict thread matches above, loose matches on Subject: below --
2013-12-16 15:50 Marek Puzyniak
2013-12-16 20:31 ` Johannes Berg
2013-12-17 14:44 ` Simon Wunderlich
2013-12-18  9:34   ` Johannes Berg

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).