talos-skiboot/core/errorlog.c, branch master

FSP: Notify FSP of Platform Log ID after Host Initiated Reset Reload

2017-05-10T04:32:17+00:00

Trigging a Host Initiated Reset (when the host detects the FSP has gone out to lunch and should be rebooted), would cause "Unknown Command" messages to appear in the OPAL log. This patch implements those messages How to trigger FSP RR(HIR): $ putmemproc 300000f8 0x00000000deadbeef s1 k0:n0:s0:p00 ecmd_ppc putmemproc 300000f8 0x00000000deadbeef Log showing unknown command: / # cat /sys/firmware/opal/msglog | grep -i ,3 [ 110.232114723,3] FSP: fsp_trigger_reset() entry [ 188.431793837,3] FSP #0: Link down, starting R&R [ 464.109239162,3] FSP #0: Got XUP with no pending message ! [ 466.340598554,3] FSP-DPO: Unknown command 0xce0900 [ 466.340600126,3] FSP: Unhandled message ce0900 The message we need to handle is "Get PLID after host initiated FipS reset/reload". When the FSP comes back from HIR, it asks "hey, so, which error log explains why you rebooted me?". So, we tell it. Reported-by: Pridhiviraj Paidipeddi Signed-off-by: Stewart Smith

errorlog : Modification as per coding guidelines to make the code more legible

2016-10-25T09:10:57+00:00

Some modifications related to typo errors, alignment, case letter mismatch to add more clarity to the code. Signed-off-by: Mukesh Ojha Signed-off-by: Stewart Smith

opal/errorlog :Removes the two times inclusion of header file errorlog.h from errorlog.c

2016-03-31T21:11:09+00:00

Removes two times inclusion of the same header file errorlog.h inside errorlog.c Signed-off-by: Mukesh Ojha Signed-off-by: Stewart Smith

errorlog: Simplify log calling convention

2015-07-31T05:32:42+00:00

Remove the callback functionality from log_error() and replace it with the ability to append to a user data section, or add addtional user data sections to an error log. For multiline or otherwise complex logging the convention is now to call opal_elog_create() to obtain a errorlog buffer and use log_append_msg() and log_append_data() to append to the user data section. Additional user data sections can be added to the error log via log_add_section(). The caller is then responsible for calling log_commit(). For simple logs log_simple_error() takes care of creating and committing the error log as before. Signed-off-by: Samuel Mendoza-Jonas Reviewed-by: Vasant Hegde Signed-off-by: Stewart Smith

elog: Don't call uninitialized platform elog_commit

2015-03-31T04:41:26+00:00

Some platforms fail to initialize it and bad things ensure Signed-off-by: Benjamin Herrenschmidt Acked-by: Jeremy Kerr Signed-off-by: Stewart Smith

elog: Clean up error logging headers

2014-12-02T07:54:47+00:00

Commit cf6f4e8912d29fb89ce85c84834607065ad595a5 introduced a platform independent frontend for error logging. However it failed to move the generic parts of the fsp-elog.h header into the platform independent one, instead relying on the fact that up until now fsp-elog.h was included whenever a function needed to log errors. This patch moves the platform independent defines into the frontend header file (errorlog.h) and removes the include of the platform specific header in generic code paths. Signed-off-by: Alistair Popple Signed-off-by: Stewart Smith

fsp/elog: Make the logging backend platform specific

2014-10-22T07:00:19+00:00

OpenPOWER boxes don't have an FSP and therefore implement their own method for passing log messages to a support processor. This patch makes the logging method platform specific. Signed-off-by: Alistair Popple Reviewed-by: Joel Stanley Signed-off-by: Stewart Smith

fsp/elog: Create a logging frontend

2014-10-22T07:00:10+00:00

In order to support fsp-less machines we need to be able to log errors using a BMC or some other mechanism. Currently the error logging code is tightly coupled to the platform making it difficult to add different platforms. This patch factors out the generic parts of the error logging code in preparation for adding different logging backends. It also adds a generic mechanism for pre-allocating a specific number of objects. Signed-off-by: Alistair Popple Signed-off-by: Stewart Smith