diff options
| author | Dan Crowell <dcrowell@us.ibm.com> | 2018-07-19 10:31:50 -0500 |
|---|---|---|
| committer | Christian R. Geddes <crgeddes@us.ibm.com> | 2018-07-25 12:36:14 -0500 |
| commit | c3dda09f76af00e8c2c603ff2adac0e73643b92e (patch) | |
| tree | 8638e3994c466096be732e8ddf99b3def64d8ab3 /src/import/chips/p9/procedures/hwp/perv | |
| parent | d334558fdeb8a40bc6f19923fb21003f16a3f602 (diff) | |
| download | talos-hostboot-c3dda09f76af00e8c2c603ff2adac0e73643b92e.tar.gz talos-hostboot-c3dda09f76af00e8c2c603ff2adac0e73643b92e.zip | |
Add constants to core checkstop handler
Added some constants to make the arguments into the core
checkstop handler code more obvious.
Change-Id: I8e29b653a925096d867dc0e97d0ecaca3d412721
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/62905
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Joseph J. McGill <jmcgill@us.ibm.com>
Reviewed-by: Thi N. Tran <thi@us.ibm.com>
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/63238
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com>
Diffstat (limited to 'src/import/chips/p9/procedures/hwp/perv')
| -rw-r--r-- | src/import/chips/p9/procedures/hwp/perv/p9_core_checkstop_handler.C | 2 | ||||
| -rw-r--r-- | src/import/chips/p9/procedures/hwp/perv/p9_core_checkstop_handler.H | 4 |
2 files changed, 5 insertions, 1 deletions
diff --git a/src/import/chips/p9/procedures/hwp/perv/p9_core_checkstop_handler.C b/src/import/chips/p9/procedures/hwp/perv/p9_core_checkstop_handler.C index d8e9bd996..042df2122 100644 --- a/src/import/chips/p9/procedures/hwp/perv/p9_core_checkstop_handler.C +++ b/src/import/chips/p9/procedures/hwp/perv/p9_core_checkstop_handler.C @@ -38,7 +38,7 @@ fapi2::ReturnCode p9_core_checkstop_handler( const fapi2::Target<fapi2::TARGET_TYPE_SYSTEM> FAPI_SYSTEM; //if true, save off the original action, and turn local xstops into system xstops. - if(i_override_restore) + if(i_override_restore == CORE_XSTOP_HNDLR__SAVE_AND_ESCALATE) { // Getting ACTION0 FAPI_TRY(fapi2::getScom(i_target_core, C_CORE_ACTION0, l_action0), diff --git a/src/import/chips/p9/procedures/hwp/perv/p9_core_checkstop_handler.H b/src/import/chips/p9/procedures/hwp/perv/p9_core_checkstop_handler.H index 4b9435c79..1ed7f3d3a 100644 --- a/src/import/chips/p9/procedures/hwp/perv/p9_core_checkstop_handler.H +++ b/src/import/chips/p9/procedures/hwp/perv/p9_core_checkstop_handler.H @@ -30,6 +30,10 @@ typedef fapi2::ReturnCode (*p9_core_checkstop_handler_FP_t)( const fapi2::Target<fapi2::TARGET_TYPE_CORE>&, bool); +constexpr bool CORE_XSTOP_HNDLR__SAVE_AND_ESCALATE = true; +constexpr bool CORE_XSTOP_HNDLR__RESTORE_ORIG = false; + + // @brief Handles core checkstops for P9 when we want to switch between // unit and system checkstops. // |
