diff options
author | Michael Chan <mchan@broadcom.com> | 2007-07-19 00:49:10 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2007-07-19 00:49:10 -0400 |
commit | ee6a99b539a50b4e9398938a0a6d37f8bf911550 (patch) | |
tree | 51547e0b9c299be15cc3f54ff3c33b6e3e9c346c /drivers/net/tg3.h | |
parent | 7dc12d6dd6cc1aa489c6f3e34a75e8023c945da8 (diff) |
[TG3]: Fix msi issue with kexec/kdump.
Tina Yang <tina.yang@oracle.com> discovered an MSI related problem
when doing kdump. The problem is that the kexec kernel is booted
without going through system reset, and as a result, MSI may already
be enabled when tg3_init_one() is called. tg3_init_one() calls
pci_save_state() which will save the stale MSI state. Later on in
tg3_open(), we call pci_enable_msi() to reconfigure MSI on the chip
before we reset the chip. After chip reset, we call
pci_restore_state() which will put the stale MSI address/data back
onto the chip.
This is no longer a problem in the latest kernel because
pci_restore_state() has been changed to restore MSI state from
internal data structures which will guarantee restoring the proper
MSI state.
But I think we should still fix it. Our save and restore sequence
can still cause very subtle problems down the road. The fix is to
have our own functions save and restore precisely what we need. We
also change it to save and restore state inside tg3_chip_reset() in a
more straight forward way.
Thanks to Tina for helping to test and debug the problem.
[ Bump driver version and release date. -DaveM ]
Signed-off-by: Michael Chan <mchan@broadcom.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/tg3.h')
-rw-r--r-- | drivers/net/tg3.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/net/tg3.h b/drivers/net/tg3.h index d84e75e7365..5c21f49026c 100644 --- a/drivers/net/tg3.h +++ b/drivers/net/tg3.h | |||
@@ -2345,6 +2345,7 @@ struct tg3 { | |||
2345 | #define PHY_REV_BCM5411_X0 0x1 /* Found on Netgear GA302T */ | 2345 | #define PHY_REV_BCM5411_X0 0x1 /* Found on Netgear GA302T */ |
2346 | 2346 | ||
2347 | u32 led_ctrl; | 2347 | u32 led_ctrl; |
2348 | u32 pci_cmd; | ||
2348 | 2349 | ||
2349 | char board_part_number[24]; | 2350 | char board_part_number[24]; |
2350 | char fw_ver[16]; | 2351 | char fw_ver[16]; |