diff options
| author | Amit Tendolkar <amit.tendolkar@in.ibm.com> | 2017-05-25 11:13:48 -0500 |
|---|---|---|
| committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2017-07-24 17:01:32 -0400 |
| commit | 87b510d8d6462204a3f99e7a2bda6cc1665b8c65 (patch) | |
| tree | ff129e688b8dd1b94bc7215e62b92439a855f671 /src/import/chips/p9/procedures/hwp/perv | |
| parent | b4e2b378ed92856eb86d5ff5db5874003b4c7191 (diff) | |
| download | talos-hostboot-87b510d8d6462204a3f99e7a2bda6cc1665b8c65.tar.gz talos-hostboot-87b510d8d6462204a3f99e7a2bda6cc1665b8c65.zip | |
Optimized PPE FFDC collection framework
Allows SBE to collect CME / SGPE FFDC for errors from p9_sbe_check_master_stop15
in istep16
Change-Id: Ie31336fae9d29e14dbb7995cd438b87ef615d4b6
RTC: 174610
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/41375
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: Brian T. Vanderpool <vanderp@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com>
Reviewed-by: Michael S. Floyd <mfloyd@us.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/42601
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Diffstat (limited to 'src/import/chips/p9/procedures/hwp/perv')
| -rw-r--r-- | src/import/chips/p9/procedures/hwp/perv/p9_sbe_check_master_stop15.C | 22 |
1 files changed, 16 insertions, 6 deletions
diff --git a/src/import/chips/p9/procedures/hwp/perv/p9_sbe_check_master_stop15.C b/src/import/chips/p9/procedures/hwp/perv/p9_sbe_check_master_stop15.C index 9b2e14b45..3a501ec69 100644 --- a/src/import/chips/p9/procedures/hwp/perv/p9_sbe_check_master_stop15.C +++ b/src/import/chips/p9/procedures/hwp/perv/p9_sbe_check_master_stop15.C @@ -5,7 +5,7 @@ /* */ /* OpenPOWER HostBoot Project */ /* */ -/* Contributors Listed Below - COPYRIGHT 2016 */ +/* Contributors Listed Below - COPYRIGHT 2016,2017 */ /* [+] International Business Machines Corp. */ /* */ /* */ @@ -58,6 +58,10 @@ #include <p9_pm_stop_history.H> #include <p9_quad_scom_addresses.H> +#ifdef DD2 + #include <p9_collect_deadman_ffdc.H> +#endif + // ----------------------------------------------------------------------------- // Function definitions // ----------------------------------------------------------------------------- @@ -124,10 +128,17 @@ fapi2::ReturnCode p9_sbe_check_master_stop15( } else { - FAPI_ASSERT(false, - fapi2::CHECK_MASTER_STOP15_INVALID_STATE() - .set_STOP_HISTORY(l_data64), - "STOP 15 error"); +#ifdef DD2 + FAPI_TRY ( p9_collect_deadman_ffdc ( + i_target, + CHECK_MASTER_STOP15_INVALID_STATE )); +#else + // DD1 has a memory crunch on SBE + FAPI_ASSERT ( false, + fapi2::CHECK_MASTER_STOP15_INVALID_STATE() + .set_STOP_HISTORY(l_data64), + "STOP 15 error" ); +#endif } // @todo RTC 162331 These should work but don't..... follow-up later @@ -153,4 +164,3 @@ fapi_try_exit: return fapi2::current_err; } // END p9_sbe_check_master_stop15 - |
