public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Auke Kok <auke-jan.h.kok@intel.com>
To: Molle Bestefich <molle.bestefich@gmail.com>
Cc: linux-kernel@vger.kernel.org
Subject: Re: [bug] e100 bug: checksum mismatch on 82551ER rev10
Date: Mon, 10 Jul 2006 10:20:26 -0700	[thread overview]
Message-ID: <44B28C5A.5090605@intel.com> (raw)
In-Reply-To: <62b0912f0607100945o574dcce8w8f734e5828bdcaa8@mail.gmail.com>

Molle Bestefich wrote:
> Auke Kok wrote:
>> If you have received a motherboard or card with a broken EEPROM then 
>> your card
>> is in a limbo state - it might work but results are unreliable and may 
>> cause
>> your entire system to break (and even data corruption).
>>
>> You should contact the hardware vendor and have the board replaced or 
>> upgraded
>> with a proper EEPROM. Continuing to work with the corrupted EEPROM 
>> image that
>> you have now can seriously hurt you later on.
> 
> Every single IP130 I've had my hands on has had an EEPROM that the
> Linux driver declared bad.

that means that whoever is selling you the IP130's is consistently putting on 
bad EEPROMs, which is *very* bad. Which vendor is that? They can fix this 
problem for you and for *everyone* else they have sold and will sell IP130's 
to in the future.

> I'm afraid that it's not the board that's at fault, it's the driver.

No it is not. The NIC is supported (you can even call Intel for first line 
support) but if your vendor put a bad EEPROM image on it then all bets are 
off.  Intel provides the vendors with the proper tools to make valid EEPROMs, 
the driver checks them for a very good reason.

> The NICs are working perfectly.

How can you tell? Do you know if jumbo frames work correctly?  Is the device 
properly checksumming? is flow control working properly?  These and many, many 
more settings are determined by the EEPROM.  Seemingly it may work correctly, 
but there is no guarantee whatsoever that it will work correctly at all if the 
checksum is bad.  Again, you can lose data, or worse, you could corrupt memory 
in the system causing massive failure (DMA timings, etc). Unlikely? sure, but 
not impossible.

> (Also, it seems mighty odd to refuse to drive the hardware based on an
> EEPROM checksum failure, when the e100 driver will happily load for a
> device where for example IRQ routing is broken.  Just another
> indication that erroring out in this situation is overkill.)

That is another discussion.  All wifi drivers bail out if the firmware is 
corrupted, why shouldn't e1000 be allowed to do so either? Are you willing to 
risk your data?

Cheers,

Auke

  reply	other threads:[~2006-07-10 17:21 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-07-09 11:34 [bug] e100 bug: checksum mismatch on 82551ER rev10 Molle Bestefich
2006-07-10 15:25 ` Auke Kok
2006-07-10 16:45   ` Molle Bestefich
2006-07-10 17:20     ` Auke Kok [this message]
2006-07-10 17:41       ` Molle Bestefich
2006-07-10 17:58         ` Auke Kok
  -- strict thread matches above, loose matches on Subject: below --
2006-07-31 21:07 Charlie Brady

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=44B28C5A.5090605@intel.com \
    --to=auke-jan.h.kok@intel.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=molle.bestefich@gmail.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox