From 7e0ca5fe37c2f5b2398d4ca53c6cf941140dff33 Mon Sep 17 00:00:00 2001 From: Greg Still Date: Thu, 17 Aug 2017 15:51:15 -0500 Subject: [PATCH] PM (Cronus): Add processor id for errors from p9_activate_stop15_cores - reformatted error output for readability - removed FAPI_ASSERT FFDC as these are not useful in Cronus; moved the FFDC to printfs before the ASSERT - Added isHalted() to ppe_utils - added SSH and better SGPE oriented register FFDC - removed extra setBit to core_sb buffer Change-Id: Ie23f294b843b15a78b8d3ee036310631c0595904 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/44758 Tested-by: FSP CI Jenkins Dev-Ready: Gregory S. Still Tested-by: Jenkins Server Tested-by: Hostboot CI Reviewed-by: RANGANATHPRASAD G. BRAHMASAMUDRA Reviewed-by: Juan R. Medina Reviewed-by: Jennifer A. Stofer Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/47138 Reviewed-by: Sachin Gupta --- .../chips/p9/procedures/hwp/lib/p9_ppe_utils.H | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/src/import/chips/p9/procedures/hwp/lib/p9_ppe_utils.H b/src/import/chips/p9/procedures/hwp/lib/p9_ppe_utils.H index 59c6ef135..9e0ffa67c 100644 --- a/src/import/chips/p9/procedures/hwp/lib/p9_ppe_utils.H +++ b/src/import/chips/p9/procedures/hwp/lib/p9_ppe_utils.H @@ -491,6 +491,19 @@ fapi2::ReturnCode ppe_write_iar( const uint64_t i_base_address, const uint64_t i_address ); + +///-------------------------------------------------------------------------------------- +/// @brief Determine if the PPE is halted +/// @param[in] i_target target +/// @param[in] i_base_address base SCOM address of the PPE +/// @param[out] o_halted boolean true if halted; false otherwise +/// @return fapi2::ReturnCode +///-------------------------------------------------------------------------------------- +fapi2::ReturnCode ppe_isHalted( + const fapi2::Target& i_target, + const uint64_t i_base_address, + bool* o_halted); + #ifndef __HOSTBOOT_MODULE ///--------------------------------------------------------------------------------------