Skip to content

Commit c105383

Browse files
author
Mauro Carvalho Chehab
committed
sb_edac: properly handle error count
Instead of reporting the error count via driver-specific details, use the new way provided by edac_mc_handle_error. Signed-off-by: Mauro Carvalho Chehab <[email protected]>
1 parent 00d1833 commit c105383

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

drivers/edac/sb_edac.c

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1432,8 +1432,7 @@ static void sbridge_mce_output_error(struct mem_ctl_info *mci,
14321432
* to the group of dimm's where the error may be happening.
14331433
*/
14341434
snprintf(msg, sizeof(msg),
1435-
"count:%d%s%s area:%s err_code:%04x:%04x socket:%d channel_mask:%ld rank:%d",
1436-
core_err_cnt,
1435+
"%s%s area:%s err_code:%04x:%04x socket:%d channel_mask:%ld rank:%d",
14371436
overflow ? " OVERFLOW" : "",
14381437
(uncorrected_error && recoverable) ? " recoverable" : "",
14391438
area_type,
@@ -1447,13 +1446,13 @@ static void sbridge_mce_output_error(struct mem_ctl_info *mci,
14471446
/* FIXME: need support for channel mask */
14481447

14491448
/* Call the helper to output message */
1450-
edac_mc_handle_error(tp_event, mci, 1,
1449+
edac_mc_handle_error(tp_event, mci, core_err_cnt,
14511450
m->addr >> PAGE_SHIFT, m->addr & ~PAGE_MASK, 0,
14521451
channel, dimm, -1,
14531452
optype, msg);
14541453
return;
14551454
err_parsing:
1456-
edac_mc_handle_error(tp_event, mci, 1, 0, 0, 0,
1455+
edac_mc_handle_error(tp_event, mci, core_err_cnt, 0, 0, 0,
14571456
-1, -1, -1,
14581457
msg, "");
14591458

0 commit comments

Comments
 (0)