summaryrefslogtreecommitdiffstats
path: root/src/import/chips/p9/procedures/hwp/memory/lib
diff options
context:
space:
mode:
authorTsung Yeung <tyeung@us.ibm.com>2019-07-18 16:46:10 -0400
committerDaniel M Crowell <dcrowell@us.ibm.com>2019-07-22 12:56:59 -0500
commit533e2008783a3158e4b16393a1b1e92eaf6388b7 (patch)
treee1c63d6d4b67e42c0b01c905725b2ac1d45d1465 /src/import/chips/p9/procedures/hwp/memory/lib
parent3a513ca00ae74a46de34d5e95abbd43adbc7a32e (diff)
downloadtalos-hostboot-533e2008783a3158e4b16393a1b1e92eaf6388b7.tar.gz
talos-hostboot-533e2008783a3158e4b16393a1b1e92eaf6388b7.zip
Stop mcbist during nvdimm restore to prevent broadcast out of sync
If mcbist is running during nvdimm restore it could detect UE due to the drams not in the right state and could lead to broadcast out of sycn error Change-Id: I1dbf79fb4b3c66f2100eb3dbe202cfae22d35677 CQ:SW470102 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/80645 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com> Reviewed-by: STEPHEN GLANCY <sglancy@us.ibm.com> Reviewed-by: Louis Stermole <stermole@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/80666 Tested-by: Jenkins OP Build CI <op-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/memory/lib')
-rw-r--r--src/import/chips/p9/procedures/hwp/memory/lib/dimm/ddr4/nvdimm_utils.C4
1 files changed, 4 insertions, 0 deletions
diff --git a/src/import/chips/p9/procedures/hwp/memory/lib/dimm/ddr4/nvdimm_utils.C b/src/import/chips/p9/procedures/hwp/memory/lib/dimm/ddr4/nvdimm_utils.C
index 8b49ad80f..9178379dd 100644
--- a/src/import/chips/p9/procedures/hwp/memory/lib/dimm/ddr4/nvdimm_utils.C
+++ b/src/import/chips/p9/procedures/hwp/memory/lib/dimm/ddr4/nvdimm_utils.C
@@ -266,10 +266,14 @@ template<>
fapi2::ReturnCode self_refresh_entry( const fapi2::Target<fapi2::TARGET_TYPE_MCA>& i_target )
{
fapi2::buffer<uint64_t> l_mbarpc0_data, l_mbastr0_data;
+ const auto& l_mcbist = mss::find_target<fapi2::TARGET_TYPE_MCBIST>(i_target);
// Entry time to 0 for immediate entry
constexpr uint64_t l_str_entry_time = 0;
+ // Stop mcbist (scrub) in case of MPIPL. It will get restarted in the later istep
+ FAPI_TRY(mss::mcbist::start_stop(l_mcbist, mss::states::STOP));
+
// Step 1 - In MBARPC0Q, disable power domain control, set domain to MAXALL_MIN0,
// and disable minimum domain reduction (allow immediate entry of STR)
FAPI_TRY(mss::mc::read_mbarpc0(i_target, l_mbarpc0_data));
OpenPOWER on IntegriCloud