From owner-cvs-src@FreeBSD.ORG Tue Oct 19 10:16:35 2004 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 97C7316A4CE; Tue, 19 Oct 2004 10:16:35 +0000 (GMT) Received: from fledge.watson.org (fledge.watson.org [204.156.12.50]) by mx1.FreeBSD.org (Postfix) with ESMTP id 3630343D48; Tue, 19 Oct 2004 10:16:35 +0000 (GMT) (envelope-from robert@fledge.watson.org) Received: from fledge.watson.org (localhost [127.0.0.1]) by fledge.watson.org (8.13.1/8.13.1) with ESMTP id i9JAGPHJ066077; Tue, 19 Oct 2004 06:16:25 -0400 (EDT) (envelope-from robert@fledge.watson.org) Received: from localhost (robert@localhost)i9JAGPIN066074; Tue, 19 Oct 2004 06:16:25 -0400 (EDT) (envelope-from robert@fledge.watson.org) Date: Tue, 19 Oct 2004 06:16:25 -0400 (EDT) From: Robert Watson X-Sender: robert@fledge.watson.org To: Scott Long In-Reply-To: <200410190239.i9J2dRF8029843@repoman.freebsd.org> Message-ID: 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/em if_em.c 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: Tue, 19 Oct 2004 10:16:35 -0000 Nice catch. I suspect we have a few more things like this floating around... Robert N M Watson FreeBSD Core Team, TrustedBSD Projects robert@fledge.watson.org Principal Research Scientist, McAfee Research On Tue, 19 Oct 2004, Scott Long wrote: > scottl 2004-10-19 02:39:27 UTC > > FreeBSD src repository > > Modified files: > sys/dev/em if_em.c > Log: > Use an alignment of 1 instead of PAGE_SIZE for the rx and tx buffer tags. > Since the e1000 DMA engines hava no constraints on the alignment of buffer > transfers, there is no reason to tell busdma that there is. This save a > minimum of 1 malloc call per packet, which translates to eliminating 4 locks. > It also means that buffers are not needlessly bounced when transfered. The > end result is a 38% improvement in pps in a 4 way bridging environment. > > Obtained from: Sandvine, Inc. > > Revision Changes Path > 1.48 +2 -12 src/sys/dev/em/if_em.c >