Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 9 Feb 2008 00:46:23 GMT
From:      Sam Leffler <sam@FreeBSD.org>
To:        Perforce Change Reviews <perforce@freebsd.org>
Subject:   PERFORCE change 135080 for review
Message-ID:  <200802090046.m190kNte024871@repoman.freebsd.org>

next in thread | raw e-mail | index | archive | help
http://perforce.freebsd.org/chv.cgi?CH=135080

Change 135080 by sam@sam_ebb on 2008/02/09 00:46:13

	move debug msg down so it only shows up when an ADDBA request
	actually is to be sent

Affected files ...

.. //depot/projects/vap/sys/net80211/ieee80211_ht.c#10 edit

Differences ...

==== //depot/projects/vap/sys/net80211/ieee80211_ht.c#10 (text+ko) ====

@@ -1539,11 +1539,6 @@
 	int tid, dialogtoken;
 	static int tokens = 0;	/* XXX */
 
-	IEEE80211_NOTE(ni->ni_vap, IEEE80211_MSG_11N, ni,
-	    "%s: enable AMPDU on %s, avgpps %d pkts %d",
-	    __func__, ieee80211_wme_acnames[tap->txa_ac],
-	    tap->txa_avgpps, tap->txa_pkts);
-
 	/* XXX locking */
 	if ((tap->txa_flags & IEEE80211_AGGR_SETUP) == 0) {
 		/* do deferred setup of state */
@@ -1564,6 +1559,11 @@
 	/* XXX hack for not doing proper locking */
 	tap->txa_flags &= ~IEEE80211_AGGR_NAK;
 
+	IEEE80211_NOTE(ni->ni_vap, IEEE80211_MSG_11N, ni,
+	    "%s: enable AMPDU on %s, avgpps %d pkts %d",
+	    __func__, ieee80211_wme_acnames[tap->txa_ac],
+	    tap->txa_avgpps, tap->txa_pkts);
+
 	dialogtoken = (tokens+1) % 63;		/* XXX */
 
 	tid = WME_AC_TO_TID(tap->txa_ac);



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200802090046.m190kNte024871>