All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jamie webb <j@jwebb.sygneca.com>
To: Michael Chan <mchan@broadcom.com>
Cc: Andi Kleen <andi@firstfloor.org>, netdev@vger.kernel.org
Subject: Re: PROBLEM: tg3 spitting out uninitialized memory
Date: Fri, 13 Apr 2007 18:51:40 +0100	[thread overview]
Message-ID: <461FC32C.4060309@jwebb.sygneca.com> (raw)
In-Reply-To: <1176410380.11425.5.camel@dell>

Michael Chan wrote:
> On Thu, 2007-04-12 at 16:50 +0200, Andi Kleen wrote:
>> Jamie webb <j@jwebb.sygneca.com> writes:
>>
>>> Hi there
>>>
>>> I have a Dell PE860 with built-in BCM5721, which is reported as
>>> working fine with the tg3 driver, however I have been getting sporadic
>>> data corruption, mostly evident as SSH MAC errors.
>> FWIW i also saw this (data corruption with tg3) occasionally,
>> but never repeatable or with a packet dump.
>>
> My suggestion is to try turning tx checksum off (ethtool -K eth0 tx off)
> to see if it makes a difference.  I'm not aware of checksum problem on
> 5721, but it is worth trying.  See if the other end is reporting TCP
> checksum errors also.

Well, so far so good. I'll let you know if it happens again, but it 
looks like that's fixed it.

Further testing showed that I also had to disable rx checksumming, 
otherwise I was getting random kernel crashes. Presumably it was not 
only reading data from random memory locations, but also writing in the 
wrong place...

So, do I understand correctly that this is causing the CPU rather than 
the NIC to do the checksumming?

Is this a reasonable permanent solution?

Crashing aside, I'm a little nervous about putting into production a box 
that might for example randomly decide to serve up its SSL private keys 
halfway through an email message...

Cheers

/J


  reply	other threads:[~2007-04-13 17:51 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-04-11 21:49 PROBLEM: tg3 spitting out uninitialized memory Jamie webb
2007-04-12 14:50 ` Andi Kleen
2007-04-12 20:39   ` Michael Chan
2007-04-13 17:51     ` Jamie webb [this message]
2007-04-16  6:25       ` Michael Chan
2007-04-16  6:31         ` Andi Kleen
2007-04-16  6:43           ` Michael Chan

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=461FC32C.4060309@jwebb.sygneca.com \
    --to=j@jwebb.sygneca.com \
    --cc=andi@firstfloor.org \
    --cc=mchan@broadcom.com \
    --cc=netdev@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.