All of lore.kernel.org
 help / color / mirror / Atom feed
From: Zefir Kurtisi <zefir.kurtisi@neratec.com>
To: Adrian Chadd <adrian@freebsd.org>
Cc: "Luis R. Rodriguez" <rodrigue@qca.qualcomm.com>,
	Christian Lamparter <chunkeey@googlemail.com>,
	kgiori@qca.qualcomm.com, ath9k-devel@lists.ath9k.org,
	linux-wireless@vger.kernel.org
Subject: Re: [ath9k-devel] [RFC 5/6] ath9k: enable DFS pulse detection
Date: Fri, 07 Oct 2011 10:48:43 +0200	[thread overview]
Message-ID: <4E8EBCEB.80705@neratec.com> (raw)
In-Reply-To: <CAJ-VmonYaTW3xX7CKzU4Okb1nW86mJMV0RRDnqkMSJ-mASJ+4Q@mail.gmail.com>

On 10/07/2011 05:06 AM, Adrian Chadd wrote:
> Just FYI, that's what I'm likely to do for FreeBSD 10 (as I just don't
> have time to try and make all the required regulatory changes before
> the upcoming 9.0 release.)
> 
> Ie:
> 
> * DFS station mode (net80211) is going to be compiled and enabled by default;
> * DFS master mode (net80211) is going to be compiled and enabled by default;
> * The drivers which support radar detection in firmware (currently
> only if_mwl) will set the DFS net80211 flag (ie, for master mode radar
> detection);
> * ath won't ship with the DFS enable flag (for master mode);
> * I'll modify the regulatory database code to include per-band DFS
> information (DFS domain, CAC/NOL timeout, interference timeout, etc)
> and some device information (eg whether it supports HT20/HT40/etc DFS
> detection);
> * I'll then flip on the DFS channel enforcement in the net80211 code
> so disables master mode on channels requiring DFS.
> 
> The radar detection code and channel interference code will live in
> the ath driver but the DFS machinery (CAC, NOL, CSA, etc) is in
> net80211. That way other NICs (eg if_mwl Marvell NICs) with DFS radar
> detection support can leverage this.
> 
Quite some work ahead. Good luck on that!

So you are confident to get existing code re-factored and split into multiple layers?

> I'll then likely ship two DFS modules - dfs_null (no DFS support, just
> a placeholder for the API) and whatever code is ported from the
> reference driver. Maybe I'll also include the code from Neratec if
> it's dual-licenced. But I won't include radar patterns by default -
> I'll include those on a documentation page which explains the how and
> why of regulatory domain stuff.
> 
Initially we planned to have common pattern detectors in mac80211 to handle pulses reported by all drivers, but so far only TI is working on DFS -- with pattern matching done in firmware. So, no need yet for common detectors, we might end up going the other way around, i.e. taking your port and integrate in into ath9k -- if licensing allowed us to do.

> Once FreeBSD ships DFS radar detection code, I'll make sure it isn't
> compiled by default and even if enabled, it won't advertise DFS
> channel support unless a valid radar pattern and radar parameter
> configuration is loaded in. That way users won't inadvertently enable
> it without being compliant.
> 
> Finally, I'm hoping to get all of this documented as much as possible
> so the community can pick this stuff up and run with it. I was hoping
> someone would throw me a 5ghz SDR (software defined radio) so I could
> prototype up some open source radar pulse generation code, just to
> lower the entry barrier to all of this.
> 
Yeah, an AR9003 card used as cheap radar generator is on top of my wish list for DFS testing without our always otherwise occupied vector signal generator. But fur obvious reasons we wont get the required documentation for such unofficial hacks.

> HTH,
> 
> 
> Adrian

Thanks
Zefir

