cvs commit: src/sys/dev/re if_re.c

Mike Silbersack silby at silby.com
Sat Jan 15 21:30:48 PST 2005


On Tue, 4 Jan 2005, John-Mark Gurney wrote:

> Olivier Houchard wrote this message on Wed, Jan 05, 2005 at 00:06 +0000:
>>   Disable checksum offloading by default. It seems to produce corrupted packets
>>   with some revisions of the chip (particularly when using multiple TX
>>   descriptors).
>
> Ummm... do you have documentation on this?  i.e. PR numbers or message
> id's, or even better, an errata from RealTek?
>
> Also, make sure you update the documentation...
>
> -- 
>  John-Mark Gurney				Voice: +1 415 225 5579

Maybe, and I'm just throwing this out there... maybe when INVARIANTS is 
compiled in, the kernel should re-check RX checksums, and print a warning 
if the card goofed.  It'd be interesting to see if that showed previously 
unknown bugs.

Mike "Silby" Silbersack


More information about the cvs-src mailing list