diff options
author | Claus Michael Olsen <cmolsen@us.ibm.com> | 2016-09-06 14:59:11 -0500 |
---|---|---|
committer | William G. Hoffa <wghoffa@us.ibm.com> | 2016-09-08 11:21:17 -0400 |
commit | f802d0bbc5a33966d22c45b35a9625bc55f5e72b (patch) | |
tree | 5f5ac44b6f739fa8b42a999424f9e58f70146851 /src/import/chips/p9 | |
parent | 33a392720347794d6dd0f6d7502e615930241e1d (diff) | |
download | talos-hostboot-f802d0bbc5a33966d22c45b35a9625bc55f5e72b.tar.gz talos-hostboot-f802d0bbc5a33966d22c45b35a9625bc55f5e72b.zip |
xip_customize: More removal of trace output noise for Cronus.
Change-Id: I67153e2e00c2f09b407020fb14039b1fab2cf794
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/29286
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Prachi Gupta <pragupta@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/29294
Reviewed-by: Hostboot Team <hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com>
Diffstat (limited to 'src/import/chips/p9')
-rw-r--r-- | src/import/chips/p9/procedures/hwp/accessors/p9_mvpd_ring_funcs.C | 13 | ||||
-rw-r--r-- | src/import/chips/p9/utils/imageProcs/p9_tor.C | 86 |
2 files changed, 76 insertions, 23 deletions
diff --git a/src/import/chips/p9/procedures/hwp/accessors/p9_mvpd_ring_funcs.C b/src/import/chips/p9/procedures/hwp/accessors/p9_mvpd_ring_funcs.C index 1c6650825..db02498ec 100644 --- a/src/import/chips/p9/procedures/hwp/accessors/p9_mvpd_ring_funcs.C +++ b/src/import/chips/p9/procedures/hwp/accessors/p9_mvpd_ring_funcs.C @@ -284,10 +284,15 @@ extern "C" //@TODO: Uncomment the following after PowerOn. Also, need to come // to agreement whether this should be fatal error or not. // For now, for PO, it's considered benign and noise and is - // being commented out... most of it at least. - FAPI_ASSERT( l_ringLen != 0, - fapi2::MVPD_RING_NOT_FOUND(). - set_CHIP_TARGET(i_fapiTarget) ); + // being commented out and we just exit with manually setting + // the RC ring not found but which doesn't activates the FFDC + // capturing. + if (l_ringLen == 0) + { + fapi2::current_err = RC_MVPD_RING_NOT_FOUND; + goto fapi_try_exit; + } + //FAPI_ASSERT(l_ringLen != 0, // fapi2::MVPD_RING_NOT_FOUND(). // set_CHIP_TARGET(i_fapiTarget). diff --git a/src/import/chips/p9/utils/imageProcs/p9_tor.C b/src/import/chips/p9/utils/imageProcs/p9_tor.C index a0534a546..cd0a51de3 100644 --- a/src/import/chips/p9/utils/imageProcs/p9_tor.C +++ b/src/import/chips/p9/utils/imageProcs/p9_tor.C @@ -316,7 +316,7 @@ int get_ring_from_sbe_image ( void* i_ringSection, // Image pointer return IMGBUILD_TGR_BUFFER_TOO_SMALL; } - if(i_dbgl > 1) + if(i_dbgl > 0) { MY_INF(" ring container of %s is found in the SBE image container \n", o_ringName); @@ -360,8 +360,12 @@ int get_ring_from_sbe_image ( void* i_ringSection, // Image pointer } else { - MY_INF(" ring container of %s is not found in the SBE image container \n", - o_ringName); + if(i_dbgl > 0) + { + MY_INF(" ring container of %s is not found in the SBE image container \n", + o_ringName); + } + return IMGBUILD_TGR_RING_NOT_FOUND; } } @@ -369,7 +373,11 @@ int get_ring_from_sbe_image ( void* i_ringSection, // Image pointer { if(chiplet_offset) { - MY_INF("Ring container is already present in the ring section \n"); + if(i_dbgl > 0) + { + MY_INF("Ring container is already present in the ring section \n"); + } + return IMGBUILD_TGR_RING_AVAILABLE_IN_RINGSECTION; } @@ -486,8 +494,12 @@ int get_ring_from_sbe_image ( void* i_ringSection, // Image pointer } else { - MY_INF(" ring container of %s is not found in the SBE image container \n", - o_ringName); + if(i_dbgl > 0) + { + MY_INF(" ring container of %s is not found in the SBE image container \n", + o_ringName); + } + return IMGBUILD_TGR_RING_NOT_FOUND; } } @@ -495,7 +507,11 @@ int get_ring_from_sbe_image ( void* i_ringSection, // Image pointer { if(chiplet_offset) { - MY_INF("Ring container is already present in the ring section \n"); + if(i_dbgl > 0) + { + MY_INF("Ring container is already present in the ring section \n"); + } + return IMGBUILD_TGR_RING_AVAILABLE_IN_RINGSECTION; } @@ -667,8 +683,12 @@ int get_ring_from_sgpe_image ( void* i_ringSection, // Image pointer } else { - MY_INF(" ring container of %s is not found in the SGPE image container \n", - o_ringName); + if(i_dbgl > 0) + { + MY_INF(" ring container of %s is not found in the SGPE image container \n", + o_ringName); + } + return IMGBUILD_TGR_RING_NOT_FOUND; } } @@ -676,7 +696,11 @@ int get_ring_from_sgpe_image ( void* i_ringSection, // Image pointer { if(chiplet_offset) { - MY_INF("Ring container is already present in the ring section \n"); + if(i_dbgl > 0) + { + MY_INF("Ring container is already present in the ring section \n"); + } + return IMGBUILD_TGR_RING_AVAILABLE_IN_RINGSECTION; } @@ -796,8 +820,12 @@ int get_ring_from_sgpe_image ( void* i_ringSection, // Image pointer } else { - MY_INF(" ring container of %s is not found in the SGPE image container \n", - o_ringName); + if(i_dbgl > 0) + { + MY_INF(" ring container of %s is not found in the SGPE image container \n", + o_ringName); + } + return IMGBUILD_TGR_RING_NOT_FOUND; } } @@ -805,7 +833,11 @@ int get_ring_from_sgpe_image ( void* i_ringSection, // Image pointer { if(chiplet_offset) { - MY_INF("Ring container is already present in the ring section \n"); + if(i_dbgl > 0) + { + MY_INF("Ring container is already present in the ring section \n"); + } + return IMGBUILD_TGR_RING_AVAILABLE_IN_RINGSECTION; } @@ -976,8 +1008,12 @@ int get_ring_from_cme_image ( void* } else { - MY_INF(" ring container of %s is not found in the CME image container \n", - o_ringName); + if(i_dbgl > 0) + { + MY_INF(" ring container of %s is not found in the CME image container \n", + o_ringName); + } + return IMGBUILD_TGR_RING_NOT_FOUND; } } @@ -985,7 +1021,11 @@ int get_ring_from_cme_image ( void* { if(chiplet_offset) { - MY_INF("Ring container is already present in the ring section \n"); + if(i_dbgl > 0) + { + MY_INF("Ring container is already present in the ring section \n"); + } + return IMGBUILD_TGR_RING_AVAILABLE_IN_RINGSECTION; } @@ -1109,8 +1149,12 @@ int get_ring_from_cme_image ( void* } else { - MY_INF(" ring container of %s is not found in the CME image container \n", - o_ringName); + if(i_dbgl > 0) + { + MY_INF(" ring container of %s is not found in the CME image container \n", + o_ringName); + } + return IMGBUILD_TGR_RING_NOT_FOUND; } } @@ -1118,7 +1162,11 @@ int get_ring_from_cme_image ( void* { if(chiplet_offset) { - MY_INF("Ring container is already present in the ring section \n"); + if(i_dbgl > 0) + { + MY_INF("Ring container is already present in the ring section \n"); + } + return IMGBUILD_TGR_RING_AVAILABLE_IN_RINGSECTION; } |