linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
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
>   

  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).