Re: e1000: "fix" it on thinkpad x60 / eeprom checksum read fails

Re: e1000: "fix" it on thinkpad x60 / eeprom checksum read fails

[Auke Kok]

Pavel Machek wrote:
> Hi!
> 
>>> e1000 in thinkpad x60 fails without this dirty hack. What to do with
>>> it?
>>>
>>> Signed-off-by: Pavel Machek <pavel@suse.cz>
>> NAK, certainly this should never be merged in any tree...
>>
>> this is a known issue that we're tracking here:
>>
>> http://sourceforge.net/tracker/index.php?func=detail&aid=1474679&group_id=42302&atid=447449
>>
>> Summary of the issue:
>>
>> Lenovo has used certain BIOS versions where ASPD/DSPD was turned on which 
>> turns the PHY off when no cable is inserted to save power. The e1000 driver 
>> already turns off this feature but can't do this until the driver is 
>> loaded. It seems that turning this feature on causes the MAC to give read 
>> errors.
>>
>> Lenovo seems to have the feature turned off in their latest BIOS versions, 
>> we encourage all people to upgrade their BIOS with the latest version from 
>> Lenovo (available from their website). It seems that for at least 2 people, 
>> this has fixed the problem.
>>
>> Inserting a cable obviously might also work :)
> 
> Hehe.
> 
>> We did reproduce the problem initially with the old BIOS (1.01-1.03) on a 
>> T60 system, but unfortunately the bug disappeared into nothingness.
>>
>> Bypassing the checksum leaves the NIC in an uncertain state and is not 
>> recommended.
> 
> Okay, perhaps this should be inserted as a comment into the driver,
> and printk should be fixed to point at this explanation?
> 
> Can't we enable the driver with the bad checksum, then read the _real_
> data?

no.

We're working on a solution where we make sure that the PHY is physically 
turned on properly before we read the EEPROM, which would be the proper fix. 
It's completely not acceptable to run when the EEPROM checksum fails - you 
might even be running with the wrong MAC address, or worse. Lets fix this the 
right way instead.

Auke

PS: adding netdev to the CC...

Re: e1000: "fix" it on thinkpad x60 / eeprom checksum read fails

[Andrew Morton]

On Thu, 20 Jul 2006 10:34:23 -0700
Auke Kok <auke-jan.h.kok@intel.com> wrote:

> > Okay, perhaps this should be inserted as a comment into the driver,
> > and printk should be fixed to point at this explanation?
> > 
> > Can't we enable the driver with the bad checksum, then read the _real_
> > data?
> 
> no.
> 
> We're working on a solution where we make sure that the PHY is physically 
> turned on properly before we read the EEPROM, which would be the proper fix. 
> It's completely not acceptable to run when the EEPROM checksum fails - you 
> might even be running with the wrong MAC address, or worse. Lets fix this the 
> right way instead.

A printk which helps the user to understand all this saga would be very nice.

Re: e1000: "fix" it on thinkpad x60 / eeprom checksum read fails

[Theodore Tso]

On Fri, Jul 21, 2006 at 06:41:05AM -0700, Andrew Morton wrote:
> > It's completely not acceptable to run when the EEPROM checksum fails - you 
> > might even be running with the wrong MAC address, or worse. Lets fix this the 
> > right way instead.
> 
> A printk which helps the user to understand all this saga would be very nice.
> -

And if someone who understands all of these details could put a note
in the thinkwiki (say, here:
http://www.thinkwiki.org/wiki/Ethernet_Controllers#Intel_Gigabit_.2810.2F100.2F1000.29)
it would be greatly appreciated.

						- Ted

Re: e1000: "fix" it on thinkpad x60 / eeprom checksum read fails

[Auke Kok]

Theodore Tso wrote:
> On Fri, Jul 21, 2006 at 06:41:05AM -0700, Andrew Morton wrote:
>>> It's completely not acceptable to run when the EEPROM checksum fails - you 
>>> might even be running with the wrong MAC address, or worse. Lets fix this the 
>>> right way instead.
>> A printk which helps the user to understand all this saga would be very nice.
>> -
> 
> And if someone who understands all of these details could put a note
> in the thinkwiki (say, here:
> http://www.thinkwiki.org/wiki/Ethernet_Controllers#Intel_Gigabit_.2810.2F100.2F1000.29)
> it would be greatly appreciated.
> 

why don't I do that :)


Andrew: I'm contemplating that printk...


Cheers,

Auke

Re: e1000: "fix" it on thinkpad x60 / eeprom checksum read fails

[Andrew Morton]

On Fri, 21 Jul 2006 08:22:38 -0700
Auke Kok <auke-jan.h.kok@intel.com> wrote:

> > And if someone who understands all of these details could put a note
> > in the thinkwiki (say, here:
> > http://www.thinkwiki.org/wiki/Ethernet_Controllers#Intel_Gigabit_.2810.2F100.2F1000.29)
> > it would be greatly appreciated.
> > 
> 
> why don't I do that :)
> 
> 
> Andrew: I'm contemplating that printk...

Print the wiki URL ;)