netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* RE: e1000: driver reboot/kexec bug.
@ 2005-02-17 17:02 Venkatesan, Ganesh
  2005-02-17 17:57 ` Eric W. Biederman
  0 siblings, 1 reply; 6+ messages in thread
From: Venkatesan, Ganesh @ 2005-02-17 17:02 UTC (permalink / raw)
  To: eric; +Cc: jgarzik, Chilakala, Mallikarjuna, netdev

Hi Eric:

Could you send us 'lspci -vvv' output for 3:4.0? Could you also help us
with some more information on the platform you are using?

Thanks,
Ganesh.

>-----Original Message-----
>From: eric@ebiederm.dsl.xmission.com
>[mailto:eric@ebiederm.dsl.xmission.com] On Behalf Of Eric W. Biederman
>Sent: Wednesday, February 16, 2005 4:26 AM
>To: Chilakala, Mallikarjuna
>Cc: jgarzik@pobox.com; netdev
>Subject: e1000: driver reboot/kexec bug.
>
>
>When I kexec a new kernel on hardware that includes
>some revs of the e1000 (see below for lscpi -n) the
>e1000 driver is not able to reinitialize the NIC.  I
>have seen this in both 2.4.29 and 2.6.10.
>
>Tracking it down it appears to be some side effect to powering down
>the nic.  If I remove the pci_set_power_state call in e1000_suspend
>or I simply apply the attached patch so I get that affect when
>rebooting everything works.  pci_enable_device brings the device
>up to full power before the driver initialization code does anything
>else so I don't have a clue what is really going on but it is.
>
>
>Boot messages on failure:
>> Intel(R) PRO/1000 Network Driver - version 5.6.10.1-k1
>> Copyright (c) 1999-2004 Intel Corporation.
>> PCI: Enabling device 03:04.0 (0000 -> 0003)
>> e1000: 03:04.0: e1000_probe: The EEPROM Checksum Is Not Valid
>> PCI: Enabling device 03:04.1 (0000 -> 0003)
>> e1000: 03:04.1: e1000_probe: The EEPROM Checksum Is Not Valid
>
>lspci -n of the problem onboard e1000 NIC.
>> 03:04.0 Class 0200: 8086:1079 (rev 03)
>> 03:04.1 Class 0200: 8086:1079 (rev 03)
>
>
>Patch which avoids the problem.
>diff -uNrX linux-exclude-files linux-2.4.29-kexec-apic-virtwire-on-
>shutdownx86_64/drivers/net/e1000/e1000_main.c linux-2.4.29-
>kexec7.build.x86_64/drivers/net/e1000/e1000_main.c
>--- linux-2.4.29-kexec-apic-virtwire-on-
>shutdownx86_64/drivers/net/e1000/e1000_main.c	Tue Feb 15 14:17:09 2005
>+++ linux-2.4.29-kexec7.build.x86_64/drivers/net/e1000/e1000_main.c
Wed
>Feb 16 04:58:18 2005
>@@ -2777,7 +2777,7 @@
> 	case SYS_POWER_OFF:
> 		while((pdev = pci_find_device(PCI_ANY_ID, PCI_ANY_ID,
pdev))) {
> 			if(pci_dev_driver(pdev) == &e1000_driver)
>-				e1000_suspend(pdev, 3);
>+				e1000_suspend(pdev, (event ==
SYS_DOWN)?0:3);
> 		}
> 	}
> 	return NOTIFY_DONE;
>
>
>Any help to track down why this is happening so we can apply
>a clean fix would be appreciated.
>
>Eric

^ permalink raw reply	[flat|nested] 6+ messages in thread
* RE: e1000: driver reboot/kexec bug.
@ 2005-02-17 18:33 Venkatesan, Ganesh
  2005-02-17 19:53 ` Eric W. Biederman
  0 siblings, 1 reply; 6+ messages in thread
From: Venkatesan, Ganesh @ 2005-02-17 18:33 UTC (permalink / raw)
  To: Eric W. Biederman; +Cc: jgarzik, Chilakala, Mallikarjuna, netdev

>Do you know enough about kexec to attempt to reproduce this problem
>that way?

Not much. All I have is an old paper by Andy Pfiffer. Could you point me
to more resources on this?

Ganesh.

^ permalink raw reply	[flat|nested] 6+ messages in thread
* RE: e1000: driver reboot/kexec bug.
@ 2005-02-17 17:04 Venkatesan, Ganesh
  0 siblings, 0 replies; 6+ messages in thread
From: Venkatesan, Ganesh @ 2005-02-17 17:04 UTC (permalink / raw)
  To: Venkatesan, Ganesh, eric; +Cc: jgarzik, Chilakala, Mallikarjuna, netdev

Eric:

Please send me 'ethtool -e eth?' for the interface corresponding to one
of the 8086:1079 devices.

Ganesh.

