From owner-cvs-src-old@FreeBSD.ORG Mon Jan 31 15:43:07 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 B6A931065679 for ; Mon, 31 Jan 2011 15:43:07 +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 A4DDA8FC1D for ; Mon, 31 Jan 2011 15:43:07 +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 p0VFh7h3013965 for ; Mon, 31 Jan 2011 15:43:07 GMT (envelope-from adrian@repoman.freebsd.org) Received: (from svn2cvs@localhost) by repoman.freebsd.org (8.14.4/8.14.4/Submit) id p0VFh7Cs013964 for cvs-src-old@freebsd.org; Mon, 31 Jan 2011 15:43:07 GMT (envelope-from adrian@repoman.freebsd.org) Message-Id: <201101311543.p0VFh7Cs013964@repoman.freebsd.org> X-Authentication-Warning: repoman.freebsd.org: svn2cvs set sender to adrian@repoman.freebsd.org using -f From: Adrian Chadd Date: Mon, 31 Jan 2011 15:42:42 +0000 (UTC) To: cvs-src-old@freebsd.org X-FreeBSD-CVS-Branch: HEAD Subject: cvs commit: src/sys/dev/ath/ath_hal/ar5416 ar5416_xmit.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: Mon, 31 Jan 2011 15:43:07 -0000 adrian 2011-01-31 15:42:42 UTC FreeBSD src repository Modified files: sys/dev/ath/ath_hal/ar5416 ar5416_xmit.c Log: SVN rev 218131 on 2011-01-31 15:42:42Z by adrian Don't incorrectly set the burst duration setting in the TX descriptor. After inspecting the ath9k source, it seems the AR5416 and later MACs don't take an explicit RTS/CTS duration. A per-scenario (ie, what multi- rate retry became) rts/cts control flag and packet duration is provided; the hardware then apparently fills in whatever details are required. The per-rate sp/lpack duration calculation just isn't used anywhere in the ath9k TX packet length calculations. The burst duration register controls something different; it seems to be involved with RTS/CTS protection of 11n aggregate frames and is set via a call to ar5416Set11nBurstDuration(). I've done some light testing with rts/cts protected frames and nothing seems to break; but this may break said RTS/CTS and CTS-to-self protection. Revision Changes Path 1.10 +0 -2 src/sys/dev/ath/ath_hal/ar5416/ar5416_xmit.c