Set environment variable for PDBG_DTB

Override the default PDBG library device tree by setting a device tree
using the PDBG_DTB environment variable.

Signed-off-by: Ben Tyner <ben.tyner@ibm.com>
Ch

Set environment variable for PDBG_DTB

Override the default PDBG library device tree by setting a device tree
using the PDBG_DTB environment variable.

Signed-off-by: Ben Tyner <ben.tyner@ibm.com>
Change-Id: I012f48cbc3261be95ca47ffd434aa7ebf7d1ed61

show more ...

Attn: Change dump request parameter to integer

The dump request interface is changing from requiring the event log id
to be passed as a string to requiring the event log id to be passed as a
64-bit

Attn: Change dump request parameter to integer

The dump request interface is changing from requiring the event log id
to be passed as a string to requiring the event log id to be passed as a
64-bit integer.

Signed-off-by: Ben Tyner <ben.tyner@ibm.com>
Change-Id: I2862ab51cf707b2e6e1a7d3cd9009b9102b70c61

show more ...

Attn: Do not always trace hostboot registers

Only trace hostboot registers if we are handling a checkstop or TI.

Signed-off-by: Ben Tyner <ben.tyner@ibm.com>
Change-Id: I66f234b424dbd36e8789c64c9a9

Attn: Do not always trace hostboot registers

Only trace hostboot registers if we are handling a checkstop or TI.

Signed-off-by: Ben Tyner <ben.tyner@ibm.com>
Change-Id: I66f234b424dbd36e8789c64c9a9e82a3a0e75d44

show more ...

Attn: Delay start until after CFAM override

Avoid potential conflicts between CFAM override DBus target and the
attention handler by not starting attention handler before/during CFAM
override DBus t

Attn: Delay start until after CFAM override

Avoid potential conflicts between CFAM override DBus target and the
attention handler by not starting attention handler before/during CFAM
override DBus target start.

Signed-off-by: Ben Tyner <ben.tyner@ibm.com>
Change-Id: Iba4e5775dbb3b4a5b4aeff5c904c19ffab7b2e04

show more ...

pdbg v3.3 related changes

Separate sbe api into new header libpdbg_sbe.h related
changes

Signed-off-by: Jayanth Othayoth <ojayanth@in.ibm.com>
Change-Id: I4428d3a3d34d3032ff14e03b8d66c3d7db2db632

Attn: Fix attn handler crash when cfam read fails

When attention handler encounters an error while handling attentions it
will generate a PEL to track these errors. There was a bug in the code
causi

Attn: Fix attn handler crash when cfam read fails

When attention handler encounters an error while handling attentions it
will generate a PEL to track these errors. There was a bug in the code
causing attention handler to think it was handling a TI and try to
determine the subsystem (hostboot or phyp) which is only valid in a real
TI PEL case. The result was a stoi() on an invalid subsystem id.

Signed-off-by: Ben Tyner <ben.tyner@ibm.com>
Change-Id: Icc1cc4762a38854f14f3fe992fa645ccfed4c8f1

show more ...

Attn: Dump request and monitoring support

Support for requesting a dump from the dump manager and monitoring the
dump progress.

Signed-off-by: Ben Tyner <ben.tyner@ibm.com>
Change-Id: I03acd2b8c13c

Attn: Dump request and monitoring support

Support for requesting a dump from the dump manager and monitoring the
dump progress.

Signed-off-by: Ben Tyner <ben.tyner@ibm.com>
Change-Id: I03acd2b8c13ce0b5ced63ff7fbb32f7225467aee

show more ...

ATTN: Add hostboot debug regs during chkstop and TIs

Signed-off-by: Brian Stegmiller <steg2021@linux.vnet.ibm.com>
Change-Id: Ie4f35f169ce8ddefc48f7e47e1987fc12cfe8cd6

Attn: Remove --defaultbreakpoint add --defaultti

In order to support breakpoint handling and TI handling on systems that
did not support the get TI info interface the --defaultbreakpoint
configurati

Attn: Remove --defaultbreakpoint add --defaultti

