ATTN: Removed "... waiting for events ..." trace

This trace message seems to confuse people, making them think the
attention handler is hung.

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

ATTN: Removed "... waiting for events ..." trace

This trace message seems to confuse people, making them think the
attention handler is hung.

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

show more ...

ATTN: Start attention handler AFTER fsi scan

Load the attention handler after the fsi scan and stop the attention
handler when the host stops. This to make sure the attention handler is
able to succ

ATTN: Start attention handler AFTER fsi scan

Load the attention handler after the fsi scan and stop the attention
handler when the host stops. This to make sure the attention handler is
able to successfully read cfam data using the libpdbg library.

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

show more ...

ATTN: Get TI Info PIB target from path

Changed code to find PIB target, for get TI info chipop, to use the path
from PROC rather than iterate through targets by class.

Signed-off-by: Ben Tyner <ben

ATTN: Get TI Info PIB target from path

Changed code to find PIB target, for get TI info chipop, to use the path
from PROC rather than iterate through targets by class.

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

show more ...

Use pib target for sbe_mpipl_get_ti_info call

Attention handler was incorrectly using the fsi target for the call to
sbe_mpipl_get_ti_info. Changed code to use a pib target.

Signed-off-by: Ben Tyne

Use pib target for sbe_mpipl_get_ti_info call

Attention handler was incorrectly using the fsi target for the call to
sbe_mpipl_get_ti_info. Changed code to use a pib target.

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

show more ...

Do not qualify active attentions with true-mask

We were using cfam 0x100d to qualify active attentions as valid
reasons for invocation of the attention handler. This is not
needed as the attention h

Do not qualify active attentions with true-mask

We were using cfam 0x100d to qualify active attentions as valid
reasons for invocation of the attention handler. This is not
needed as the attention handler will only be invoked by a GPIO
event which should only occur when 0x100d is configured correclty.

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

show more ...

Normalize includes relative to source root

For unit test cases it is cleaner to have include file paths relative to
source root (e.g. #include attn/attn_handler.hpp).

Signed-off-by: Ben Tyner <ben.

Normalize includes relative to source root

For unit test cases it is cleaner to have include file paths relative to
source root (e.g. #include attn/attn_handler.hpp).

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

show more ...

Allow building locally without phosphor-logging

Check for phosphor-logging available. It is only available when building
via CI, SDK or if phosphor-logging and associated dbus support has been
insta

Allow building locally without phosphor-logging

Check for phosphor-logging available. It is only available when building
via CI, SDK or if phosphor-logging and associated dbus support has been
installed locally. If phosphor-logging is not available then compile in
minimal logging code. This allows for local testing of changes that are
not affected by or depend on phosphor-logging.

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

show more ...

Add support for Get TI Info chipop

Use the Get TI Info chipop to determine if a special attention was
due to a TI or a breakpoint.

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

Add support for Get TI Info chipop

Use the Get TI Info chipop to determine if a special attention was
due to a TI or a breakpoint.

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

show more ...

Return code cleanup

Moved attention handler return codes to hpp file and removed their
usage in the analyzer code since technically the analyzer is a
seperate library.

Signed-off-by: Ben Tyner <ben

Return code cleanup

Moved attention handler return codes to hpp file and removed their
usage in the analyzer code since technically the analyzer is a
seperate library.

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

show more ...

Platform Event Log (PEL) generation support added

Attention events will now generate an associated PEL

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

Platform Event Log (PEL) generation support added

Attention events will now generate an associated PEL

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

show more ...

Find attention gpio rather than hard code value

Find the attention gpio by line-name (checkstop) rather than by
chip and pin number. This allows the attention handler to work on
platforms with the a

Find attention gpio rather than hard code value

Find the attention gpio by line-name (checkstop) rather than by
chip and pin number. This allows the attention handler to work on
platforms with the attention signal tied to different GPIO's.

Also fixed issue of pdbg targets not getting initialized when
attention handler is built to run in no-listener mode (build option
nlmode, the default for now).

Added/changed a few trace messages to aid in future debugging.

Changed build to not link to libpdbg using "whole-archive" strategy
as it does not seem to be needed anymore.

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

show more ...

Moved configuration handling into cli handler code

Cleanup: moved the updating of configuration object to
command line handling code to remove some single-use booleans.

Signed-off-by: Ben Tyner <be

Moved configuration handling into cli handler code

Cleanup: moved the updating of configuration object to
command line handling code to remove some single-use booleans.

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

show more ...

Enable/disable logic for each attention type

Allow the handler for each attention type to be enabled or
disabled. The default is all attention handlers are enabled.

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

Enable/disable logic for each attention type

Allow the handler for each attention type to be enabled or
disabled. The default is all attention handlers are enabled.

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

show more ...

Log message cleanup

Removed std::endl from log messages.

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

Add attention event handling priority logic

All active attention events are gathered and prioritized and the highest
priority event is handled.

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

Add attention event handling priority logic

All active attention events are gathered and prioritized and the highest
priority event is handled.

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

show more ...

Added end2end test case

Added end2end test case for exercising code from attention event to isolator.

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

Added end2end test case

Added end2end test case for exercising code from attention event to isolator.

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

show more ...

Combine attn handler and openpower hwdiags

The main binary is now openpower-hw-diags. This application will take a
command line option --daemon to load it as a daemon. As a daemon it will
register t

Combine attn handler and openpower hwdiags

The main binary is now openpower-hw-diags. This application will take a
command line option --daemon to load it as a daemon. As a daemon it will
register the attention handler portion of application as the attention
gpio event handler. If the application is not loaded as a daemon it will
operate as a stand alone application which accepts command line options
for requesting hardware analyses and diagnostics operations.

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

show more ...

Add command line interface to attention handler

Allows configuring attention handler before starting attention
GPIO monitor.

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

Add command line interface to attention handler

Allows configuring attention handler before starting attention
GPIO monitor.

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

show more ...

Add attention handler systemd unit file

Moved attn_handler.service file out of recipe and into
local repository and installing it via local meson.build.

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

Add attention handler systemd unit file

Moved attn_handler.service file out of recipe and into
local repository and installing it via local meson.build.

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

show more ...

For TI start host diagnostic mode service

For TI special attention start the obmc-host-diagnostic-mode systemd service.

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

For TI start host diagnostic mode service

For TI special attention start the obmc-host-diagnostic-mode systemd service.

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

show more ...

Attention Handler gpio monitor

Register an async-io service for monitoring the attention GPIO and use the attention handler as the GPIO event handler. The GPIO monitor will respond to changes on the

Attention Handler gpio monitor

Register an async-io service for monitoring the attention GPIO and use the attention handler as the GPIO event handler. The GPIO monitor will respond to changes on the attention GPIO and in response will call the attention handler base logic to handle the attention events.

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

show more ...

Attention Handler base logic

Added base logic for the attention handler application (attn) as a starting point for further refinement. Attn will handle SBE vital, system checkstop and special attent

Attention Handler base logic

Added base logic for the attention handler application (attn) as a starting point for further refinement. Attn will handle SBE vital, system checkstop and special attention events. These events will be handled through a combination local logic and external hardware and software interfaces.

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

show more ...

Attention Handler design markdown

Added attention handler design document and enabled clang-format support. This design document is a work in progress and will continue to be updated as the attentio

Attention Handler design markdown

Added attention handler design document and enabled clang-format support. This design document is a work in progress and will continue to be updated as the attention handler design evolves.

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

show more ...