WARNING: multiple messages have this Message-ID (diff)
From: Zefir Kurtisi <zefir.kurtisi@neratec.com>
To: ath9k-devel@lists.ath9k.org
Subject: [ath9k-devel] [RFC 5/6] ath9k: enable DFS pulse detection
Date: Fri, 07 Oct 2011 10:48:43 +0200	[thread overview]
Message-ID: <4E8EBCEB.80705@neratec.com> (raw)
In-Reply-To: <CAJ-VmonYaTW3xX7CKzU4Okb1nW86mJMV0RRDnqkMSJ-mASJ+4Q@mail.gmail.com>

On 10/07/2011 05:06 AM, Adrian Chadd wrote:
> Just FYI, that's what I'm likely to do for FreeBSD 10 (as I just don't
> have time to try and make all the required regulatory changes before
> the upcoming 9.0 release.)
> 
> Ie:
> 
> * DFS station mode (net80211) is going to be compiled and enabled by default;
> * DFS master mode (net80211) is going to be compiled and enabled by default;
> * The drivers which support radar detection in firmware (currently
> only if_mwl) will set the DFS net80211 flag (ie, for master mode radar
> detection);
> * ath won't ship with the DFS enable flag (for master mode);
> * I'll modify the regulatory database code to include per-band DFS
> information (DFS domain, CAC/NOL timeout, interference timeout, etc)
> and some device information (eg whether it supports HT20/HT40/etc DFS
> detection);
> * I'll then flip on the DFS channel enforcement in the net80211 code
> so disables master mode on channels requiring DFS.
> 
> The radar detection code and channel interference code will live in
> the ath driver but the DFS machinery (CAC, NOL, CSA, etc) is in
> net80211. That way other NICs (eg if_mwl Marvell NICs) with DFS radar
> detection support can leverage this.
> 
Quite some work ahead. Good luck on that!

So you are confident to get existing code re-factored and split into multiple layers?

> I'll then likely ship two DFS modules - dfs_null (no DFS support, just
> a placeholder for the API) and whatever code is ported from the
> reference driver. Maybe I'll also include the code from Neratec if
> it's dual-licenced. But I won't include radar patterns by default -
> I'll include those on a documentation page which explains the how and
> why of regulatory domain stuff.
> 
Initially we planned to have common pattern detectors in mac80211 to handle pulses reported by all drivers, but so far only TI is working on DFS -- with pattern matching done in firmware. So, no need yet for common detectors, we might end up going the other way around, i.e. taking your port and integrate in into ath9k -- if licensing allowed us to do.

> Once FreeBSD ships DFS radar detection code, I'll make sure it isn't
> compiled by default and even if enabled, it won't advertise DFS
> channel support unless a valid radar pattern and radar parameter
> configuration is loaded in. That way users won't inadvertently enable
> it without being compliant.
> 
> Finally, I'm hoping to get all of this documented as much as possible
> so the community can pick this stuff up and run with it. I was hoping
> someone would throw me a 5ghz SDR (software defined radio) so I could
> prototype up some open source radar pulse generation code, just to
> lower the entry barrier to all of this.
> 
Yeah, an AR9003 card used as cheap radar generator is on top of my wish list for DFS testing without our always otherwise occupied vector signal generator. But fur obvious reasons we wont get the required documentation for such unofficial hacks.

> HTH,
> 
> 
> Adrian

Thanks
Zefir

  parent reply	other threads:[~2011-10-07  8:48 UTC|newest]

