From owner-cvs-all@FreeBSD.ORG Sat Sep 9 20:08:40 2006 Return-Path: X-Original-To: cvs-all@freebsd.org Delivered-To: cvs-all@freebsd.org Received: by hub.freebsd.org (Postfix, from userid 916) id 115E216A40F; Sat, 9 Sep 2006 20:08:40 +0000 (UTC) Date: Sat, 9 Sep 2006 20:08:40 +0000 From: Prafulla Deuskar To: Max Laier Message-ID: <20060909200840.GA56461@hub.freebsd.org> References: <200609090619.k896JK0m001637@repoman.freebsd.org> <200609091819.50634.max@love2party.net> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <200609091819.50634.max@love2party.net> User-Agent: Mutt/1.4.2.1i X-Operating-System: FreeBSD 6.1-STABLE on an i386 Cc: cvs-src@freebsd.org, src-committers@freebsd.org, Andre Oppermann , cvs-all@freebsd.org Subject: Re: cvs commit: src/sys/dev/em if_em.c if_em.h 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: Sat, 09 Sep 2006 20:08:40 -0000 Max Laier [max@love2party.net] wrote: > On Saturday 09 September 2006 08:19, Prafulla Deuskar wrote: > > pdeuskar 2006-09-09 06:19:20 UTC > > > > FreeBSD src repository > > > > Modified files: > > sys/dev/em if_em.c if_em.h > > Log: > > Add support for TSO. Thanks to Andre for adding support in the stack > > and Jack Vogel for driver changes. > > > > Submitted by: Jack Vogel > > This broke the build due to missing prototype for in_pseudo(). According > to Andre in the TSO thread: > > On Thursday 07 September 2006 16:02, Andre Oppermann wrote: > > The TSO code is committed. There has been a slight change with the > > ifcapabilities to differentiate between TSO for IPv4 and IPv6 which can > > be set independently. > > > > The pseudo-header checksum is always provided in m_pkthdr.csum_data, > > you don't have to compute it yourself in the driver. Problem is we require a "partial" pseudo checksum. The module build should have been fixed after I added the correct include file.