diff options
7 files changed, 21 insertions, 25 deletions
diff --git a/src/import/chips/ocmb/explorer/procedures/hwp/memory/exp_scrub.C b/src/import/chips/ocmb/explorer/procedures/hwp/memory/exp_scrub.C index 45cd07ee0..2ea07e54e 100644 --- a/src/import/chips/ocmb/explorer/procedures/hwp/memory/exp_scrub.C +++ b/src/import/chips/ocmb/explorer/procedures/hwp/memory/exp_scrub.C @@ -48,7 +48,10 @@ extern "C" fapi2::ReturnCode exp_scrub(const fapi2::Target<fapi2::TARGET_TYPE_OCMB_CHIP>& i_target) { FAPI_INF("Start exp scrub for %s", mss::c_str(i_target)); - FAPI_TRY(mss::memdiags::mss_scrub_helper(i_target)); + // Initialize memory and set firs accordingly + FAPI_TRY(mss::memdiags::mss_initialize_memory(i_target)); + // Kickoff background scrub and unmask firs + FAPI_TRY(mss::memdiags::mss_background_scrub_helper(i_target)); fapi_try_exit: FAPI_INF("End exp scrub for %s", mss::c_str(i_target)); diff --git a/src/import/chips/p9/procedures/hwp/memory/p9_mss_scrub.C b/src/import/chips/p9/procedures/hwp/memory/p9_mss_scrub.C index c64a97100..caba47caf 100644 --- a/src/import/chips/p9/procedures/hwp/memory/p9_mss_scrub.C +++ b/src/import/chips/p9/procedures/hwp/memory/p9_mss_scrub.C @@ -61,7 +61,10 @@ extern "C" fapi2::ReturnCode p9_mss_scrub( const fapi2::Target<TARGET_TYPE_MCBIST>& i_target ) { FAPI_INF("Start mss scrub for %s", mss::c_str(i_target)); - FAPI_TRY(mss::memdiags::mss_scrub_helper(i_target)); + // Initialize memory and set firs accordingly + FAPI_TRY(mss::memdiags::mss_initialize_memory(i_target)); + // Kickoff background scrub and unmask firs + FAPI_TRY(mss::memdiags::mss_background_scrub_helper(i_target)); fapi_try_exit: return fapi2::current_err; diff --git a/src/import/chips/p9a/procedures/hwp/memory/p9a_mss_bulk_pwr_throttles.mk b/src/import/chips/p9a/procedures/hwp/memory/p9a_mss_bulk_pwr_throttles.mk index aff426ea7..8c936264d 100644 --- a/src/import/chips/p9a/procedures/hwp/memory/p9a_mss_bulk_pwr_throttles.mk +++ b/src/import/chips/p9a/procedures/hwp/memory/p9a_mss_bulk_pwr_throttles.mk @@ -25,5 +25,5 @@ -include 00p9a_common.mk PROCEDURE=p9a_mss_bulk_pwr_throttles -$(eval $(call ADD_MEMORY_INCDIRS,$(PROCEDURE))) +$(eval $(call ADD_P9A_MEMORY_INCDIRS,$(PROCEDURE))) $(call BUILD_PROCEDURE) diff --git a/src/import/chips/p9a/procedures/hwp/memory/p9a_mss_eff_config_thermal.mk b/src/import/chips/p9a/procedures/hwp/memory/p9a_mss_eff_config_thermal.mk index 07adb9c70..316749d4e 100644 --- a/src/import/chips/p9a/procedures/hwp/memory/p9a_mss_eff_config_thermal.mk +++ b/src/import/chips/p9a/procedures/hwp/memory/p9a_mss_eff_config_thermal.mk @@ -5,7 +5,7 @@ # # OpenPOWER HostBoot Project # -# Contributors Listed Below - COPYRIGHT 2018 +# Contributors Listed Below - COPYRIGHT 2018,2019 # [+] International Business Machines Corp. # # @@ -27,5 +27,5 @@ -include 00p9a_common.mk PROCEDURE=p9a_mss_eff_config_thermal -$(eval $(call ADD_MEMORY_INCDIRS,$(PROCEDURE))) +$(eval $(call ADD_P9A_MEMORY_INCDIRS,$(PROCEDURE))) $(call BUILD_PROCEDURE) diff --git a/src/import/chips/p9a/procedures/hwp/memory/p9a_mss_utils_to_throttle.mk b/src/import/chips/p9a/procedures/hwp/memory/p9a_mss_utils_to_throttle.mk index a60770713..95a11aab6 100644 --- a/src/import/chips/p9a/procedures/hwp/memory/p9a_mss_utils_to_throttle.mk +++ b/src/import/chips/p9a/procedures/hwp/memory/p9a_mss_utils_to_throttle.mk @@ -25,5 +25,5 @@ -include 00p9a_common.mk PROCEDURE=p9a_mss_utils_to_throttle -$(eval $(call ADD_MEMORY_INCDIRS,$(PROCEDURE))) +$(eval $(call ADD_P9A_MEMORY_INCDIRS,$(PROCEDURE))) $(call BUILD_PROCEDURE) diff --git a/src/import/chips/p9a/procedures/hwp/memory/p9a_mss_volt.mk b/src/import/chips/p9a/procedures/hwp/memory/p9a_mss_volt.mk index 045691c45..70f0fe675 100644 --- a/src/import/chips/p9a/procedures/hwp/memory/p9a_mss_volt.mk +++ b/src/import/chips/p9a/procedures/hwp/memory/p9a_mss_volt.mk @@ -5,7 +5,7 @@ # # OpenPOWER HostBoot Project # -# Contributors Listed Below - COPYRIGHT 2018 +# Contributors Listed Below - COPYRIGHT 2018,2019 # [+] International Business Machines Corp. # # @@ -27,5 +27,5 @@ -include 00p9a_common.mk PROCEDURE=p9a_mss_volt -$(eval $(call ADD_MEMORY_INCDIRS,$(PROCEDURE))) +$(eval $(call ADD_P9A_MEMORY_INCDIRS,$(PROCEDURE))) $(call BUILD_PROCEDURE) diff --git a/src/import/generic/memory/lib/utils/mcbist/gen_mss_memdiags.H b/src/import/generic/memory/lib/utils/mcbist/gen_mss_memdiags.H index 33a3156f3..dad5df32e 100644 --- a/src/import/generic/memory/lib/utils/mcbist/gen_mss_memdiags.H +++ b/src/import/generic/memory/lib/utils/mcbist/gen_mss_memdiags.H @@ -1115,18 +1115,18 @@ fapi_try_exit: } /// -/// @brief Begin scrub helper +/// @brief Begin initialize memory /// @tparam MC the mc type of the T -/// @tparam T fapi2::TargetType of the MCBIST engine +/// @tparam T fapi2::TargetType of the MC engine /// @tparam TT the mcbistTraits associated with T -/// @param[in] i_target MCBIST +/// @param[in] i_target MC /// @return FAPI2_RC_SUCCESS iff ok /// template< mss::mc_type MC = DEFAULT_MC_TYPE, fapi2::TargetType T, typename TT = mcbistTraits<MC, T> > -fapi2::ReturnCode mss_scrub_helper(const fapi2::Target<T>& i_target ) +fapi2::ReturnCode mss_initialize_memory(const fapi2::Target<T>& i_target ) { using ET = mss::mcbistMCTraits<MC>; - FAPI_INF("Start mss_scrub_helper for %s", mss::c_str(i_target)); + FAPI_INF("Start mss_initialize_memory for %s", mss::c_str(i_target)); // If there are no DIMM we don't need to bother. In fact, we can't as we didn't setup // attributes for the PHY, etc. @@ -1167,7 +1167,6 @@ fapi2::ReturnCode mss_scrub_helper(const fapi2::Target<T>& i_target ) // Unmask firs and turn off FIFO mode before returning FAPI_TRY ( mss::unmask::after_memdiags<MC>( i_target ) ); - FAPI_TRY ( mss::unmask::after_background_scrub<MC>( i_target ) ); FAPI_TRY ( mss::reset_reorder_queue_settings<MC>(i_target) ); return l_rc; @@ -1192,21 +1191,12 @@ fapi2::ReturnCode mss_scrub_helper(const fapi2::Target<T>& i_target ) FAPI_ASSERT( l_poll_results == true, ET::memdiags_sf_init_failed_init().set_MC_TARGET(i_target), - "p9_mss_scrub (init) timedout %s", mss::c_str(i_target) ); + "sf init for scrub/memdiags timedout %s", mss::c_str(i_target) ); // Unmask firs after memdiags and turn off FIFO mode FAPI_TRY ( mss::unmask::after_memdiags<MC>( i_target ) ); FAPI_TRY ( mss::reset_reorder_queue_settings<MC>(i_target) ); - // Start background scrub - FAPI_TRY ( mss::memdiags::background_scrub<MC>( i_target, - mss::mcbist::stop_conditions<MC>(), - mss::mcbist::speed::BG_SCRUB, - mss::mcbist::address() ) ); - - // Unmask firs after background scrub is started - FAPI_TRY ( mss::unmask::after_background_scrub<MC>( i_target ) ); - fapi_try_exit: return fapi2::current_err; } @@ -1216,7 +1206,7 @@ fapi_try_exit: /// @tparam MC the mc type of the T /// @tparam T fapi2::TargetType of the MCBIST engine /// @tparam TT the mcbistTraits associated with T -/// @param[in] i_target MCBIST +/// @param[in] i_target MC /// @return FAPI2_RC_SUCCESS iff ok /// template< mss::mc_type MC = DEFAULT_MC_TYPE, fapi2::TargetType T, typename TT = mcbistTraits<MC, T> > |