From owner-cvs-src-old@FreeBSD.ORG Wed Mar 23 03:59:15 2011 Return-Path: Delivered-To: cvs-src-old@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id BE2731065680 for ; Wed, 23 Mar 2011 03:59:15 +0000 (UTC) (envelope-from adrian@FreeBSD.org) Received: from repoman.freebsd.org (repoman.freebsd.org [IPv6:2001:4f8:fff6::29]) by mx1.freebsd.org (Postfix) with ESMTP id AC5748FC0A for ; Wed, 23 Mar 2011 03:59:15 +0000 (UTC) Received: from repoman.freebsd.org (localhost [127.0.0.1]) by repoman.freebsd.org (8.14.4/8.14.4) with ESMTP id p2N3xFAd082318 for ; Wed, 23 Mar 2011 03:59:15 GMT (envelope-from adrian@repoman.freebsd.org) Received: (from svn2cvs@localhost) by repoman.freebsd.org (8.14.4/8.14.4/Submit) id p2N3xFxl082317 for cvs-src-old@freebsd.org; Wed, 23 Mar 2011 03:59:15 GMT (envelope-from adrian@repoman.freebsd.org) Message-Id: <201103230359.p2N3xFxl082317@repoman.freebsd.org> X-Authentication-Warning: repoman.freebsd.org: svn2cvs set sender to adrian@repoman.freebsd.org using -f From: Adrian Chadd Date: Wed, 23 Mar 2011 03:58:55 +0000 (UTC) To: cvs-src-old@freebsd.org X-FreeBSD-CVS-Branch: HEAD Subject: cvs commit: src/sys/dev/ath/ath_hal/ar5416 ar5416_ani.c X-BeenThere: cvs-src-old@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: **OBSOLETE** CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 23 Mar 2011 03:59:15 -0000 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