diff options
| author | Dan Crowell <dcrowell@us.ibm.com> | 2018-08-20 16:35:49 -0500 |
|---|---|---|
| committer | William G. Hoffa <wghoffa@us.ibm.com> | 2018-10-02 08:57:58 -0500 |
| commit | 46b6d71b01b4936cfdf1e3236cde199a3940a231 (patch) | |
| tree | f413bdd97c15a311b154a971a7338ef3d0772f25 /src/usr/htmgt/runtime | |
| parent | 01fcdb647ea0a6c899b9de21ed144f16dc2f2afb (diff) | |
| download | talos-hostboot-46b6d71b01b4936cfdf1e3236cde199a3940a231.tar.gz talos-hostboot-46b6d71b01b4936cfdf1e3236cde199a3940a231.zip | |
Add consistent enter-exit traces for all runtime interfaces
Created a new 'HBRT' trace buffer that is exclusively used to
bound the external calls into our runtime image.
Modified the return code values to be the reasoncode of the error
log we commit instead of a generic '-1' value that is not
very helpful.
Change-Id: Id41288ea1903bf6d11e967fcb10a8184153943c8
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/64871
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
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: Brian J. Stegmiller <bjs@us.ibm.com>
Reviewed-by: Matt Derksen <mderkse1@us.ibm.com>
Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com>
Diffstat (limited to 'src/usr/htmgt/runtime')
| -rw-r--r-- | src/usr/htmgt/runtime/rt_occ.C | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/src/usr/htmgt/runtime/rt_occ.C b/src/usr/htmgt/runtime/rt_occ.C index 7c453ffc2..652e767ed 100644 --- a/src/usr/htmgt/runtime/rt_occ.C +++ b/src/usr/htmgt/runtime/rt_occ.C @@ -31,6 +31,7 @@ using namespace TARGETING; +extern trace_desc_t* g_trac_hbrt; namespace HTMGT @@ -40,6 +41,7 @@ namespace HTMGT void process_occ_error (uint64_t i_chipId) { + TRACFCOMP(g_trac_hbrt, ENTER_MRK" process_occ_error"); #ifdef CONFIG_HTMGT TARGETING::Target* l_reportingOccTarget = NULL; errlHndl_t err = RT_TARG::getHbTarget(i_chipId,l_reportingOccTarget); @@ -57,12 +59,14 @@ namespace HTMGT TMGT_ERR("Unexpected call to process_occ_error(%d)" " when HTMGT is not enabled", i_chipId); #endif + TRACFCOMP(g_trac_hbrt, EXIT_MRK" process_occ_error"); } //------------------------------------------------------------------------ void process_occ_reset (uint64_t i_chipId) { + TRACFCOMP(g_trac_hbrt, ENTER_MRK" process_occ_reset"); #ifdef CONFIG_HTMGT TARGETING::Target* l_failedOccTarget = NULL; errlHndl_t err = RT_TARG::getHbTarget(i_chipId,l_failedOccTarget); @@ -80,12 +84,14 @@ namespace HTMGT TMGT_ERR("Unexpected call to process_occ_reset(%d)" " when HTMGT is not enabled", i_chipId); #endif + TRACFCOMP(g_trac_hbrt, EXIT_MRK" process_occ_reset"); } //------------------------------------------------------------------------ int enable_occ_actuation (int i_occ_activation) { + TRACFCOMP(g_trac_hbrt, ENTER_MRK" enable_occ_actuation"); int rc = 0; #ifdef CONFIG_HTMGT errlHndl_t err = HTMGT::enableOccActuation(0 != i_occ_activation); @@ -107,6 +113,7 @@ namespace HTMGT TMGT_ERR("Unexpected call to enable_occ_actuation(%d)" " when HTMGT is not enabled", i_occ_activation); #endif + TRACFCOMP(g_trac_hbrt, EXIT_MRK" enable_occ_actuation: rc=0x%X",rc); return rc; } @@ -117,6 +124,7 @@ namespace HTMGT uint16_t * o_rspLength, uint8_t * o_rspData) { + TRACFCOMP(g_trac_hbrt, ENTER_MRK" mfg_htmgt_pass_thru"); int rc = 0; #ifdef CONFIG_HTMGT errlHndl_t err = HTMGT::passThruCommand(i_cmdLength, i_cmdData, @@ -140,6 +148,7 @@ namespace HTMGT #else o_rspLength = 0; #endif + TRACFCOMP(g_trac_hbrt, EXIT_MRK" mfg_htmgt_pass_thru: rc=0x%X",rc); return rc; } |
