diff options
author | Nick Bofferding <bofferdn@us.ibm.com> | 2018-09-10 12:10:02 -0500 |
---|---|---|
committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2018-10-01 15:37:58 -0500 |
commit | 01fcdb647ea0a6c899b9de21ed144f16dc2f2afb (patch) | |
tree | 79f83d480cb50c10f7a3bb6ea662850b5460c1ea /src/usr/runtime | |
parent | 7209f9061a3cbc8563e583d4abdcb8ffa4653ceb (diff) | |
download | talos-hostboot-01fcdb647ea0a6c899b9de21ed144f16dc2f2afb.tar.gz talos-hostboot-01fcdb647ea0a6c899b9de21ed144f16dc2f2afb.zip |
Shutdown: Move attribute sync to shutdown handler
Moved synching attributes into an attribute resource provider shutdown path
to avoid race conditions along the shutdown path where a normal error path
coupled with parallel shutdown leads to FSP attribute sync errors.
Change-Id: Ibdb828f8132da1e251f880ef0e7d4fea2e4619a3
CQ: SW443737
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/66431
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: Daniel M. Crowell <dcrowell@us.ibm.com>
Diffstat (limited to 'src/usr/runtime')
-rw-r--r-- | src/usr/runtime/populate_hbruntime.C | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/usr/runtime/populate_hbruntime.C b/src/usr/runtime/populate_hbruntime.C index 7f47c4138..340ee3de7 100644 --- a/src/usr/runtime/populate_hbruntime.C +++ b/src/usr/runtime/populate_hbruntime.C @@ -3497,9 +3497,9 @@ errlHndl_t persistent_rwAttrRuntimeCheck( void ) assert(l_msg != nullptr, "Bug! Message allocation failed!"); - l_msg->type = TARGETING::MSG_MM_RP_RUNTIME_PREP; + l_msg->type = TARGETING::AttrRP::MSG_MM_RP_RUNTIME_PREP; - l_msg->data[0] = TARGETING::MSG_MM_RP_RUNTIME_PREP_BEGIN; + l_msg->data[0] = TARGETING::AttrRP::MSG_MM_RP_RUNTIME_PREP_BEGIN; int rc = msg_sendrecv(l_msgQ, l_msg); @@ -3539,8 +3539,8 @@ errlHndl_t persistent_rwAttrRuntimeCheck( void ) validateAllRwNvAttr( *targets ); } - l_msg->type = TARGETING::MSG_MM_RP_RUNTIME_PREP; - l_msg->data[0] = TARGETING::MSG_MM_RP_RUNTIME_PREP_END; + l_msg->type = TARGETING::AttrRP::MSG_MM_RP_RUNTIME_PREP; + l_msg->data[0] = TARGETING::AttrRP::MSG_MM_RP_RUNTIME_PREP_END; int rc = msg_sendrecv(l_msgQ, l_msg); |