From owner-cvs-src@FreeBSD.ORG Sun Dec 28 14:07:12 2003 Return-Path: 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 CEE6A16A4CE for ; Sun, 28 Dec 2003 14:07:12 -0800 (PST) Received: from rootlabs.com (root.org [67.118.192.226]) by mx1.FreeBSD.org (Postfix) with SMTP id 0644443D5E for ; Sun, 28 Dec 2003 14:07:04 -0800 (PST) (envelope-from nate@rootlabs.com) Received: (qmail 9983 invoked by uid 1000); 28 Dec 2003 22:07:05 -0000 Date: Sun, 28 Dec 2003 14:07:05 -0800 (PST) From: Nate Lawson To: Sam Leffler In-Reply-To: <20031228035615.89C5316A4ED@hub.freebsd.org> Message-ID: <20031228140636.S9675@root.org> References: <20031228035615.89C5316A4ED@hub.freebsd.org> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII cc: cvs-src@FreeBSD.org cc: src-committers@FreeBSD.org cc: cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/dev/ath if_ath.c src/sys/dev/iicbus if_ic.c src/sys/dev/ppbus if_plip.c src/sys/dev/wi if_wi.c src/sys/net bpf.c bpf.h if_disc.c if_ef.c if_faith.c if_gif.c i X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.1 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: Sun, 28 Dec 2003 22:07:12 -0000 On Sat, 27 Dec 2003, Sam Leffler wrote: > Modified files: > sys/dev/ath if_ath.c > sys/dev/iicbus if_ic.c > sys/dev/ppbus if_plip.c > sys/dev/wi if_wi.c > sys/net bpf.c bpf.h if_disc.c if_ef.c if_faith.c > if_gif.c if_gre.c if_loop.c if_stf.c > if_tun.c > sys/netinet ip_gre.c > sys/netgraph ng_iface.c > Log: > o eliminate widespread on-stack mbuf use for bpf by introducing > a new bpf_mtap2 routine that does the right thing for an mbuf > and a variable-length chunk of data that should be prepended. > o while we're sweeping the drivers, use u_int32_t uniformly when > when prepending the address family (several places were assuming > sizeof(int) was 4) > o return M_ASSERTVALID to BPF_MTAP* now that all stack-allocated > mbufs have been eliminated; this may better be moved to the bpf > routines FYI, we're preferring the C99 uint32_t to u_int32_t. -Nate