Date: Wed, 11 Apr 2007 12:40:21 +0100 From: Rui Paulo <rpaulo@fnop.net> To: Pyun YongHyeon <yongari@FreeBSD.org> Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/dev/msk if_msk.c Message-ID: <461CC925.5090400@fnop.net> In-Reply-To: <200704110047.l3B0lTcE093763@repoman.freebsd.org> References: <200704110047.l3B0lTcE093763@repoman.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Pyun YongHyeon wrote: > yongari 2007-04-11 00:47:29 UTC > > FreeBSD src repository > > Modified files: > sys/dev/msk if_msk.c > Log: > Add work around for hardware Tx checksum offload bug in Yukon II. > Yukon II generated corrupted TCP checksum for short TCP packets > that's less than 60 bytes in size(e.g. window probe packet, pure ACK > packet etc). Padding the frame with zeros to make the frame minimum > ethernet frame size didn't work at all. Instead of dropping Tx > checksum offload support we calculate TCP checksum with S/W method > when we encounter short TCP frames. > Fortunately it seems that short UDP datagrams appear to be handled > correctly by Yukon II. > > While I'm here simplify ethernet/VLAN header size calculation logic. Thanks! :) -- Rui Paulo | PGP: F0E4 C7C7 1653 79B7 78DC DD73 64FA B2C6 CF45 1F84
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?461CC925.5090400>