diff options
| author | Chris Phan <cphan@us.ibm.com> | 2013-04-09 15:19:46 -0500 |
|---|---|---|
| committer | A. Patrick Williams III <iawillia@us.ibm.com> | 2013-04-10 12:06:54 -0500 |
| commit | 5dc5ba07f67dbd159bce6d9e0ba1464d34d88132 (patch) | |
| tree | 9bce2d9914c183fb97f6eb79e8123c42e6e6fe42 /src/usr/diag/prdf/common/framework | |
| parent | 075327ec0d0bc44c92f806bd29e072426c4e63dc (diff) | |
| download | talos-hostboot-5dc5ba07f67dbd159bce6d9e0ba1464d34d88132.tar.gz talos-hostboot-5dc5ba07f67dbd159bce6d9e0ba1464d34d88132.zip | |
PRD: fix comparator operator in ScomRegister
- threw off the entry search in FlyWeight resulting in
duplicate entries added resulting in allocation of
859 KB for each subsequent PRD config build
- Memory number for 2 PRD Config builds:
Before fix: Total Memory: 3018582 bytes
After fix: Total Memory: 2140470 bytes
Change-Id: I516f62af52479b177aa718ae5aa45eb818539725
Reviewed-on: http://gfw160.austin.ibm.com:8080/gerrit/3974
Tested-by: Jenkins Server
Reviewed-by: Zane Shelley <zshelle@us.ibm.com>
Reviewed-by: Sachin Gupta <sgupta2m@in.ibm.com>
Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
Reviewed-on: http://gfw160.austin.ibm.com:8080/gerrit/3991
Diffstat (limited to 'src/usr/diag/prdf/common/framework')
| -rwxr-xr-x | src/usr/diag/prdf/common/framework/register/prdfScomRegister.C | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/usr/diag/prdf/common/framework/register/prdfScomRegister.C b/src/usr/diag/prdf/common/framework/register/prdfScomRegister.C index c2631ebfa..803fb5c8f 100755 --- a/src/usr/diag/prdf/common/framework/register/prdfScomRegister.C +++ b/src/usr/diag/prdf/common/framework/register/prdfScomRegister.C @@ -230,7 +230,7 @@ bool ScomRegister::operator < ( const ScomRegister & i_rightRegister ) const { if( iv_scomAddress == i_rightRegister.GetAddress() ) { - return ( iv_chipType == i_rightRegister.getChipType() ); + return ( iv_chipType < i_rightRegister.getChipType() ); } else { |
