attn: Add dependency to service file

Start the attn_handler.service after the
openpower-update-bios-attr-table.service since that's the service that
sets up the DEVTREE file. Also update the path to

attn: Add dependency to service file

Start the attn_handler.service after the
openpower-update-bios-attr-table.service since that's the service that
sets up the DEVTREE file. Also update the path to DEVTREE since pnor is
being deprecated.

Signed-off-by: Adriana Kobylak <anoo@us.ibm.com>
Change-Id: Ia160ea5d529acb618690f8fa6e586bc0ef0729c0

show more ...

attn: Remove stub code intended for native build

Removed some stub code that was originally used to build
openpower-hw-diags natively (via local meson/ninja). Building in this
manner is not supporte

attn: Remove stub code intended for native build

Removed some stub code that was originally used to build
openpower-hw-diags natively (via local meson/ninja). Building in this
manner is not supported right now due to the numerous external
dependencies required by the final binary.

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

show more ...

Header file cleanup in attn

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

attn: Make restart after fail more restrictive

Override the default service restart after fail behavior. The current
behavior is maximum of two failures within 30 seconds before the
attention handle

attn: Make restart after fail more restrictive

Override the default service restart after fail behavior. The current
behavior is maximum of two failures within 30 seconds before the
attention handler is considered failed. The new behavior is maximum of
two failures within 10 minutes before the attention handler is
considered failed. This avoids a potential endless fail and restart
loop due to analyses possibly taking longer than 15 seconds during which
time the attention handler could potentially crash, avoiding the current
30 second fail time window. The 10 minute value is an arbitrarily
large value as we expect the cause of the attention handler crash to
immediately go away or not at all.

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

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

update to C++20

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

attn: Always request a dump for TI with SRC

Attention handler should only honor the TI info hostboot dump flag for
TI with EID. For TI with SRC case always request a dump.

Signed-off-by: Ben Tyner

attn: Always request a dump for TI with SRC

Attention handler should only honor the TI info hostboot dump flag for
TI with EID. For TI with SRC case always request a dump.

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

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: Fix no PEL created when TI info unavailable

When handling HB TI where TI info is not available the logic path was
incorrectly handling this event as a TI with EID and no PEL was
being created.

attn: Fix no PEL created when TI info unavailable

When handling HB TI where TI info is not available the logic path was
incorrectly handling this event as a TI with EID and no PEL was
being created. This is fixed (a PEL is created) by handling this
event correctly as TI with SRC (SRC == BC801B99).

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

show more ...

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

Attn: On TI with recoverable errors call analyzer

While servicing a TI attention handler will check if recoverable
error attentions are present and if so it will call the analyzer.

Signed-off-by: B

Attn: On TI with recoverable errors call analyzer

While servicing a TI attention handler will check if recoverable
error attentions are present and if so it will call the analyzer.

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

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: Qualify MPIPL request with dump policy

Check the value of the dumpPolicyEnabled property and if it is true
(dumps enabled) then request MPIPL for PHYP TI cases, otherwise quiesce
the host.

Si

Attn: Qualify MPIPL request with dump policy

Check the value of the dumpPolicyEnabled property and if it is true
(dumps enabled) then request MPIPL for PHYP TI cases, otherwise quiesce
the host.

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

show more ...

Attn: Properly handle hostboot TI with EID/PLID

Terminate events from hostboot with EID/PLID already have a log
committed so attention handler does not need to generate a PEL.

Signed-off-by: Ben Ty

Attn: Properly handle hostboot TI with EID/PLID

Terminate events from hostboot with EID/PLID already have a log
committed so attention handler does not need to generate a PEL.

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

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

Attn: No event log entry for hostboot FFDC error

If errors are encountered when collecting hostboot FFDC then just trace
the error since we are already in a error path and an event log will be
creat

Attn: No event log entry for hostboot FFDC error

If errors are encountered when collecting hostboot FFDC then just trace
the error since we are already in a error path and an event log will be
created.

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

show more ...

Attn: Self boot engine (SBE) attention support

When an SBE vital attention is detected a plateform event log entry will
be created and an SBE dump will be requested. A request to transition
the host

Attn: Self boot engine (SBE) attention support

When an SBE vital attention is detected a plateform event log entry will
be created and an SBE dump will be requested. A request to transition
the host will be issued once the dump is completed.

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

show more ...

stop ATTN service during watchdog time out handling

A watchdog time out will initiate a dump. As a side-effect, the dump
will likely trigger a checkstop attention. We do not want ATTN to handle
this

stop ATTN service during watchdog time out handling

A watchdog time out will initiate a dump. As a side-effect, the dump
will likely trigger a checkstop attention. We do not want ATTN to handle
this attention. So we are stopping the ATTN service before initiating
the dump. The ATTN service will be restarted on the next IPL.

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

show more ...

Fix for setting PDBG_DTB environment variable

The code for setting PDBG_DTB environment variable conditionally in the
service file was not working correctly which resulted in the environment
variabl

Fix for setting PDBG_DTB environment variable

The code for setting PDBG_DTB environment variable conditionally in the
service file was not working correctly which resulted in the environment
variable not getting set. This was fixed.

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

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

util::dbus: Make common the autorebootEnabled code

Move the code for checking autoreboot property from attention handler
specific code to common utility dbus code.

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

util::dbus: Make common the autorebootEnabled code

Move the code for checking autoreboot property from attention handler
specific code to common utility dbus code.

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

show more ...

util::dbus: Make common the transition host support

Move transition host support from attention handler specific code to
common util code.

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

util::dbus: Make common the transition host support

Move transition host support from attention handler specific code to
common util code.

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

show more ...