From owner-cvs-all@FreeBSD.ORG Fri Jun 2 23:24:26 2006 Return-Path: X-Original-To: cvs-all@FreeBSD.org Delivered-To: cvs-all@FreeBSD.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 0FCC816A41F; Fri, 2 Jun 2006 23:24:26 +0000 (UTC) (envelope-from sam@errno.com) Received: from ebb.errno.com (ebb.errno.com [69.12.149.25]) by mx1.FreeBSD.org (Postfix) with ESMTP id CD44443D68; Fri, 2 Jun 2006 23:24:21 +0000 (GMT) (envelope-from sam@errno.com) Received: from [10.0.0.199] ([10.0.0.199]) (authenticated bits=0) by ebb.errno.com (8.13.6/8.12.6) with ESMTP id k52NOK6s041305 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Fri, 2 Jun 2006 16:24:21 -0700 (PDT) (envelope-from sam@errno.com) Message-ID: <4480C8A4.2020403@errno.com> Date: Fri, 02 Jun 2006 16:24:20 -0700 From: Sam Leffler Organization: Errno Consulting User-Agent: Thunderbird 1.5.0.4 (Macintosh/20060530) MIME-Version: 1.0 To: "Christian S.J. Peron" References: <200606021959.k52JxXlr022148@repoman.freebsd.org> In-Reply-To: <200606021959.k52JxXlr022148@repoman.freebsd.org> X-Enigmail-Version: 0.94.0.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/dev/ath if_ath.c src/sys/dev/ipw if_ipw.c src/sys/dev/iwi if_iwi.c src/sys/dev/ral rt2560.c rt2661.c src/sys/dev/usb if_ural.c src/sys/dev/wi if_wi.c src/sys/net bpf.c bpf.h bpfdesc.h if_disc.c if_gif.c if_gre.c if_loop.c if_sl.c ... X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 02 Jun 2006 23:24:31 -0000 For the record we can eliminate the indirect ref through if_bpf if we embed the structure in the ifnet. For ifnet's that hookup to bpf this is a no-cost improvement. The downside is that ifnet's that don't work with bpf incur the overhead for the structure despite not using it. We decided to do this change separately in case anyone has an argument against it. Comments? Sam