Linux-Wireless Archive on lore.kernel.org
 help / color / Atom feed
From: Peter Oh <peter.oh@eero.com>
To: Aaron Komisar <aaron.komisar@tandemg.com>
Cc: "linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>
Subject: Re: [PATCH] Can't scan on ETSI domains when operating ch is DFS
Date: Wed, 11 Sep 2019 09:28:51 -0700
Message-ID: <CALhWmc2qPf-qCrx3JdejkOi-SQnURnehF43byCk-nZtCDWoJ3Q@mail.gmail.com> (raw)
In-Reply-To: <1568188529-14653-1-git-send-email-aaron.komisar@tandemg.com>

On Wed, Sep 11, 2019 at 12:59 AM Aaron Komisar
<aaron.komisar@tandemg.com> wrote:
>
> In non-ETSI reg domains scan is blocked when operating channel is a DFS ch.
> For ETSI domains, however, once DFS channel is marked as available afer
> the CAC, this channel will remain available even after leaving this channel.
> Therefore a new CAC will not be required when scan is done.
>
> In cfg80211 scan is not blocked for ETSI reg domains.
> This patch enables scan in mac80211 as well when operating channel is a radar
> channel for ETSI reg domains.
>
> Signed-off-by: Aaron Komisar <aaron.komisar@tandemg.com>
> ---
>  include/net/cfg80211.h | 8 ++++++++
>  net/mac80211/scan.c    | 6 ++++--
>  net/wireless/reg.c     | 1 +
>  net/wireless/reg.h     | 8 --------
>  4 files changed, 13 insertions(+), 10 deletions(-)
>
> diff --git a/include/net/cfg80211.h b/include/net/cfg80211.h
> index 5253e7f667bd..5349898ed3e6 100644
> --- a/include/net/cfg80211.h
> +++ b/include/net/cfg80211.h
> @@ -5546,6 +5546,14 @@ const struct ieee80211_reg_rule *freq_reg_info(struct wiphy *wiphy,
>  const char *reg_initiator_name(enum nl80211_reg_initiator initiator);
>
>  /**
> + * regulatory_pre_cac_allowed - if pre-CAC allowed in the current dfs domain
> + * @wiphy: wiphy for which pre-CAC capability is checked.
> +
> + * Pre-CAC is allowed only in ETSI domain.
> + */
> +bool regulatory_pre_cac_allowed(struct wiphy *wiphy);
> +
> +/**
>   * DOC: Internal regulatory db functions
>   *
>   */
> diff --git a/net/mac80211/scan.c b/net/mac80211/scan.c
> index adf94ba1ed77..812ff820844b 100644
> --- a/net/mac80211/scan.c
> +++ b/net/mac80211/scan.c
> @@ -523,7 +523,8 @@ static int ieee80211_start_sw_scan(struct ieee80211_local *local,
>  static bool ieee80211_can_scan(struct ieee80211_local *local,
>                                struct ieee80211_sub_if_data *sdata)
>  {
> -       if (ieee80211_is_radar_required(local))
> +       if (ieee80211_is_radar_required(local) &&
> +           !regulatory_pre_cac_allowed(local->hw.wiphy))
"pre-CAC allowed" doesn't mean that you can use the channel without CAC is done.
>                 return false;
>
>         if (!list_empty(&local->roc_list))
> @@ -630,7 +631,8 @@ static int __ieee80211_start_scan(struct ieee80211_sub_if_data *sdata,
>
>         lockdep_assert_held(&local->mtx);
>
> -       if (local->scan_req || ieee80211_is_radar_required(local))
> +       if (local->scan_req || (ieee80211_is_radar_required(local) &&
> +           !regulatory_pre_cac_allowed(local->hw.wiphy)))
>                 return -EBUSY;
>
>         if (!ieee80211_can_scan(local, sdata)) {
> diff --git a/net/wireless/reg.c b/net/wireless/reg.c
> index 5311d0ae2454..50d3242c4985 100644
> --- a/net/wireless/reg.c
> +++ b/net/wireless/reg.c
> @@ -3883,6 +3883,7 @@ bool regulatory_pre_cac_allowed(struct wiphy *wiphy)
>
>         return pre_cac_allowed;
>  }
> +EXPORT_SYMBOL(regulatory_pre_cac_allowed);
>
>  void regulatory_propagate_dfs_state(struct wiphy *wiphy,
>                                     struct cfg80211_chan_def *chandef,
> diff --git a/net/wireless/reg.h b/net/wireless/reg.h
> index 504133d76de4..dc8f689bd469 100644
> --- a/net/wireless/reg.h
> +++ b/net/wireless/reg.h
> @@ -156,14 +156,6 @@ bool regulatory_indoor_allowed(void);
>  #define REG_PRE_CAC_EXPIRY_GRACE_MS 2000
>
>  /**
> - * regulatory_pre_cac_allowed - if pre-CAC allowed in the current dfs domain
> - * @wiphy: wiphy for which pre-CAC capability is checked.
> -
> - * Pre-CAC is allowed only in ETSI domain.
> - */
> -bool regulatory_pre_cac_allowed(struct wiphy *wiphy);
> -
> -/**
>   * regulatory_propagate_dfs_state - Propagate DFS channel state to other wiphys
>   * @wiphy - wiphy on which radar is detected and the event will be propagated
>   *     to other available wiphys having the same DFS domain
> --
> 2.11.0
>

      reply index

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-09-11  7:58 Aaron Komisar
2019-09-11 16:28 ` Peter Oh [this message]

Reply instructions:

You may reply publically to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CALhWmc2qPf-qCrx3JdejkOi-SQnURnehF43byCk-nZtCDWoJ3Q@mail.gmail.com \
    --to=peter.oh@eero.com \
    --cc=aaron.komisar@tandemg.com \
    --cc=linux-wireless@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

Linux-Wireless Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-wireless/0 linux-wireless/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-wireless linux-wireless/ https://lore.kernel.org/linux-wireless \
		linux-wireless@vger.kernel.org linux-wireless@archiver.kernel.org
	public-inbox-index linux-wireless

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-wireless


AGPL code for this site: git clone https://public-inbox.org/ public-inbox