From owner-cvs-all@FreeBSD.ORG  Sun Jan 16 05:30:48 2005
Return-Path: <owner-cvs-all@FreeBSD.ORG>
Delivered-To: cvs-all@freebsd.org
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP id 8B7B816A4D0
	for <cvs-all@FreeBSD.org>; Sun, 16 Jan 2005 05:30:48 +0000 (GMT)
Received: from relay02.pair.com (relay02.pair.com [209.68.5.16])
	by mx1.FreeBSD.org (Postfix) with SMTP id BD38043D31
	for <cvs-all@FreeBSD.org>; Sun, 16 Jan 2005 05:30:47 +0000 (GMT)
	(envelope-from silby@silby.com)
Received: (qmail 7908 invoked from network); 16 Jan 2005 05:30:46 -0000
Received: from unknown (HELO localhost) (unknown)
  by unknown with SMTP; 16 Jan 2005 05:30:46 -0000
X-pair-Authenticated: 209.68.2.70
Date: Sat, 15 Jan 2005 23:30:45 -0600 (CST)
From: Mike Silbersack <silby@silby.com>
To: John-Mark Gurney <gurney_j@resnet.uoregon.edu>
In-Reply-To: <20050105001620.GZ19624@funkthat.com>
Message-ID: <20050115232853.P1020@odysseus.silby.com>
References: <200501050006.j0506FvV057370@repoman.freebsd.org>
 <20050105001620.GZ19624@funkthat.com>
MIME-Version: 1.0
Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed
cc: cvs-src@FreeBSD.org
cc: src-committers@FreeBSD.org
cc: Olivier Houchard <cognet@FreeBSD.org>
cc: cvs-all@FreeBSD.org
Subject: Re: cvs commit: src/sys/dev/re if_re.c
X-BeenThere: cvs-all@freebsd.org
X-Mailman-Version: 2.1.1
Precedence: list
List-Id: CVS commit messages for the entire tree <cvs-all.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-all>
List-Post: <mailto:cvs-all@freebsd.org>
List-Help: <mailto:cvs-all-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Sun, 16 Jan 2005 05:30:48 -0000


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