diff options
author | Mauro Carvalho Chehab <mchehab@redhat.com> | 2012-06-04 12:27:43 -0400 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2012-06-12 11:15:47 -0400 |
commit | 9eb07a7fb8a90ee39fa9d5489afc0330cfcfbea7 (patch) | |
tree | 2d034ec31225f2b82267ab3e081c4e8263e7b85f /drivers/edac/i82860_edac.c | |
parent | 03f7eae80f4b913929be84e0c883ee98196fd6ff (diff) |
edac: edac_mc_handle_error(): add an error_count parameter
In order to avoid loosing error events, it is desirable to group
error events together and generate a single trace for several identical
errors.
The trace API already allows reporting multiple errors. Change the
handle_error function to also allow that.
The changes at the drivers were made by this small script:
$file .=$_ while (<>);
$file =~ s/(edac_mc_handle_error)\s*\(([^\,]+)\,([^\,]+)\,/$1($2,$3, 1,/g;
print $file;
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'drivers/edac/i82860_edac.c')
-rw-r--r-- | drivers/edac/i82860_edac.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/edac/i82860_edac.c b/drivers/edac/i82860_edac.c index ff4f590e9b9..1faa7497151 100644 --- a/drivers/edac/i82860_edac.c +++ b/drivers/edac/i82860_edac.c | |||
@@ -109,7 +109,7 @@ static int i82860_process_error_info(struct mem_ctl_info *mci, | |||
109 | return 1; | 109 | return 1; |
110 | 110 | ||
111 | if ((info->errsts ^ info->errsts2) & 0x0003) { | 111 | if ((info->errsts ^ info->errsts2) & 0x0003) { |
112 | edac_mc_handle_error(HW_EVENT_ERR_UNCORRECTED, mci, 0, 0, 0, | 112 | edac_mc_handle_error(HW_EVENT_ERR_UNCORRECTED, mci, 1, 0, 0, 0, |
113 | -1, -1, -1, "UE overwrote CE", ""); | 113 | -1, -1, -1, "UE overwrote CE", ""); |
114 | info->errsts = info->errsts2; | 114 | info->errsts = info->errsts2; |
115 | } | 115 | } |
@@ -119,12 +119,12 @@ static int i82860_process_error_info(struct mem_ctl_info *mci, | |||
119 | dimm = mci->csrows[row]->channels[0]->dimm; | 119 | dimm = mci->csrows[row]->channels[0]->dimm; |
120 | 120 | ||
121 | if (info->errsts & 0x0002) | 121 | if (info->errsts & 0x0002) |
122 | edac_mc_handle_error(HW_EVENT_ERR_UNCORRECTED, mci, | 122 | edac_mc_handle_error(HW_EVENT_ERR_UNCORRECTED, mci, 1, |
123 | info->eap, 0, 0, | 123 | info->eap, 0, 0, |
124 | dimm->location[0], dimm->location[1], -1, | 124 | dimm->location[0], dimm->location[1], -1, |
125 | "i82860 UE", ""); | 125 | "i82860 UE", ""); |
126 | else | 126 | else |
127 | edac_mc_handle_error(HW_EVENT_ERR_UNCORRECTED, mci, | 127 | edac_mc_handle_error(HW_EVENT_ERR_UNCORRECTED, mci, 1, |
128 | info->eap, 0, info->derrsyn, | 128 | info->eap, 0, info->derrsyn, |
129 | dimm->location[0], dimm->location[1], -1, | 129 | dimm->location[0], dimm->location[1], -1, |
130 | "i82860 CE", ""); | 130 | "i82860 CE", ""); |