diff options
| author | Vasant Hegde <hegdevasant@linux.vnet.ibm.com> | 2015-08-10 11:16:43 +0530 |
|---|---|---|
| committer | Stewart Smith <stewart@linux.vnet.ibm.com> | 2015-08-14 14:58:05 +1000 |
| commit | a5299ba29b250aa9c87eb59d500fb1489c915af2 (patch) | |
| tree | b0fbf66a0931f9ce90df9ab9e6eb4887776bfaba /hw/ipmi | |
| parent | 1999773d8f2f033308a63b004649aec8d8681971 (diff) | |
| download | skiboot-a5299ba29b250aa9c87eb59d500fb1489c915af2.zip skiboot-a5299ba29b250aa9c87eb59d500fb1489c915af2.tar.gz skiboot-a5299ba29b250aa9c87eb59d500fb1489c915af2.tar.bz2 | |
IPMI: Only log events that needs attention
Presently we are logging all the events to service processor (FSP/BMC).
But on BMC machines we should only log events that requires attention.
As per PEL spec, we should log events with severity >= 0x22 and "service
action flag" is "on". But in our case, all logs OPAL originagted logs
are makred as report externally.
So lets log all events that are originated from OAPL (presently all logs
as payload is not logging any PEL event) and severity >= 0x22.
Signed-off-by: Vasant Hegde <hegdevasant@linux.vnet.ibm.com>
Cc: Ananth N Mavinakayanahalli <ananth@in.ibm.com>
Cc: Alistair Popple <alistair@popple.id.au>
Cc: Jeremy Kerr <jk@ozlabs.org>
[stewart@linux.vnet.ibm.com: s/SEL:/IPMI:/ in prlog]
Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
Diffstat (limited to 'hw/ipmi')
| -rw-r--r-- | hw/ipmi/ipmi-sel.c | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/hw/ipmi/ipmi-sel.c b/hw/ipmi/ipmi-sel.c index e672ecf..5fb1dc1 100644 --- a/hw/ipmi/ipmi-sel.c +++ b/hw/ipmi/ipmi-sel.c @@ -167,6 +167,13 @@ int ipmi_elog_commit(struct errorlog *elog_buf) { struct ipmi_msg *msg; + /* Only log events that needs attention */ + if (elog_buf->event_severity < OPAL_PREDICTIVE_ERR_FAULT_RECTIFY_REBOOT || + elog_buf->elog_origin != ORG_SAPPHIRE) { + prlog(PR_INFO, "IPMI: dropping non severe PEL event\n"); + return 0; + } + /* We pass a large request size in to mkmsg so that we have a * large enough allocation to reuse the message to pass the * PEL data via a series of partial add commands. */ |