Date: Wed, 23 Mar 2011 03:58:55 +0000 (UTC) From: Adrian Chadd <adrian@FreeBSD.org> To: cvs-src-old@freebsd.org Subject: cvs commit: src/sys/dev/ath/ath_hal/ar5416 ar5416_ani.c Message-ID: <201103230359.p2N3xFxl082317@repoman.freebsd.org>
index | next in thread | raw e-mail
adrian 2011-03-23 03:58:55 UTC
FreeBSD src repository
Modified files:
sys/dev/ath/ath_hal/ar5416 ar5416_ani.c
Log:
SVN rev 219894 on 2011-03-23 03:58:55Z by adrian
The AR5416+ chips all have MIB counters (which the AR5416 ANI code assumes)
so there's no need to enable the RX of invalid frames just to do ANI.
The if_ath code and AR5212 ANI code setup the RX filter bits to enable
receiving OFDM/CCK errors if the device doesn't have the hardware
MIB counters. It isn't initialising it for the AR5416+ because all of
those chips have hardware MIB counters.
This fixes the odd (and performance affecting!) situation where if ani
is enabled (via sysctl dev.ath.X.intmit) then suddenly there's be a very
large volume of phy errors - which is good to track, but not what was
intended. Since each PHY error is a received (0 length) frame, it can
significantly tie up the RX side of things.
Revision Changes Path
1.9 +0 -4 src/sys/dev/ath/ath_hal/ar5416/ar5416_ani.c
help
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201103230359.p2N3xFxl082317>