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 negotiated and then not used by the AP sending frames to the STA; the STA would then treat non A-MPDU frames that are retransmits as out of the window and get plain confused. The hardware RX status descriptor has a "I'm part of an aggregate" bit; so this should eventually be tested and then punted to the A-MPDU reorder handling only if it has this bit set. Modified: head/sys/dev/ath/if_ath.c Modified: head/sys/dev/ath/if_ath.c ============================================================================== --- head/sys/dev/ath/if_ath.c Thu Feb 3 20:10:16 2011 (r218237) +++ head/sys/dev/ath/if_ath.c Thu Feb 3 20:26:26 2011 (r218238) @@ -3892,9 +3892,15 @@ rx_accept: IEEE80211_KEYIX_NONE : rs->rs_keyix); sc->sc_lastrs = rs; if (ni != NULL) { +#if NOTYET /* tag AMPDU aggregates for reorder processing */ + /* + * XXX this should only tag frames marked as aggregate; rather + * XXX than all frames. + */ if (ni->ni_flags & IEEE80211_NODE_HT) m->m_flags |= M_AMPDU; +#endif /* * Sending station is known, dispatch directly. _______________________________________________ svn-src-head@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/svn-src-head To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"