diff options
| author | Claus Michael Olsen <cmolsen@us.ibm.com> | 2016-09-21 08:37:45 -0500 |
|---|---|---|
| committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2016-09-25 13:33:43 -0400 |
| commit | d6de3a38537e649d1662d3d13d8d0c1bdac57e1e (patch) | |
| tree | 605cebf8f8d80d739ea22b6ad4cfcb7109054ca1 /src/import/chips/p9/procedures/hwp | |
| parent | 9f572b779b2cb655405489af31fa7ac256de2400 (diff) | |
| download | talos-hostboot-d6de3a38537e649d1662d3d13d8d0c1bdac57e1e.tar.gz talos-hostboot-d6de3a38537e649d1662d3d13d8d0c1bdac57e1e.zip | |
Updating hcode_image_build to catch empty .rings section.
Change-Id: I0dfbc22cd80e61cd6eb4d37c0b6a5616d5c9a345
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/30019
Reviewed-by: Richard J. Knight <rjknight@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Dean Sanner <dsanner@us.ibm.com>
Reviewed-by: Martin Peschke <mpeschke@de.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/30029
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/import/chips/p9/procedures/hwp')
| -rw-r--r-- | src/import/chips/p9/procedures/hwp/pm/p9_hcode_image_build.C | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/src/import/chips/p9/procedures/hwp/pm/p9_hcode_image_build.C b/src/import/chips/p9/procedures/hwp/pm/p9_hcode_image_build.C index 083883cc0..9752ba539 100644 --- a/src/import/chips/p9/procedures/hwp/pm/p9_hcode_image_build.C +++ b/src/import/chips/p9/procedures/hwp/pm/p9_hcode_image_build.C @@ -747,6 +747,13 @@ extern "C" break; } + if( ppeSection.iv_size == 0 ) + { + FAPI_ERR("Empty .rings section not allowed: <.rings>.iv_size=%d", ppeSection.iv_size); + rc = BUILD_FAIL_CMN_RINGS; + break; + } + uint8_t* pScanRing = ppeSection.iv_offset + (uint8_t*) (i_pImageIn ); if( ( PLAT_CME != i_platId ) && ( PLAT_SGPE != i_platId ) ) |