>-----Original Message-----
>From: Venkatesan, Ganesh
>Sent: Thursday, February 17, 2005 9:02 AM
>To: 'eric@ebiederm.dsl.xmission.com'
>Cc: jgarzik@pobox.com; Chilakala, Mallikarjuna; netdev@oss.sgi.com
>Subject: RE: e1000: driver reboot/kexec bug.
>
>Hi Eric:
>
>Could you send us 'lspci -vvv' output for 3:4.0? Could you also help us
>with some more information on the platform you are using?
>
>Thanks,
>Ganesh.
>
>>-----Original Message-----
>>From: eric@ebiederm.dsl.xmission.com
>>[mailto:eric@ebiederm.dsl.xmission.com] On Behalf Of Eric W. Biederman
>>Sent: Wednesday, February 16, 2005 4:26 AM
>>To: Chilakala, Mallikarjuna
>>Cc: jgarzik@pobox.com; netdev
>>Subject: e1000: driver reboot/kexec bug.
>>
>>
>>When I kexec a new kernel on hardware that includes
>>some revs of the e1000 (see below for lscpi -n) the
>>e1000 driver is not able to reinitialize the NIC.  I
>>have seen this in both 2.4.29 and 2.6.10.
>>
>>Tracking it down it appears to be some side effect to powering down
>>the nic.  If I remove the pci_set_power_state call in e1000_suspend
>>or I simply apply the attached patch so I get that affect when
>>rebooting everything works.  pci_enable_device brings the device
>>up to full power before the driver initialization code does anything
>>else so I don't have a clue what is really going on but it is.
>>
>>
>>Boot messages on failure:
>>> Intel(R) PRO/1000 Network Driver - version 5.6.10.1-k1
>>> Copyright (c) 1999-2004 Intel Corporation.
>>> PCI: Enabling device 03:04.0 (0000 -> 0003)
>>> e1000: 03:04.0: e1000_probe: The EEPROM Checksum Is Not Valid
>>> PCI: Enabling device 03:04.1 (0000 -> 0003)
>>> e1000: 03:04.1: e1000_probe: The EEPROM Checksum Is Not Valid
>>
>>lspci -n of the problem onboard e1000 NIC.
>>> 03:04.0 Class 0200: 8086:1079 (rev 03)
>>> 03:04.1 Class 0200: 8086:1079 (rev 03)
>>
>>
>>Patch which avoids the problem.
>>diff -uNrX linux-exclude-files linux-2.4.29-kexec-apic-virtwire-on-
>>shutdownx86_64/drivers/net/e1000/e1000_main.c linux-2.4.29-
>>kexec7.build.x86_64/drivers/net/e1000/e1000_main.c
>>--- linux-2.4.29-kexec-apic-virtwire-on-
>>shutdownx86_64/drivers/net/e1000/e1000_main.c	Tue Feb 15 14:17:09 2005
>>+++ linux-2.4.29-kexec7.build.x86_64/drivers/net/e1000/e1000_main.c
Wed
>>Feb 16 04:58:18 2005
>>@@ -2777,7 +2777,7 @@
>> 	case SYS_POWER_OFF:
>> 		while((pdev = pci_find_device(PCI_ANY_ID, PCI_ANY_ID,
pdev))) {
>> 			if(pci_dev_driver(pdev) == &e1000_driver)
>>-				e1000_suspend(pdev, 3);
>>+				e1000_suspend(pdev, (event ==
SYS_DOWN)?0:3);
>> 		}
>> 	}
>> 	return NOTIFY_DONE;
>>
>>
>>Any help to track down why this is happening so we can apply
>>a clean fix would be appreciated.
>>
>>Eric

^ permalink raw reply	[flat|nested] 6+ messages in thread
* [PATCH net-drivers-2.6 0/10] e1000: driver update
@ 2005-02-15 21:24 Malli Chilakala
  2005-02-16 12:25 ` e1000: driver reboot/kexec bug Eric W. Biederman
  0 siblings, 1 reply; 6+ messages in thread
From: Malli Chilakala @ 2005-02-15 21:24 UTC (permalink / raw)
  To: jgarzik@pobox.com; +Cc: netdev

Signed-off-by: Mallikarjuna R Chilakala <mallikarjuna.chilakala@intel.com>
Signed-off-by: Ganesh Venkatesan <ganesh.venkatesan@intel.com>
Signed-off-by: John Ronciak <john.ronciak@intel.com>
1 Robert Olsson's fix and refinement to the poll routine
2 use netif_poll_{enable|disable}
3 Avoid race between e1000_watchdog and e1000_clean_tx_irq
4 Delay clean-up of last Tx buffer
5 Fix WOL settings in 82544 based adapters
6 Patch from Peter Kjellstroem -- fix lockup with 82547
7 Checks for desc ring/rx data bufs spanning 64k boundary
8 Report failure code when loopback test fails
9 Fixes related to Cable length estimation
10 Driver version white space, device id, comments & other

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2005-02-17 19:53 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2005-02-17 17:02 e1000: driver reboot/kexec bug Venkatesan, Ganesh
2005-02-17 17:57 ` Eric W. Biederman
  -- strict thread matches above, loose matches on Subject: below --
2005-02-17 18:33 Venkatesan, Ganesh
2005-02-17 19:53 ` Eric W. Biederman
2005-02-17 17:04 Venkatesan, Ganesh
2005-02-15 21:24 [PATCH net-drivers-2.6 0/10] e1000: driver update Malli Chilakala
2005-02-16 12:25 ` e1000: driver reboot/kexec bug Eric W. Biederman

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).