Date: Tue, 18 May 2010 22:53:27 +0000 (UTC) From: "Bjoern A. Zeeb" <bz@FreeBSD.org> To: Pyun YongHyeon <pyunyh@gmail.com> Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, Pyun YongHyeon <yongari@freebsd.org> Subject: Re: svn commit: r207851 - head/sys/dev/sge Message-ID: <20100518224555.G23815@maildrop.int.zabbadoz.net> In-Reply-To: <20100518171315.GA5968@michelle.cdnetworks.com> References: <201005101714.o4AHEFuu015079@svn.freebsd.org> <20100518032628.S23815@maildrop.int.zabbadoz.net> <20100518171315.GA5968@michelle.cdnetworks.com>
index | next in thread | previous in thread | raw e-mail
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 <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. > 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. /bz -- 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.home | help
Want to link to this message? Use this
URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20100518224555.G23815>