Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 27 Sep 2005 23:50:04 GMT
From:      Sam Leffler <sam@FreeBSD.org>
To:        Perforce Change Reviews <perforce@freebsd.org>
Subject:   PERFORCE change 84402 for review
Message-ID:  <200509272350.j8RNo4aj054452@repoman.freebsd.org>

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

Change 84402 by sam@sam_ebb on 2005/09/27 23:49:03

	better

Affected files ...

.. //depot/projects/wifi/sys/dev/ath/if_ath.c#106 edit

Differences ...

==== //depot/projects/wifi/sys/dev/ath/if_ath.c#106 (text+ko) ====

@@ -3351,6 +3351,13 @@
 		 */
 		type = ieee80211_input(ic, m, ni,
 			ds->ds_rxstat.rs_rssi, ds->ds_rxstat.rs_tstamp);
+		/*
+		 * Arrange to update the last rx timestamp only for
+		 * frames from our ap.  This is a crude check but
+		 * should be good enough for our purpose.
+		 */
+		if (ni == ic->ic_bss)
+			ngood++;
 		ieee80211_free_node(ni);
 		if (sc->sc_diversity) {
 			/*
@@ -3378,16 +3385,6 @@
 			} else if (ticks - sc->sc_ledevent >= sc->sc_ledidle)
 				ath_led_event(sc, ATH_LED_POLL);
 		}
-		/*
-		 * Arrange to update the last rx timestamp only for
-		 * data frames; this avoids false positives caused by
-		 * hearing beacon frames from ap's on the same channel.
-		 * If the only frames we receive are management frames
-		 * when we take a bmiss then we can probably cope with
-		 * falling back to the state machine as we're not busy.
-		 */
-		if (type == IEEE80211_FC0_TYPE_DATA)
-			ngood++;
 rx_next:
 		STAILQ_INSERT_TAIL(&sc->sc_rxbuf, bf, bf_list);
 	} while (ath_rxbuf_init(sc, bf) == 0);



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