In order to support breakpoint handling and TI handling on systems that
did not support the get TI info interface the --defaultbreakpoint
configuration option was added. This switch is no longer needed and
breakpoint handling is now the default action for available but invalid
TI info data. A new option --defaultti was added to force handling of
TI's when TI info is available but not valid.

Signed-off-by: Ben Tyner <ben.tyner@ibm.com>
Change-Id: I4bdf90dcbbf20b3428a1cef6f1a71ec32b3ca238

show more ...

Attn: Add support for raw pel symptom-id

Update raw pel symptom-id based on TI info data.

Signed-off-by: Ben Tyner <ben.tyner@ibm.com>
Change-Id: I501938e0fc9dba189999ab1491d23acd15628068

Attn: Submit event logs before transitioning host

For terminating events we will generate and submit the events logs
before we request to transition the host.

Signed-off-by: Ben Tyner <ben.tyner@ib

Attn: Submit event logs before transitioning host

For terminating events we will generate and submit the events logs
before we request to transition the host.

Signed-off-by: Ben Tyner <ben.tyner@ibm.com>
Change-Id: Ie600bbdbb34eb18154ea080fe1ac7d1861691e04

show more ...

Attn: Determine TI source by host running state

In cases where the TI info is not available use the host running state
to make an educated guess as to the source of the TI.

Signed-off-by: Ben Tyner

Attn: Determine TI source by host running state

In cases where the TI info is not available use the host running state
to make an educated guess as to the source of the TI.

Signed-off-by: Ben Tyner <ben.tyner@ibm.com>
Change-Id: I95c59ecad90b3aec417a64df4eac0ac5cb50dc09

show more ...

Attn: Procedure specific error codes in error logs

When creating error logs for attention handler failures an error code is
presented in the error log. Make these specific to the attention handler
p

Attn: Procedure specific error codes in error logs

When creating error logs for attention handler failures an error code is
presented in the error log. Make these specific to the attention handler
procedure that encountered the error.

Signed-off-by: Ben Tyner <ben.tyner@ibm.com>
Change-Id: Ic65343c2cc231ad6340cb5182a1fa75da7c76bcb

show more ...

Attn: Limit trace length messages

Limit trace length messages specifically for exception messages which
tend to be long.

Signed-off-by: Ben Tyner <ben.tyner@ibm.com>
Change-Id: I50360121541c92b732f

Attn: Limit trace length messages

Limit trace length messages specifically for exception messages which
tend to be long.

Signed-off-by: Ben Tyner <ben.tyner@ibm.com>
Change-Id: I50360121541c92b732ffe4502c5084497ebb0357

show more ...

Attn: Treat HB primary src as hex not ascii

When retrieving the HB primary src from TI info the field needs to be
treated as hex (versus ascii in hypervisor case).

Signed-off-by: Ben Tyner <ben.tyn

Attn: Treat HB primary src as hex not ascii

When retrieving the HB primary src from TI info the field needs to be
treated as hex (versus ascii in hypervisor case).

Signed-off-by: Ben Tyner <ben.tyner@ibm.com>
Change-Id: I2f9bfb7a04693d5435e242381025eb15a1177641

show more ...

Attn: Cleanup trace messages 2

Replaced some uses of stringstream with std::string

Signed-off-by: Ben Tyner <ben.tyner@ibm.com>
Change-Id: I15118d47b8a9ad16dad96494136cbae9b08a6f43

Attn: Add some sanity checking for TI info pointer

Add some nullptr checks for TI info. An earlier commit should ensure
that this pointer is never null however during debug it proved useful
to verif

Attn: Add some sanity checking for TI info pointer

Add some nullptr checks for TI info. An earlier commit should ensure
that this pointer is never null however during debug it proved useful
to verify this in a few places.

Signed-off-by: Ben Tyner <ben.tyner@ibm.com>
Change-Id: Idae4a9dd22d61bbac5e76f0a79059efb69981e03

show more ...

Attn: Provide default TI info for hypervisor TI

If TI info is not available for some reason we provide default TI info
data so that we can be consistent in our error logging.

Signed-off-by: Ben Tyn

Attn: Provide default TI info for hypervisor TI

