diff options
author | Dan Crowell <dcrowell@us.ibm.com> | 2018-02-22 09:59:11 -0600 |
---|---|---|
committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2018-02-24 21:42:48 -0500 |
commit | 431a3cc0aa10884e9f9f27a12c6d1c97206bc83a (patch) | |
tree | e8361b4da7d862f05141536537854ae3ee162241 /src/include/usr/runtime | |
parent | 24bc6a1bee514ae580733f1695c7d82edc4da4cb (diff) | |
download | talos-hostboot-431a3cc0aa10884e9f9f27a12c6d1c97206bc83a.tar.gz talos-hostboot-431a3cc0aa10884e9f9f27a12c6d1c97206bc83a.zip |
Bug fixes for concurrent update of HBRT
Fixed a bug where the wrong copy of the attribute was being used
after our code update preparation was run.
Fixed a bug due to not having enough memory allocated to update
the attribute values during the transition.
Added space to initial reserved memory to allow for new
attributes to be added at runtime.
Change-Id: I97c4d20ff1c369398be470fef718bbd001f71c1b
CQ: SW417526
ForwardPort: yes
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/54685
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/include/usr/runtime')
-rw-r--r-- | src/include/usr/runtime/rt_targeting.H | 18 |
1 files changed, 10 insertions, 8 deletions
diff --git a/src/include/usr/runtime/rt_targeting.H b/src/include/usr/runtime/rt_targeting.H index 3f1ccb480..9f3d96818 100644 --- a/src/include/usr/runtime/rt_targeting.H +++ b/src/include/usr/runtime/rt_targeting.H @@ -68,11 +68,11 @@ namespace RT_TARG * @param[in] Pointer to current Reserved Memory targeting binary data * @param[in/out] Pointer to new LID Structure targeting binary data * @param[out] Error log userdata2 value associated with non-zero rtn code - * @return 0 on success, else return code + * @return nullptr on success, else error log */ - int saveRestoreAttrs(void *i_rsvdMemPtr, - void *io_lidStructPtr, - uint64_t& o_userdata2); + errlHndl_t saveRestoreAttrs(void *i_rsvdMemPtr, + void *io_lidStructPtr, + uint64_t& o_userdata2); /** * @brief Validate LID Structure against Reserved Memory. Check that the @@ -81,12 +81,14 @@ namespace RT_TARG * match. * @param[in] Pointer to new LID Structure targeting binary data * @param[in] Pointer to current Reserved Memory targeting binary data + * @param[out] Total size of all sections in the new lid * @param[out] Error log userdata2 value associated with non-zero rtn code - * @return 0 on success, else return code + * @return nullptr on success, else error log */ - int validateData(void *i_lidStructPtr, - void *i_rsvdMemPtr, - uint64_t& o_userdata2); + errlHndl_t validateData(void *i_lidStructPtr, + void *i_rsvdMemPtr, + size_t& o_lidTotalSize, + uint64_t& o_userdata2); /** * @brief Apply ATTR_TMP overrides to be available for run time |