Thread overview: 110+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-10-03 10:29 [RFC 0/6] ath9k: DFS pattern detection Zefir Kurtisi
2011-10-03 10:29 ` [ath9k-devel] " Zefir Kurtisi
2011-10-03 10:29 ` [RFC 1/6] ath9k: add DFS statistics to debugfs Zefir Kurtisi
2011-10-03 10:29   ` [ath9k-devel] " Zefir Kurtisi
2011-10-03 18:14   ` Luis R. Rodriguez
2011-10-03 18:14     ` [ath9k-devel] " Luis R. Rodriguez
2011-10-04  8:27     ` Zefir Kurtisi
2011-10-04  8:27       ` [ath9k-devel] " Zefir Kurtisi
2011-10-03 10:29 ` [RFC 2/6] ath9k: add DFS debug flag Zefir Kurtisi
2011-10-03 10:29   ` [ath9k-devel] " Zefir Kurtisi
2011-10-03 18:15   ` Luis R. Rodriguez
2011-10-03 18:15     ` [ath9k-devel] " Luis R. Rodriguez
2011-10-04  8:31     ` Zefir Kurtisi
2011-10-04  8:31       ` [ath9k-devel] " Zefir Kurtisi
2011-10-04  9:40       ` Mohammed Shafi
2011-10-04  9:40         ` [ath9k-devel] " Mohammed Shafi
2011-10-03 10:29 ` [RFC 3/6] ath9k: initial radar pulse detection for DFS Zefir Kurtisi
2011-10-03 10:29   ` [ath9k-devel] " Zefir Kurtisi
2011-10-03 11:57   ` Adrian Chadd
2011-10-03 11:57     ` [ath9k-devel] " Adrian Chadd
2011-10-03 12:23     ` Zefir Kurtisi
2011-10-03 12:23       ` [ath9k-devel] " Zefir Kurtisi
2011-10-03 12:43       ` Adrian Chadd
2011-10-03 12:43         ` [ath9k-devel] " Adrian Chadd
2011-10-03 14:21         ` Zefir Kurtisi
2011-10-03 14:21           ` [ath9k-devel] " Zefir Kurtisi
2011-10-03 14:23           ` Adrian Chadd
2011-10-03 14:23             ` [ath9k-devel] " Adrian Chadd
2011-10-03 10:29 ` [RFC 4/6] ath9k: add DFS build parameter Zefir Kurtisi
2011-10-03 10:29   ` [ath9k-devel] " Zefir Kurtisi
2011-10-03 18:26   ` Luis R. Rodriguez
2011-10-03 18:26     ` [ath9k-devel] " Luis R. Rodriguez
2011-10-04  9:55     ` Zefir Kurtisi
2011-10-04  9:55       ` [ath9k-devel] " Zefir Kurtisi
2011-10-04 10:37       ` Felix Fietkau
2011-10-04 10:37         ` [ath9k-devel] " Felix Fietkau
2011-10-04 12:25         ` Adrian Chadd
2011-10-04 12:25           ` [ath9k-devel] " Adrian Chadd
2011-10-05 22:20         ` Luis R. Rodriguez
2011-10-05 22:20           ` [ath9k-devel] " Luis R. Rodriguez
2011-10-03 10:29 ` [RFC 5/6] ath9k: enable DFS pulse detection Zefir Kurtisi
2011-10-03 10:29   ` [ath9k-devel] " Zefir Kurtisi
2011-10-03 18:27   ` Luis R. Rodriguez
2011-10-03 18:27     ` [ath9k-devel] " Luis R. Rodriguez
2011-10-03 19:24     ` Christian Lamparter
2011-10-03 19:24       ` [ath9k-devel] " Christian Lamparter
2011-10-03 19:31       ` Luis R. Rodriguez
2011-10-03 19:31         ` [ath9k-devel] " Luis R. Rodriguez
2011-10-04 13:38         ` Christian Lamparter
2011-10-04 13:38           ` [ath9k-devel] " Christian Lamparter
2011-10-04 14:17           ` Zefir Kurtisi
2011-10-04 14:17             ` [ath9k-devel] " Zefir Kurtisi
2011-10-04 14:34             ` Adrian Chadd
2011-10-04 14:34               ` [ath9k-devel] " Adrian Chadd
2011-10-05 22:31               ` Luis R. Rodriguez
2011-10-05 22:31                 ` [ath9k-devel] " Luis R. Rodriguez
2011-10-05 22:53                 ` Peter Stuge
2011-10-05 23:02                   ` Luis R. Rodriguez
2011-10-04 14:42             ` Christian Lamparter
2011-10-04 14:42               ` [ath9k-devel] " Christian Lamparter
2011-10-04 14:50               ` Adrian Chadd
2011-10-04 14:50                 ` [ath9k-devel] " Adrian Chadd
2011-10-04 15:26                 ` Christian Lamparter
2011-10-04 15:26                   ` [ath9k-devel] " Christian Lamparter
2011-10-04 15:57                   ` Adrian Chadd
2011-10-04 15:57                     ` [ath9k-devel] " Adrian Chadd
2011-10-04 16:42                     ` Christian Lamparter
2011-10-04 16:42                       ` [ath9k-devel] " Christian Lamparter
2011-10-04 17:03                       ` Adrian Chadd
2011-10-04 17:03                         ` [ath9k-devel] " Adrian Chadd
2011-10-04 17:49                         ` Christian Lamparter
2011-10-04 17:49                           ` [ath9k-devel] " Christian Lamparter
2011-10-05 22:37                 ` Luis R. Rodriguez
2011-10-05 22:37                   ` [ath9k-devel] " Luis R. Rodriguez
2011-10-04 16:26               ` Zefir Kurtisi
2011-10-04 16:26                 ` [ath9k-devel] " Zefir Kurtisi
2011-10-05 22:30             ` Luis R. Rodriguez
2011-10-05 22:30               ` [ath9k-devel] " Luis R. Rodriguez
2011-10-05 22:27           ` Luis R. Rodriguez
2011-10-05 22:27             ` Luis R. Rodriguez
2011-10-06 16:49             ` Christian Lamparter
2011-10-06 16:49               ` Christian Lamparter
2011-10-06 18:36               ` Luis R. Rodriguez
2011-10-06 18:36                 ` Luis R. Rodriguez
2011-10-06 18:41                 ` Luis R. Rodriguez
2011-10-06 18:41                   ` Luis R. Rodriguez
2011-10-06 20:32                 ` Zefir Kurtisi
2011-10-06 20:32                   ` Zefir Kurtisi
2011-10-06 20:41                   ` Luis R. Rodriguez
2011-10-06 20:41                     ` Luis R. Rodriguez
2011-10-06 21:08                     ` Zefir Kurtisi
2011-10-06 21:08                       ` Zefir Kurtisi
2011-10-06 21:12                       ` Luis R. Rodriguez
2011-10-06 21:12                         ` Luis R. Rodriguez
2011-10-07  3:06                       ` Adrian Chadd
2011-10-07  3:06                         ` Adrian Chadd
2011-10-07  7:54                         ` Luis R. Rodriguez
2011-10-07  7:54                           ` Luis R. Rodriguez
2011-10-07  8:48                         ` Zefir Kurtisi [this message]
2011-10-07  8:48                           ` Zefir Kurtisi
2011-10-07 11:43                           ` Adrian Chadd
2011-10-07 11:43                             ` Adrian Chadd
2011-10-04 10:11     ` Zefir Kurtisi
2011-10-04 10:11       ` [ath9k-devel] " Zefir Kurtisi
2011-10-05 22:23       ` Luis R. Rodriguez
2011-10-05 22:23         ` [ath9k-devel] " Luis R. Rodriguez
2011-10-03 10:29 ` [RFC 6/6] ath9k: handle pulse data reported by DFS HW Zefir Kurtisi
2011-10-03 10:29   ` [ath9k-devel] " Zefir Kurtisi
2011-10-03 18:30   ` Luis R. Rodriguez
2011-10-03 18:30     ` [ath9k-devel] " Luis R. Rodriguez

Reply instructions:

You may reply publicly 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=4E8EBCEB.80705@neratec.com \
    --to=zefir.kurtisi@neratec.com \
    --cc=adrian@freebsd.org \
    --cc=ath9k-devel@lists.ath9k.org \
    --cc=chunkeey@googlemail.com \
    --cc=kgiori@qca.qualcomm.com \
    --cc=linux-wireless@vger.kernel.org \
    --cc=rodrigue@qca.qualcomm.com \
    /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
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.