Re: svn commit: r218238 - head/sys/dev/ath

2011-02-05 Thread Peter Jeremy
On 2011-Feb-04 09:23:01 -0700, Warner Losh i...@bsdimp.com wrote: On 02/03/2011 16:07, Bruce Cran wrote: On Thu, Feb 03, 2011 at 08:26:26PM +, Adrian Chadd wrote: if (ni != NULL) { +#if NOTYET /* tag AMPDU aggregates for reorder processing */ This seems to have

Re: svn commit: r218238 - head/sys/dev/ath

2011-02-04 Thread Warner Losh
On 02/03/2011 16:07, Bruce Cran wrote: On Thu, Feb 03, 2011 at 08:26:26PM +, Adrian Chadd wrote: if (ni != NULL) { +#if NOTYET /* tag AMPDU aggregates for reorder processing */ This seems to have broken the build because NOTYET isn't defined. ANSI C states

svn commit: r218238 - head/sys/dev/ath

2011-02-03 Thread Adrian Chadd
Author: adrian Date: Thu Feb 3 20:26:26 2011 New Revision: 218238 URL: http://svn.freebsd.org/changeset/base/218238 Log: Disable the code I previously added from Rui's 802.11n branch. A-MPDU RX interferes with packet retransmission/reordering. In local testing, I was seeing A-MPDU being

Re: svn commit: r218238 - head/sys/dev/ath

2011-02-03 Thread Bruce Cran
On Thu, Feb 03, 2011 at 08:26:26PM +, Adrian Chadd wrote: if (ni != NULL) { +#if NOTYET /* tag AMPDU aggregates for reorder processing */ This seems to have broken the build because NOTYET isn't defined. -- Bruce Cran

Re: svn commit: r218238 - head/sys/dev/ath

2011-02-03 Thread Bruce Evans
On Thu, 3 Feb 2011, Bruce Cran wrote: On Thu, Feb 03, 2011 at 08:26:26PM +, Adrian Chadd wrote: if (ni != NULL) { +#if NOTYET /* tag AMPDU aggregates for reorder processing */ This seems to have broken the build because NOTYET isn't defined. It is a