clang-format: re-format for clang-18

clang-format-18 isn't compatible with the clang-format-17 output, so we
need to reformat the code with the latest version. The way clang-18
handles lambda forma

clang-format: re-format for clang-18

clang-format-18 isn't compatible with the clang-format-17 output, so we
need to reformat the code with the latest version. The way clang-18
handles lambda formatting also changed, so we have made changes to the
organization default style format to better handle lambda formatting.

See I5e08687e696dd240402a2780158664b7113def0e for updated style.
See Iea0776aaa7edd483fa395e23de25ebf5a6288f71 for clang-18 enablement.

Change-Id: I1d204ff0a167c43688f2217e9dee5504c71cd4f0
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>

show more ...

clang-format: copy latest and re-format

clang-format-16 has some backwards incompatible changes that require
additional settings for best compatibility and re-running the formatter.
Copy the latest

clang-format: copy latest and re-format

clang-format-16 has some backwards incompatible changes that require
additional settings for best compatibility and re-running the formatter.
Copy the latest .clang-format from the docs repository and reformat the
repository.

Change-Id: I27b0d1357211259edb6ec2776924729052f238d6
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>

show more ...

Add SBE hreset support to SBE attention handler

When a SBE attention is being handled we will
request a SBE hreset and try to clear the SBE
attentions. If this is not successful we will
request a SB

Add SBE hreset support to SBE attention handler

When a SBE attention is being handled we will
request a SBE hreset and try to clear the SBE
attentions. If this is not successful we will
request a SBE dump and host re-IPL.

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

show more ...

Add power fault check to checkstop handler

If a power fault is being reported during a
checkstop condition we will not call the
analyzer.

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

Add power fault check to checkstop handler

If a power fault is being reported during a
checkstop condition we will not call the
analyzer.

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

show more ...

Add Attn config option for clearing Attn Interrupts

Added attn config option
Added UT for attn config object
Added UT for cli that uses attn config object

Signed-off-by: austinfcui <austinfcui@gmai

Add Attn config option for clearing Attn Interrupts

Added attn config option
Added UT for attn config object
Added UT for cli that uses attn config object

Signed-off-by: austinfcui <austinfcui@gmail.com>
Change-Id: I0e30ebf4d94de676814bf5e173691c4d1ee362cb

show more ...

Attn: Clear attention interrupts

Clear attention interrupts when starting the attention handler. This
code will try to clear active attention interrupts. If an attention is
active the interrupt will

Attn: Clear attention interrupts

Clear attention interrupts when starting the attention handler. This
code will try to clear active attention interrupts. If an attention is
active the interrupt will remain active as well. This code is
meant to address cases where attentions have been cleared but the
associated interrupt has not been cleared. This fixes the problem of
attention handler not handling attentions after an MPIPL.

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

show more ...

attn: Fix incorrect trace message formatting

Some of the TI info data being traced for logging was incorrectly
formatted when trace support was changed to using the util functions.

Signed-off-by: B

attn: Fix incorrect trace message formatting

Some of the TI info data being traced for logging was incorrectly
formatted when trace support was changed to using the util functions.

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

show more ...

attn: Move create pel support to util

Move the create pel support from attention handler the util library.

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

attn: Move create pel support to util

Move the create pel support from attention handler the util library.

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

show more ...

Attention handler trace message refactor

Signed-off-by: austinfcui <austinfcui@gmail.com>
Change-Id: If86af8fb88b0ce15f6a626676a412a5cee88acc0

attn: Collect SBE FFDC on getTiInfo chip-op fail

If the PHAL library throws an exception during the call to
getTiInfo it may provide some SBE FFDC data which the attention
handler will collect and p

attn: Collect SBE FFDC on getTiInfo chip-op fail

If the PHAL library throws an exception during the call to
getTiInfo it may provide some SBE FFDC data which the attention
handler will collect and put into an event log entry.

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

show more ...

add analysis type to analyzer main function

The analyzer behavior will vary depending on the type of event that
triggered analysis. For example, we do not want to perform any service
actions when ma

add analysis type to analyzer main function

The analyzer behavior will vary depending on the type of event that
triggered analysis. For example, we do not want to perform any service
actions when manual analysis is done from the command line.

Signed-off-by: Zane Shelley <zshelle@us.ibm.com>
Change-Id: Iecfbac7f680b3fefebb0e02a9caa5fc06b692c7a

show more ...

change analyzerHardware() to return PEL PLID is created

Signed-off-by: Zane Shelley <zshelle@us.ibm.com>
Change-Id: Icee2909e5060c79cb95ead8fc08a5e359df42829

attn: Add wait for power fault handling

Added a 10 second delay between checkstop detection and call to analyzer
so that power fault handler logic can intervene before analyzer begins
logging event

attn: Add wait for power fault handling

Added a 10 second delay between checkstop detection and call to analyzer
so that power fault handler logic can intervene before analyzer begins
logging events.

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

show more ...

include config header for use of CONFIG_PHAL_API

CONFIG_PHAL_API is defined in config.h, which was not included in
attn_handler.cpp before use of CONFIG_PHAL_API.

Change-Id: Ic5fb0695bc0811ffcd2325

include config header for use of CONFIG_PHAL_API

CONFIG_PHAL_API is defined in config.h, which was not included in
attn_handler.cpp before use of CONFIG_PHAL_API.

Change-Id: Ic5fb0695bc0811ffcd232511b52023e33469c43a
Signed-off-by: Zane Shelley <zshelle@us.ibm.com>

show more ...

Attn: use phal library for get TI info calls

Switch from using pdbg library sbe_mpipl_get_ti_info() to using phal
library getTiInfo() for retrieving pointer to TI info shared memory
region.

Signed-

Attn: use phal library for get TI info calls

Switch from using pdbg library sbe_mpipl_get_ti_info() to using phal
library getTiInfo() for retrieving pointer to TI info shared memory
region.

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

show more ...

On checkstop request dump and transition host

After checkstop analyses completes attention handler will request a dump
and the transition the host. The type of dump requested is dependent on
the res

On checkstop request dump and transition host

After checkstop analyses completes attention handler will request a dump
and the transition the host. The type of dump requested is dependent on
the results of the isolator.

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

show more ...

util::dbus: Make common the hostRunningState code

Move the code for getting the host running state from attention handler
specific code to common utility code.

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

util::dbus: Make common the hostRunningState code

Move the code for getting the host running state from attention handler
specific code to common utility code.

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

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 ...

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: 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: 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: 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 ...