If TI info is not available for some reason we provide default TI info
data so that we can be consistent in our error logging.

Signed-off-by: Ben Tyner <ben.tyner@ibm.com>
Change-Id: Ia7768a850400e96904f8bba2a9f9335bef4098be

show more ...

Attn: Fix CI complaint of unused-results

Use the results provided by file read/write operations. For mismatch in
number of bytes written on ffdc data creation we add a trace message,
for mismatch in

Attn: Fix CI complaint of unused-results

Use the results provided by file read/write operations. For mismatch in
number of bytes written on ffdc data creation we add a trace message,
for mismatch in number of bytes read during pel creation we skip
creating the pel.

Signed-off-by: Ben Tyner <ben.tyner@ibm.com>
Change-Id: I3ec6755ef258ac869775b08afafae324a76a2884

show more ...

Attn: Cleanup trace messages

Remove occurrances of ss.str().c_str() and use std::string in lieu of
stringstream when stringstream conveniences are not needed.

Signed-off-by: Ben Tyner <ben.tyner@ib

Attn: Cleanup trace messages

Remove occurrances of ss.str().c_str() and use std::string in lieu of
stringstream when stringstream conveniences are not needed.

Signed-off-by: Ben Tyner <ben.tyner@ibm.com>
Change-Id: Ie80a64a0d63eb4a24ef905c72b2dbe1390e9bacc

show more ...

Attn: Refactor dbus calls for pel handling

Removed hard-coded system names and instead find system name by dbus
object path and interface. Add try/catch handling around dbus calls that
could throw e

Attn: Refactor dbus calls for pel handling

Removed hard-coded system names and instead find system name by dbus
object path and interface. Add try/catch handling around dbus calls that
could throw exceptions.

Signed-off-by: Ben Tyner <ben.tyner@ibm.com>
Change-Id: I9cfe5fce858ba690fc9a7f0e61a4d249d5a1ef84

show more ...

Attn: Fix non-null-terminated string in ti handler

When copying ascii data from ti info data the resulting string was not
being null-terminated.

Signed-off-by: Ben Tyner <ben.tyner@ibm.com>
Change-

Attn: Fix non-null-terminated string in ti handler

When copying ascii data from ti info data the resulting string was not
being null-terminated.

Signed-off-by: Ben Tyner <ben.tyner@ibm.com>
Change-Id: I5fc9da3faf34e2760d3353b22fbcb97ba3e553e3

show more ...

Attn: Add TI info additional data to pel ffdc

Some TI events provide additional data (up to 944 bytes) beyond the base
80 bytes of standardized data.

Signed-off-by: Ben Tyner <ben.tyner@ibm.com>
Ch

Attn: Add TI info additional data to pel ffdc

Some TI events provide additional data (up to 944 bytes) beyond the base
80 bytes of standardized data.

Signed-off-by: Ben Tyner <ben.tyner@ibm.com>
Change-Id: Iefdf7c1b57575bc7d8692893ad9693d346674956

show more ...

Attn: Code cleanup based on CI build suggestions

Lessened the scope of some variables, made explicit a constructor,
changed a variable name that was shadowing a function name, removed some
unused co

Attn: Code cleanup based on CI build suggestions

Lessened the scope of some variables, made explicit a constructor,
changed a variable name that was shadowing a function name, removed some
unused code.

Signed-off-by: Ben Tyner <ben.tyner@ibm.com>
Change-Id: I430f6dece8b606318a0ffa476bf06f9a0060f72d

show more ...

Attn: Change mpipl target to crash@0 target

To start MPIPL we need to start obmc-host-crash@0.target (changed from
obmc-host-diagnostic-mode@0.target).

Signed-off-by: Ben Tyner <ben.tyner@ibm.com>

Attn: Change mpipl target to crash@0 target

To start MPIPL we need to start obmc-host-crash@0.target (changed from
obmc-host-diagnostic-mode@0.target).

Signed-off-by: Ben Tyner <ben.tyner@ibm.com>
Change-Id: I41294357a3f08ea41b320a3b892045f6f0c81f78

show more ...