aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorVasant Hegde <hegdevasant@linux.vnet.ibm.com>2020-03-04 14:28:07 +0530
committerVasant Hegde <hegdevasant@linux.vnet.ibm.com>2020-03-20 11:22:32 +0530
commitace309c4ddf33099811172c0df4da3d2065b9442 (patch)
tree20dca2b8db73eb2f0847f81141c3995a77ee109e
parent2fc832206f2f39a6703a9c0e161bd33bcf910514 (diff)
downloadskiboot-ace309c4ddf33099811172c0df4da3d2065b9442.zip
skiboot-ace309c4ddf33099811172c0df4da3d2065b9442.tar.gz
skiboot-ace309c4ddf33099811172c0df4da3d2065b9442.tar.bz2
errorlog: Increase the severity of abnormal reboot events
[ Upstream commit daf9215c85f910043c472984683948baaf18da39 ] Currently Linux will usually call opal_cec_reboot2() in response to unrecoverable HMIs and other serious hardware errors. OPAL handles platform errors by sending an error log to the BMC / FSP and triggering a software checkstop. Sending error logs to the BMC / FSP is normally an async operation, but in this path we need to ensure that error logs are sent out before the xstop is triggered. The easiest way to do that is to escalate the severity of the generated error log from "abnormal reboot" to "panic" since we force panic logs to be send synchronusly. It's also a more accurate description of what's happening. CC: Mahesh Salgaonkar <mahesh@linux.vnet.ibm.com> Signed-off-by: Vasant Hegde <hegdevasant@linux.vnet.ibm.com> Reviewed-by: Klaus Heinrich Kiwi <klaus@linux.vnet.ibm.com> [oliver: commit message] Signed-off-by: Oliver O'Halloran <oohall@gmail.com> Signed-off-by: Vasant Hegde <hegdevasant@linux.vnet.ibm.com>
-rw-r--r--core/platform.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/core/platform.c b/core/platform.c
index 1246f84..b9ebac3 100644
--- a/core/platform.c
+++ b/core/platform.c
@@ -27,7 +27,7 @@ bool manufacturing_mode = false;
struct platform platform;
DEFINE_LOG_ENTRY(OPAL_RC_ABNORMAL_REBOOT, OPAL_PLATFORM_ERR_EVT, OPAL_CEC,
- OPAL_CEC_HARDWARE, OPAL_PREDICTIVE_ERR_FAULT_RECTIFY_REBOOT,
+ OPAL_CEC_HARDWARE, OPAL_ERROR_PANIC,
OPAL_ABNORMAL_POWER_OFF);
/*