summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDan Crowell <dcrowell@us.ibm.com>2016-06-06 11:40:29 -0500
committerMatthew A. Ploetz <maploetz@us.ibm.com>2016-06-15 09:22:51 -0400
commit2be22a7dde9786eeb01f2f86b7b8ad1279423be5 (patch)
treecfa90eed54dfc5ddacf98bb370c73f9e27272582
parent1107721b8a869ce5f1b93ac0b20296209a99ef85 (diff)
downloadtalos-hostboot-2be22a7dde9786eeb01f2f86b7b8ad1279423be5.tar.gz
talos-hostboot-2be22a7dde9786eeb01f2f86b7b8ad1279423be5.zip
Collect FAPI traces for all FAPI_INVOKE_HWP calls
Automatically add the FAPI and FAPI_I traces to all error logs that result from a HWP invocation. Change-Id: Id91df9ca29574669410d0a5b8053cacf3a9ec754 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/25411 Tested-by: Jenkins Server Reviewed-by: Martin Gloff <mgloff@us.ibm.com> Tested-by: FSP CI Jenkins Reviewed-by: Elizabeth K. Liner <eliner@us.ibm.com> Reviewed-by: Matthew A. Ploetz <maploetz@us.ibm.com>
-rw-r--r--src/include/usr/fapi2/plat_hwp_invoker.H10
-rw-r--r--src/usr/isteps/istep06/thread_activate/thread_activate.C6
2 files changed, 7 insertions, 9 deletions
diff --git a/src/include/usr/fapi2/plat_hwp_invoker.H b/src/include/usr/fapi2/plat_hwp_invoker.H
index 0d55c3e01..25ef6ccfe 100644
--- a/src/include/usr/fapi2/plat_hwp_invoker.H
+++ b/src/include/usr/fapi2/plat_hwp_invoker.H
@@ -59,9 +59,13 @@
#define FAPI_INVOKE_HWP(ERRHNDL, FUNC, _args_...) \
{\
- fapi2::ReturnCode l_rc; \
- FAPI_EXEC_HWP(l_rc, FUNC, ##_args_); \
- ERRHNDL = fapi2::rcToErrl(l_rc);\
+ fapi2::ReturnCode l_rc; \
+ FAPI_EXEC_HWP(l_rc, FUNC, ##_args_); \
+ ERRHNDL = fapi2::rcToErrl(l_rc);\
+ if( ERRHNDL ) {\
+ ERRHNDL->collectTrace(FAPI_IMP_TRACE_NAME,256);\
+ ERRHNDL->collectTrace(FAPI_TRACE_NAME,384);\
+ }\
}
#endif // PLATHWPINVOKER_H_
diff --git a/src/usr/isteps/istep06/thread_activate/thread_activate.C b/src/usr/isteps/istep06/thread_activate/thread_activate.C
index 2edd3f71d..1d5d2de14 100644
--- a/src/usr/isteps/istep06/thread_activate/thread_activate.C
+++ b/src/usr/isteps/istep06/thread_activate/thread_activate.C
@@ -323,9 +323,6 @@ void activate_threads( errlHndl_t& io_rtaskRetErrl )
"ERROR: 0x%.8X : p9_cpu_special_wakeup set HWP(cpu %d)",
l_errl->reasonCode(),
l_masterCoreID);
-
- l_errl->collectTrace(FAPI_TRACE_NAME,256);
- l_errl->collectTrace(FAPI_IMP_TRACE_NAME,256);
break;
}
@@ -401,9 +398,6 @@ void activate_threads( errlHndl_t& io_rtaskRetErrl )
l_masterCoreID,
thread_bitset,
l_rasStatus );
-
- l_errl->collectTrace(FAPI_TRACE_NAME,256);
- l_errl->collectTrace(FAPI_IMP_TRACE_NAME,256);
}
else
{
OpenPOWER on IntegriCloud