From: Michael Taylor <mike.taylor@apprion.com>
To: "Luis R. Rodriguez" <mcgrof@gmail.com>
Cc: Nick Kossifidis <mickflemm@gmail.com>,
John Linville <linville@tuxdriver.com>,
linux-wireless@vger.kernel.org, Jiri Slaby <jirislaby@gmail.com>
Subject: Re: [PATCH 2/6] ath5k: Enable radar detection
Date: Mon, 08 Oct 2007 14:18:41 -0700 [thread overview]
Message-ID: <470A9EB1.4030706@apprion.com> (raw)
In-Reply-To: <20071008203415.GA7633@pogo>
> So far I haven't received one AR5K_INT_RXPHY interrupt yet. Mind you we
> are already setting radar to 1 too -- we do this at the end of
> ath_reset() with ath5k_hw_enable_radar_alert() -- so not sure what's
> going on. I do live in front of a cemetery...
>
You may be using an AR5210 chip?
Look at madwifi-dfs branch in madwifi.org for an example of how to
configure the pulse detector. To get radar pulse detection to work you
have to update the pulse detection configuration, disable the filtering
of PHY radar errors, and enable the radar detection. Then you should be
able to get the errors just by playing a video (such as the one provided
by the FCC) between a client and the AP. You should get enough false
positives from the pulse detector to notice. You won't get them from an
idle AP, and you are very unlikely to detect real radar (in my
experience). I work at NASA Research Park at what was formerly Moffett
Air Base. There's an airport less than 1 mile away and I've never
detected true radar.
IMPORTANT: The Atheros hardware 5212 and higher provides only pulse
detection, NOT radar detection. The 5210 stuff doesn't even provide
radar errors or pulse detection... it's supposed to be possible to use
zero-byte packets with PHYERR or something if you disable filtering of
zero byte packets, but I haven't tried it.
The driver has to match the pulses to known patterns and filter out the
noise. It has to be able to filter out excess pulses and handle missed
pulses (you can't detect radar while transmitting). That's why the
madwifi-dfs branch has been brewing for longer than you might think. The
Atheros proprietary drivers provide one implementation of matching and
the madwifi-dfs branch currently works for all short-pulse type radars,
and isn't quite ready for prime time for long pulse type radars.
BTW, there are a lot more details to DFS than getting a pulse error from
the hardware... but I'll assume you know what has to happen at the
higher levels.
For the lower level stuff, feel free to send me any questions you have
for Atheros radar detection.
- M
> Anyway IMO we shouldn't handle radar detection but simply pass it down
> to the upper layers, in this case mac80211. It seems for DFS this
> would mean to force to change channels and blacklist the channel for
> 30 minutes. If this interrupt *does* prove to be very noisy we can
> simply disable the AR5K_RX_FILTER_RADARERR and even AR5K_RX_FILTER_PHYERR
> later. Perhaps AR5K_RX_FILTER_RADARERR should be enabled during
> configure_filter() through mac80211 to indicate when we do *need* DFS.
> Right now AR5K_RX_FILTER_PHYERR is enabled when mac80211 tells us
> about FIF_FCSFAIL | FIF_PLCPFAIL, we also keep the current
> hw settings on AR5K_PHY_ERR_FIL (AR5K_PHY_ERR_FIL_RADAR,
> AR5K_PHY_ERR_FIL_OFDM or AR5K_PHY_ERR_FIL_CCK).
>
> Comments?
>
> Luis
>
next prev parent reply other threads:[~2007-10-08 21:34 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-10-06 1:04 [PATCH 0/6] ath5k: initvals, radar, promiscuous bug and multicast Luis R. Rodriguez
2007-10-06 1:16 ` [PATCH 1/6] ath5k: Ported new initval changes from OpenBSD to ath5k Luis R. Rodriguez
2007-10-06 1:18 ` [PATCH 2/6] ath5k: Enable radar detection Luis R. Rodriguez
2007-10-06 1:19 ` [PATCH 3/6] ath5k: Fix a bug which pushed us to enable the promiscuous filter Luis R. Rodriguez
2007-10-06 1:19 ` [PATCH 4/6] ath5k: Add proper support for multicast Luis R. Rodriguez
2007-10-06 1:20 ` [PATCH 5/6] ath5k: Add documenation for atheros bssid_mask Luis R. Rodriguez
2007-10-06 1:20 ` [PATCH 6/6] ath5k: Do not let the driver through for not yet supported radios Luis R. Rodriguez
2007-10-07 10:13 ` Nick Kossifidis
2007-10-08 20:48 ` Luis R. Rodriguez
2007-10-09 2:37 ` Nick Kossifidis
2007-10-09 7:02 ` Holger Schurig
2007-10-09 8:17 ` Nick Kossifidis
2007-10-09 20:05 ` [PATCH 5/6] ath5k: Add documenation for atheros bssid_mask Johannes Berg
2007-10-09 20:01 ` [PATCH 4/6] ath5k: Add proper support for multicast Johannes Berg
2007-10-10 18:11 ` Luis R. Rodriguez
2007-10-07 9:56 ` [PATCH 2/6] ath5k: Enable radar detection Nick Kossifidis
2007-10-07 10:28 ` Nick Kossifidis
2007-10-08 20:34 ` Luis R. Rodriguez
2007-10-08 21:18 ` Michael Taylor [this message]
2007-10-09 7:05 ` Holger Schurig
2007-10-09 9:34 ` Johannes Berg
2007-10-09 15:59 ` Luis R. Rodriguez
2007-10-07 9:49 ` [PATCH 1/6] ath5k: Ported new initval changes from OpenBSD to ath5k Nick Kossifidis
2007-10-08 20:38 ` Luis R. Rodriguez
2007-10-09 2:31 ` Nick Kossifidis
2007-10-09 7:17 ` Jiri Slaby
2007-10-09 8:15 ` Nick Kossifidis
2007-10-06 1:23 ` [PATCH 0/6] ath5k: initvals, radar, promiscuous bug and multicast 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=470A9EB1.4030706@apprion.com \
--to=mike.taylor@apprion.com \
--cc=jirislaby@gmail.com \
--cc=linux-wireless@vger.kernel.org \
--cc=linville@tuxdriver.com \
--cc=mcgrof@gmail.com \
--cc=mickflemm@gmail.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 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).