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

[Mike Silbersack]

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