Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 6 Mar 2008 23:12:05 GMT
From:      Sam Leffler <sam@FreeBSD.org>
To:        Perforce Change Reviews <perforce@freebsd.org>
Subject:   PERFORCE change 137034 for review
Message-ID:  <200803062312.m26NC5t5058136@repoman.freebsd.org>

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

Change 137034 by sam@sam_ebb on 2008/03/06 23:11:22

	purge 802.3 bpf taps, these happen in ether_input (we should
	never reference if_bpf in net80211, only iv_rawbpf for
	DLT_IEEE802_11 tap'ing)

Affected files ...

.. //depot/projects/vap/sys/net80211/ieee80211_hostap.c#18 edit
.. //depot/projects/vap/sys/net80211/ieee80211_input.c#29 edit

Differences ...

==== //depot/projects/vap/sys/net80211/ieee80211_hostap.c#18 (text+ko) ====

@@ -375,13 +375,6 @@
 			/* XXX only if dwds in use? */
 			ieee80211_dwds_mcast(vap, m);
 		}
-		/*
-		 * XXX If we forward packet into transmitter of the AP,
-		 * we don't need to duplicate for DLT_EN10MB.
-		 */
-		if (bpf_peers_present(ifp->if_bpf))
-			bpf_mtap(ifp->if_bpf, m);
-
 		if (ni->ni_vlan != 0) {
 			/* attach vlan tag */
 			m->m_pkthdr.ether_vtag = ni->ni_vlan;

==== //depot/projects/vap/sys/net80211/ieee80211_input.c#29 (text+ko) ====

@@ -215,13 +215,6 @@
 	/* clear driver/net80211 flags before passing up */
 	m->m_flags &= ~M_80211_RX;
 
-	/*
-	 * XXX If we forward packet into transmitter of the AP,
-	 * we don't need to duplicate for DLT_EN10MB.
-	 */
-	if (bpf_peers_present(ifp->if_bpf))
-		bpf_mtap(ifp->if_bpf, m);
-
 	if (ni->ni_vlan != 0) {
 		/* attach vlan tag */
 		m->m_pkthdr.ether_vtag = ni->ni_vlan;



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