From owner-cvs-all@FreeBSD.ORG Mon Dec 6 04:22:59 2004 Return-Path: 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 0BF7916A4CE; Mon, 6 Dec 2004 04:22:59 +0000 (GMT) Received: from ylpvm29.prodigy.net (ylpvm29-ext.prodigy.net [207.115.57.60]) by mx1.FreeBSD.org (Postfix) with ESMTP id 85EF643D41; Mon, 6 Dec 2004 04:22:56 +0000 (GMT) (envelope-from nate@root.org) Received: from [10.0.0.115] (adsl-64-171-184-243.dsl.snfc21.pacbell.net [64.171.184.243])iB64Ms8r027136; Sun, 5 Dec 2004 23:22:54 -0500 Message-ID: <41B3DE9E.3080307@root.org> Date: Sun, 05 Dec 2004 20:22:54 -0800 From: Nate Lawson User-Agent: Mozilla Thunderbird 1.0RC1 (X11/20041205) X-Accept-Language: en-us, en MIME-Version: 1.0 To: Alan Cox References: <20041206004349.8F02916A581@hub.freebsd.org> In-Reply-To: <20041206004349.8F02916A581@hub.freebsd.org> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit cc: cvs-src@FreeBSD.org cc: src-committers@FreeBSD.org cc: cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/conf files src/sys/kern kern_subr.c uipc_jumbo.c src/sys/pci if_ti.c src/sys/sys jumbo.h X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.1 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: Mon, 06 Dec 2004 04:22:59 -0000 Alan Cox wrote: > alc 2004-12-06 00:43:40 UTC > > FreeBSD src repository > > Modified files: > sys/conf files > sys/kern kern_subr.c > sys/pci if_ti.c > Removed files: > sys/kern uipc_jumbo.c > sys/sys jumbo.h > Log: > Update the Tigon 1 and 2 driver to use the sf_buf API for implementing > zero-copy receive of jumbo frames. This eliminates the need for the > jumbo frame allocator implemented in kern/uipc_jumbo.c and sys/jumbo.h. > Remove it. > > Note: Zero-copy receive of jumbo frames did not work without these changes; > I believe there was insufficient locking on the jumbo vm object. > > Tested by: ken@ > Discussed with: gallatin@ > > Revision Changes Path > 1.973 +0 -1 src/sys/conf/files > 1.93 +4 -6 src/sys/kern/kern_subr.c > 1.12 +0 -254 src/sys/kern/uipc_jumbo.c (dead) > 1.98 +23 -15 src/sys/pci/if_ti.c > 1.6 +0 -59 src/sys/sys/jumbo.h (dead) Excellent, thanks for doing this. Having a generic zero-copy API was something we had needed for a long time. Perhaps cam_periph_mapmem() (sys/cam/cam_periph.c) could also benefit? -Nate