lkml.org 
[lkml]   [2009]   [Jan]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
SubjectRe: kexec fail with e1000 and e1000e
Date
On Friday 16 January 2009, Yinghai Lu wrote:
> On Thu, Jan 15, 2009 at 8:31 PM, Yinghai Lu <yinghai@kernel.org> wrote:
> > [ 8.652468] Intel(R) PRO/1000 Network Driver - version 7.3.20-k3-NAPI
> > [ 8.658896] Copyright (c) 1999-2006 Intel Corporation.
> > [ 8.680008] e1000 0000:05:01.0: enabling device (0000 -> 0003)
> > [ 8.685831] vendor=1022 device=7458
> > [ 8.689314] e1000 0000:05:01.0: PCI INT A -> GSI 48 (level, low) -> IRQ 48
> > [ 8.696184] e1000 0000:05:01.0: enabling bus mastering
> > [ 8.953642] e1000: 0000:05:01.0: e1000_probe: The EEPROM Checksum
> > Is Not Valid
> > [ 9.863340] /*********************/
> > [ 9.866829] Current EEPROM Checksum : 0xffff
> > [ 9.871092] Calculated : 0xbaf9
> > [ 9.875354] Offset Values
> > [ 9.878230] ======== ======
> > [ 9.881107] 00000000: ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff
> > [ 9.887622] 00000010: ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff
> > [ 9.894137] 00000020: ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff
> > [ 9.900653] 00000030: ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff
> > [ 9.907168] 00000040: ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff
> > [ 9.913684] 00000050: ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff
> > [ 9.920200] 00000060: ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff
> > [ 9.926715] 00000070: ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff
> > [ 9.933231] Include this output when contacting your support provider.
> > [ 9.939746] This is not a software error! Something bad happened to
> > your hardware or
> > [ 9.947475] EEPROM image. Ignoring this problem could result in
> > further problems,
> > [ 9.954945] possibly loss of data, corruption or system hangs!
> > [ 9.960767] The MAC Address will be reset to 00:00:00:00:00:00,
> > which is invalid
> > [ 9.968149] and requires you to set the proper MAC address manually
> > before continuing
> > [ 9.975964] to enable this network device.
> > [ 9.980053] Please inspect the EEPROM dump and report the issue to
> > your hardware vendor
> > [ 9.988041] or Intel Customer Support.
> > [ 9.991770] /*********************/
> >
> > git bisect said following patch is first bad one.
> >
> > commit 98e6e286d7b01deb7453b717aa38ebb69d6cefc0
> > Author: Rafael J. Wysocki <rjw@sisk.pl>
> > Date: Wed Jan 7 13:10:35 2009 +0100
> >
> > PCI PM: Register power state of devices during initialization
> >
> > Use the observation that the power state of a PCI device can be
> > loaded into its pci_dev structure as soon as pci_pm_init() is run for
> > it and make that happen.
> >
> > Signed-off-by: Rafael J. Wysocki <rjw@sisk.pl>
> > Acked-by: Pavel Machek <pavel@suse.cz>
> > Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
>
> reverting that patch will make kexec work with e1000 again.

Well, although I think that reverting the patch won't make much harm, I also
don't quite understand why this should matter for kexec at all.

Can you please check if the problem also happens with the appended debug patch?

Rafael


---
drivers/pci/pci.c | 3 +++
1 file changed, 3 insertions(+)

Index: linux-2.6/drivers/pci/pci.c
===================================================================
--- linux-2.6.orig/drivers/pci/pci.c
+++ linux-2.6/drivers/pci/pci.c
@@ -1266,6 +1266,7 @@ void pci_pm_init(struct pci_dev *dev)
{
int pm;
u16 pmc;
+ pci_power_t state;

dev->pm_cap = 0;

@@ -1321,7 +1322,9 @@ void pci_pm_init(struct pci_dev *dev)
}

Exit:
+ state = dev->current_state;
pci_update_current_state(dev, PCI_D0);
+ dev->current_state = state;
}

/**


\
 
 \ /
  Last update: 2009-01-16 13:31    [W:0.646 / U:0.136 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site