diff options
| author | Richard J. Knight <rjknight@us.ibm.com> | 2017-05-03 15:27:59 -0500 |
|---|---|---|
| committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2017-05-07 22:39:14 -0400 |
| commit | 5e56eae5e3b8bedeafda65e5244f2724d3724a22 (patch) | |
| tree | 245fbe525d52610309b73158133af04f523e4daf /src | |
| parent | 05404541b9c109750d1d9d43da72360a2741ef7a (diff) | |
| download | blackbird-hostboot-5e56eae5e3b8bedeafda65e5244f2724d3724a22.tar.gz blackbird-hostboot-5e56eae5e3b8bedeafda65e5244f2724d3724a22.zip | |
SW386506 - data storage exception in host_disable_memvolt
-Modify function calls using hwsvPowrMemVoltDomainRequest_t
to pass by reference instead of by value.
Change-Id: I557ca92f76601e7b17568a8fa2f3946a0304c40a
CQ:SW386506
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/40043
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Reviewed-by: Martin Gloff <mgloff@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')
| -rw-r--r-- | src/usr/isteps/hbToHwsvVoltageMsg.C | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/src/usr/isteps/hbToHwsvVoltageMsg.C b/src/usr/isteps/hbToHwsvVoltageMsg.C index e235ab35d..8b29e7ded 100644 --- a/src/usr/isteps/hbToHwsvVoltageMsg.C +++ b/src/usr/isteps/hbToHwsvVoltageMsg.C @@ -72,8 +72,8 @@ HBToHwsvVoltageMsg::~HBToHwsvVoltageMsg() /////////////////////////////////////////////////////////////////////////////// bool compareVids( - HBToHwsvVoltageMsg::hwsvPowrMemVoltDomainRequest_t i_lhs, - HBToHwsvVoltageMsg::hwsvPowrMemVoltDomainRequest_t i_rhs) + HBToHwsvVoltageMsg::hwsvPowrMemVoltDomainRequest_t& i_lhs, + HBToHwsvVoltageMsg::hwsvPowrMemVoltDomainRequest_t& i_rhs) { bool lhsLogicallyBeforeRhs = (i_lhs.domain < i_rhs.domain); @@ -90,8 +90,8 @@ bool compareVids( // compareSequenceOrder /////////////////////////////////////////////////////////////////////////////// bool compareSequenceOrder( - HBToHwsvVoltageMsg::hwsvPowrMemVoltDomainRequest_t i_lhs, - HBToHwsvVoltageMsg::hwsvPowrMemVoltDomainRequest_t i_rhs) + HBToHwsvVoltageMsg::hwsvPowrMemVoltDomainRequest_t& i_lhs, + HBToHwsvVoltageMsg::hwsvPowrMemVoltDomainRequest_t& i_rhs) { // Sequencing order is VDN->VDD->VCS // true means lhs should be ordered before rhs @@ -112,8 +112,8 @@ bool compareSequenceOrder( /////////////////////////////////////////////////////////////////////////////// bool areVidsEqual( - HBToHwsvVoltageMsg::hwsvPowrMemVoltDomainRequest_t i_lhs, - HBToHwsvVoltageMsg::hwsvPowrMemVoltDomainRequest_t i_rhs) + HBToHwsvVoltageMsg::hwsvPowrMemVoltDomainRequest_t& i_lhs, + HBToHwsvVoltageMsg::hwsvPowrMemVoltDomainRequest_t& i_rhs) { return( ( i_lhs.domain == i_rhs.domain) @@ -128,7 +128,7 @@ bool areVidsEqual( /////////////////////////////////////////////////////////////////////////////// bool isUnusedVoltageDomain( - HBToHwsvVoltageMsg::hwsvPowrMemVoltDomainRequest_t i_vid) + HBToHwsvVoltageMsg::hwsvPowrMemVoltDomainRequest_t& i_vid) { return (!i_vid.voltageMillivolts); } |
