- Previously when broadcasting to N number of interfaces we would run pfil

  hooks for each outgoing interface but also run pfil hooks _N times_ on the
  bridge interface. This is changed so pfil hooks are run once for the bridge
  interface (bridge0) and then only on the outgoing interfaces in the broadcast
  loop.

- Simplify bridge_enqueue() by moving bridge_pfil() to the callers.

- Check (inet6_pfil_hook.ph_busy_count >= 0), it may be possible to have a
  packet filter hooked for only ipv6 but we were only checking if ipv4 hooks
  were busy.

- Minor optimisation for null mbuf check after bridge_pfil(), move it into the
  if-block as it couldnt possibly be null outside.

Prodded by:	mlaier
Approved by:	re (scottl), mlaier (mentor)

1 files changed, 2 insertions, 2 deletions

diff --git a/sys/net/bridgestp.c b/sys/net/bridgestp.c
index 476e026..ccb7674 100644
--- a/sys/net/bridgestp.c
+++ b/sys/net/bridgestp.c
@@ -281,7 +281,7 @@ bstp_send_config_bpdu(struct bridge_softc *sc, struct bridge_iflist *bif,
 
 	/* XXX: safe here?!? */
 	BRIDGE_UNLOCK(sc);
-	bridge_enqueue(sc, ifp, m, 0);
+	bridge_enqueue(sc, ifp, m);
 	BRIDGE_LOCK(sc);
 }
 
@@ -397,7 +397,7 @@ bstp_transmit_tcn(struct bridge_softc *sc)
 
 	/* XXX: safe here?!? */
 	BRIDGE_UNLOCK(sc);
-	bridge_enqueue(sc, ifp, m, 0);
+	bridge_enqueue(sc, ifp, m);
 	BRIDGE_LOCK(sc);
 }