diff options
author | Dave Jiang <djiang@mvista.com> | 2007-07-19 04:49:46 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-07-19 13:04:53 -0400 |
commit | c0d121720220584bba2876b032e58a076b843fa1 (patch) | |
tree | 13ba24c6d875ded1494e1560f336b8551c663ef1 /drivers/edac/e752x_edac.c | |
parent | 28f96eeafc89643d411d54c258788a8573576127 (diff) |
drivers/edac: add new nmi rescan
Provides a way for NMI reported errors on x86 to notify the EDAC
subsystem pending ECC errors by writing to a software state variable.
Here's the reworked patch. I added an EDAC stub to the kernel so we can
have variables that are in the kernel even if EDAC is a module. I also
implemented the idea of using the chip driver to select error detection
mode via module parameter and eliminate the kernel compile option.
Please review/test. Thx!
Also, I only made changes to some of the chipset drivers since I am
unfamiliar with the other ones. We can add similar changes as we go.
Signed-off-by: Dave Jiang <djiang@mvista.com>
Signed-off-by: Douglas Thompson <dougthompson@xmission.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/edac/e752x_edac.c')
-rw-r--r-- | drivers/edac/e752x_edac.c | 14 |
1 files changed, 13 insertions, 1 deletions
diff --git a/drivers/edac/e752x_edac.c b/drivers/edac/e752x_edac.c index 8bcc887692ab..f51e79a6f891 100644 --- a/drivers/edac/e752x_edac.c +++ b/drivers/edac/e752x_edac.c | |||
@@ -22,6 +22,7 @@ | |||
22 | #include <linux/pci.h> | 22 | #include <linux/pci.h> |
23 | #include <linux/pci_ids.h> | 23 | #include <linux/pci_ids.h> |
24 | #include <linux/slab.h> | 24 | #include <linux/slab.h> |
25 | #include <linux/edac.h> | ||
25 | #include "edac_mc.h" | 26 | #include "edac_mc.h" |
26 | 27 | ||
27 | #define E752X_REVISION " Ver: 2.0.1 " __DATE__ | 28 | #define E752X_REVISION " Ver: 2.0.1 " __DATE__ |
@@ -948,6 +949,16 @@ static int e752x_probe1(struct pci_dev *pdev, int dev_idx) | |||
948 | debugf0("%s(): mci\n", __func__); | 949 | debugf0("%s(): mci\n", __func__); |
949 | debugf0("Starting Probe1\n"); | 950 | debugf0("Starting Probe1\n"); |
950 | 951 | ||
952 | /* make sure error reporting method is sane */ | ||
953 | switch(edac_op_state) { | ||
954 | case EDAC_OPSTATE_POLL: | ||
955 | case EDAC_OPSTATE_NMI: | ||
956 | break; | ||
957 | default: | ||
958 | edac_op_state = EDAC_OPSTATE_POLL; | ||
959 | break; | ||
960 | } | ||
961 | |||
951 | /* check to see if device 0 function 1 is enabled; if it isn't, we | 962 | /* check to see if device 0 function 1 is enabled; if it isn't, we |
952 | * assume the BIOS has reserved it for a reason and is expecting | 963 | * assume the BIOS has reserved it for a reason and is expecting |
953 | * exclusive access, we take care not to violate that assumption and | 964 | * exclusive access, we take care not to violate that assumption and |
@@ -1123,4 +1134,5 @@ MODULE_DESCRIPTION("MC support for Intel e752x memory controllers"); | |||
1123 | module_param(force_function_unhide, int, 0444); | 1134 | module_param(force_function_unhide, int, 0444); |
1124 | MODULE_PARM_DESC(force_function_unhide, "if BIOS sets Dev0:Fun1 up as hidden:" | 1135 | MODULE_PARM_DESC(force_function_unhide, "if BIOS sets Dev0:Fun1 up as hidden:" |
1125 | " 1=force unhide and hope BIOS doesn't fight driver for Dev0:Fun1 access"); | 1136 | " 1=force unhide and hope BIOS doesn't fight driver for Dev0:Fun1 access"); |
1126 | 1137 | module_param(edac_op_state, int, 0444); | |
1138 | MODULE_PARM_DESC(edac_op_state, "EDAC Error Reporting state: 0=Poll,1=NMI"); | ||