diff options
author | Mauro Carvalho Chehab <mchehab@redhat.com> | 2012-06-04 10:29:25 -0400 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2012-06-11 12:23:52 -0400 |
commit | 03f7eae80f4b913929be84e0c883ee98196fd6ff (patch) | |
tree | 5babfa8755aa6f421745ae7c0d29882d5da8e355 /drivers/edac/x38_edac.c | |
parent | 075f30901e32feefd3641e6c5537611fd7e27b59 (diff) |
edac: remove arch-specific parameter for the error handler
Remove the arch-dependent parameter, as it were not used,
as the MCE tracepoint weren't implemented. It probably doesn't
make sense to have an MCE-specific tracepoint, as this will
cost more bytes at the tracepoint, and tracepoint is not free.
The changes at the EDAC drivers were done by this small perl script:
$file .=$_ while (<>);
$file =~ s/(edac_mc_handle_error)\s*\(([^\;]+)\,([^\,\)]+)\s*\)/$1($2)/g;
print $file;
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'drivers/edac/x38_edac.c')
-rw-r--r-- | drivers/edac/x38_edac.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/edac/x38_edac.c b/drivers/edac/x38_edac.c index 0e1581afadaf..3d46d40987af 100644 --- a/drivers/edac/x38_edac.c +++ b/drivers/edac/x38_edac.c | |||
@@ -217,7 +217,7 @@ static void x38_process_error_info(struct mem_ctl_info *mci, | |||
217 | if ((info->errsts ^ info->errsts2) & X38_ERRSTS_BITS) { | 217 | if ((info->errsts ^ info->errsts2) & X38_ERRSTS_BITS) { |
218 | edac_mc_handle_error(HW_EVENT_ERR_UNCORRECTED, mci, 0, 0, 0, | 218 | edac_mc_handle_error(HW_EVENT_ERR_UNCORRECTED, mci, 0, 0, 0, |
219 | -1, -1, -1, | 219 | -1, -1, -1, |
220 | "UE overwrote CE", "", NULL); | 220 | "UE overwrote CE", ""); |
221 | info->errsts = info->errsts2; | 221 | info->errsts = info->errsts2; |
222 | } | 222 | } |
223 | 223 | ||
@@ -228,13 +228,13 @@ static void x38_process_error_info(struct mem_ctl_info *mci, | |||
228 | 0, 0, 0, | 228 | 0, 0, 0, |
229 | eccerrlog_row(channel, log), | 229 | eccerrlog_row(channel, log), |
230 | -1, -1, | 230 | -1, -1, |
231 | "x38 UE", "", NULL); | 231 | "x38 UE", ""); |
232 | } else if (log & X38_ECCERRLOG_CE) { | 232 | } else if (log & X38_ECCERRLOG_CE) { |
233 | edac_mc_handle_error(HW_EVENT_ERR_CORRECTED, mci, | 233 | edac_mc_handle_error(HW_EVENT_ERR_CORRECTED, mci, |
234 | 0, 0, eccerrlog_syndrome(log), | 234 | 0, 0, eccerrlog_syndrome(log), |
235 | eccerrlog_row(channel, log), | 235 | eccerrlog_row(channel, log), |
236 | -1, -1, | 236 | -1, -1, |
237 | "x38 CE", "", NULL); | 237 | "x38 CE", ""); |
238 | } | 238 | } |
239 | } | 239 | } |
240 | } | 240 | } |