EDAC/mc: Call edac_inc_ue_error() before panic

Zhenzhong Duan · aegl · commit e9ff6636d3f9 · 2020-06-15T11:19:52.000-07:00
By calling edac_inc_ue_error() before panic, we get a correct UE error count for core dump analysis. Signed-off-by: Zhenzhong Duan <zhenzhong.duan@gmail.com> Signed-off-by: Tony Luck <tony.luck@intel.com> Link: https://lore.kernel.org/r/20200610065846.3626-2-zhenzhong.duan@gmail.com
diff --git a/drivers/edac/edac_mc.c b/drivers/edac/edac_mc.c
@@ -950,6 +950,8 @@ static void edac_ue_error(struct edac_raw_error_desc *e)
 			e->other_detail);
 	}
 
+	edac_inc_ue_error(e);
+
 	if (edac_mc_get_panic_on_ue()) {
 		panic("UE %s%son %s (%s page:0x%lx offset:0x%lx grain:%ld%s%s)\n",
 			e->msg,
@@ -959,8 +961,6 @@ static void edac_ue_error(struct edac_raw_error_desc *e)
 			*e->other_detail ? " - " : "",
 			e->other_detail);
 	}
-
-	edac_inc_ue_error(e);
 }
 
 static void edac_inc_csrow(struct edac_raw_error_desc *e, int row, int chan)

Original file line number	Diff line number	Diff line change
`@@ -950,6 +950,8 @@ static void edac_ue_error(struct edac_raw_error_desc *e)`
`950`	`950`	`e->other_detail);`
`951`	`951`	`}`
`952`	`952`
	`953`	`+ edac_inc_ue_error(e);`
	`954`	`+`
`953`	`955`	`if (edac_mc_get_panic_on_ue()) {`
`954`	`956`	`panic("UE %s%son %s (%s page:0x%lx offset:0x%lx grain:%ld%s%s)\n",`
`955`	`957`	`e->msg,`
`@@ -959,8 +961,6 @@ static void edac_ue_error(struct edac_raw_error_desc *e)`
`959`	`961`	`*e->other_detail ? " - " : "",`
`960`	`962`	`e->other_detail);`
`961`	`963`	`}`
`962`		`-`
`963`		`- edac_inc_ue_error(e);`
`964`	`964`	`}`
`965`	`965`
`966`	`966`	`static void edac_inc_csrow(struct edac_raw_error_desc *e, int row, int chan)`