From owner-cvs-src@FreeBSD.ORG Wed Jul 6 01:24:46 2005 Return-Path: X-Original-To: cvs-src@FreeBSD.org Delivered-To: cvs-src@FreeBSD.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 42A9416A41C; Wed, 6 Jul 2005 01:24:46 +0000 (GMT) (envelope-from thompsa@FreeBSD.org) Received: from repoman.freebsd.org (repoman.freebsd.org [216.136.204.115]) by mx1.FreeBSD.org (Postfix) with ESMTP id 29B5343D49; Wed, 6 Jul 2005 01:24:46 +0000 (GMT) (envelope-from thompsa@FreeBSD.org) Received: from repoman.freebsd.org (localhost [127.0.0.1]) by repoman.freebsd.org (8.13.1/8.13.1) with ESMTP id j661OjDs062208; Wed, 6 Jul 2005 01:24:46 GMT (envelope-from thompsa@repoman.freebsd.org) Received: (from thompsa@localhost) by repoman.freebsd.org (8.13.1/8.13.1/Submit) id j661Ojel062207; Wed, 6 Jul 2005 01:24:45 GMT (envelope-from thompsa) Message-Id: <200507060124.j661Ojel062207@repoman.freebsd.org> From: Andrew Thompson Date: Wed, 6 Jul 2005 01:24:45 +0000 (UTC) To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org X-FreeBSD-CVS-Branch: HEAD Cc: Subject: cvs commit: src/sys/net bridgestp.c if_bridge.c if_bridgevar.h X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 06 Jul 2005 01:24:46 -0000 thompsa 2005-07-06 01:24:45 UTC FreeBSD src repository Modified files: sys/net bridgestp.c if_bridge.c if_bridgevar.h Log: - 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) Revision Changes Path 1.3 +2 -2 src/sys/net/bridgestp.c 1.11 +86 -28 src/sys/net/if_bridge.c 1.4 +1 -2 src/sys/net/if_bridgevar.h