From owner-svn-src-head@FreeBSD.ORG Tue May 18 23:30:08 2010 Return-Path: Delivered-To: svn-src-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id DC7691065673; Tue, 18 May 2010 23:30:07 +0000 (UTC) (envelope-from bz@FreeBSD.org) Received: from mail.cksoft.de (mail.cksoft.de [IPv6:2001:4068:10::3]) by mx1.freebsd.org (Postfix) with ESMTP id 8B1638FC0C; Tue, 18 May 2010 23:30:07 +0000 (UTC) Received: from localhost (amavis.fra.cksoft.de [192.168.74.71]) by mail.cksoft.de (Postfix) with ESMTP id E5A6541C7A3; Wed, 19 May 2010 01:30:06 +0200 (CEST) X-Virus-Scanned: amavisd-new at cksoft.de Received: from mail.cksoft.de ([192.168.74.103]) by localhost (amavis.fra.cksoft.de [192.168.74.71]) (amavisd-new, port 10024) with ESMTP id 3gkwq2RUz3Xv; Wed, 19 May 2010 01:30:05 +0200 (CEST) Received: by mail.cksoft.de (Postfix, from userid 66) id BFC3341C798; Wed, 19 May 2010 01:30:05 +0200 (CEST) Received: from maildrop.int.zabbadoz.net (maildrop.int.zabbadoz.net [10.111.66.10]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by mail.int.zabbadoz.net (Postfix) with ESMTP id 1E7E54448EC; Tue, 18 May 2010 23:28:03 +0000 (UTC) Date: Tue, 18 May 2010 23:28:03 +0000 (UTC) From: "Bjoern A. Zeeb" X-X-Sender: bz@maildrop.int.zabbadoz.net To: Pyun YongHyeon In-Reply-To: <20100518231936.GH5968@michelle.cdnetworks.com> Message-ID: <20100518232729.N23815@maildrop.int.zabbadoz.net> References: <201005101714.o4AHEFuu015079@svn.freebsd.org> <20100518032628.S23815@maildrop.int.zabbadoz.net> <20100518171315.GA5968@michelle.cdnetworks.com> <20100518224555.G23815@maildrop.int.zabbadoz.net> <20100518231936.GH5968@michelle.cdnetworks.com> X-OpenPGP-Key: 0x14003F198FEFA3E77207EE8D2B58B8F83CCF1842 MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, Pyun YongHyeon Subject: Re: svn commit: r207851 - head/sys/dev/sge X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 18 May 2010 23:30:08 -0000 On Tue, 18 May 2010, Pyun YongHyeon wrote: > On Tue, May 18, 2010 at 10:53:27PM +0000, Bjoern A. Zeeb wrote: >> On Tue, 18 May 2010, Pyun YongHyeon wrote: >> >>> On Tue, May 18, 2010 at 03:27:33AM +0000, Bjoern A. Zeeb wrote: >>>> On Mon, 10 May 2010, Pyun YongHyeon wrote: >>>> >>>>> Author: yongari >>>>> Date: Mon May 10 17:14:14 2010 >>>>> New Revision: 207851 >>>>> URL: http://svn.freebsd.org/changeset/base/207851 >>>>> >>>>> Log: >>>>> Implement TSO and TSO over VLAN. Increase number of allowed >>>>> fragmentation of mbuf chain to 32 from 16 because TSO can send 64KB >>>>> sized packet which in turn requires long list of mbuf chain. Due to >>>>> lack of documentation, I'm not sure whether driver have to pull up >>>>> ethernet/IP/TCP header with options to make controller work but >>>>> driver have to parse TCP header to update pseudo TCP checksum >>>>> anyway. The controller expects pseudo TCP checksum computed by >>>>> upper stack and the checksum should follow the MS NDIS >>>>> specification to make TSO work. >>>>> >>>>> Tested by: xclin cs dot nctu dot edu dot tw > >>>> >>>> This brings in INET as a dependency for the driver and fails on my >>>> nooptions INET >>>> nooptions INET6 >>>> kernels (in_pseudo()). >>>> >>> >>> AFAIK sge(4) is not the only driver that has INET dependency. I >>> think we need IPv6 offloading framework(checksum offloading, TSO4 >> >> There is no checksum on IPv6. >> > > Yes, but controllers that computes only partial checksum for > received frames require parsing IPv4/IPv6 header to support > checksum offloading. If we have good framework we can avoid > duplicated code as well as INET/INET6 dependency. > >>> and TSO6) before removing INET dependency in various drivers. >>> Most recent ethernet controllers support IPv6 offloading and >>> FreeBSD still lacks these nice features. >> >> This is true, but we should (still) fix this and work on a better >> solution. You may remember that I tried to get this discussion >> started twice during the last 12 months. First time after I added >> the inet depency in sys/conf/files for all the other drivers that >> have the same problem(s). >> >> I will be very happy to chat with you and np@ and maybe tuexen@ for >> SCTP and Drew and Jack again to get this all sorted out. >> > > I'm also happy to go that direction. Great! > But it wouldn't be for 8.1-RELEASE, right? Right. -- Bjoern A. Zeeb (from 21) Micky Rosa: But as we've all said, this game is about the past and the future, and tonight we forget about the past. We just focus on the future.