summaryrefslogtreecommitdiffstats
path: root/src/build
Commit message (Collapse)AuthorAgeFilesLines
...
* Change code using SANDBOX ENVAR to use CLI variables start sim scriptsChristian Geddes2019-01-041-5/+16
| | | | | | | | | | | | | | | | | | The PHYP simics environment does not use an ODE sandbox for running simics. They are using the hostboot startup scripts though. Hostboot cannot rely on any artifacts from the sandbox such as sandbox related ENV variables in these scripts. Change-Id: I1e736149bb341d880c92905447614619d0d7b65c CQ: SW453636 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/70101 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: CALEB SCHLOSSIN <calebs@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Reviewed-by: Matt Derksen <mderkse1@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>
* Better logging for Partial Good Issues (1.5/2)Matt Raybuck2019-01-041-1/+1
| | | | | | | | | | | | | | | | | Add necessary files for HWSV team to be able to make changes for (2/2) Change-Id: I1cc0c799d9fb743f9ab19dd0393015f16a3d4c01 RTC:180365 CMVC-Prereq:1074242 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/70003 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: Ilya Smirnov <ismirno@us.ibm.com> Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com> Reviewed-by: Roland Veloz <rveloz@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Remove Redundant phyiscal map setupChristian Geddes2018-12-191-19/+12
| | | | | | | | | | | | | Axone simics model now does this for us natively Change-Id: I986ae45fa63cfce601ff56754fbcabe2e040d5e1 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/69940 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: Roland Veloz <rveloz@us.ibm.com> Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com>
* Write SPD to DDIMM eeproms for axone simicsChristian Geddes2018-12-181-12/+44
| | | | | | | | | | | | | | | The goal for axone is to read VPD from real HW so we need to preload the simics model with the data we want until they do it for us Change-Id: I1cf337a5bd90a62600d1e8cf43f3e429308992ba Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/69874 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: Roland Veloz <rveloz@us.ibm.com> Reviewed-by: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com>
* Axone Simics Updates -- Finds Functional Master ProcBill Hoffa2018-12-149-109/+179
| | | | | | | | | | | | | | | | | - Updates for new Simics infrastructure - Finds Functional Master Proc - Finds TPM - Boots to istep6.9 before failing target verification (more Axone FW changes needed) Change-Id: Ib3f44008ed12202777c2edacea42d5cb20a4a7a3 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/67206 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Nicholas E. Bofferding <bofferdn@us.ibm.com>
* Driver changes to support i2c muxRoland Veloz2018-12-061-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | - Updated data structures gpioAddr_t, eeprom_addr_t, misc_args_t, nvdimm_addr_t and tpm_info_t with I2C MUX data members. Also added constructors to these structures to default there data members with the correct default info. - Updated macros DEVICE_I2C_PARMS, DEVICE_I2C_ADDRESS and DEVICE_I2C_ADDRESS_OFFSET to take the I2C MUX bus selector parameter and the I2C MUX entity path. - Added method i2cAccessMux to file i2c.H/.C that will setup the call for the I2C MUX. Method i2cCommonOP calls i2cAccessMux which then calls i2cCommonOp with appropriate parameters for the I2C MUX: i2cCommonOP -> i2cAccessMux -> i2cCommonOP. - Updated i2ctest.H with new I2C MUX params to get it to pass. RTC:191352 Change-Id: I6a70860eb2286bbd23d6157d72351b8adfa21aac Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/66651 Reviewed-by: Ilya Smirnov <ismirno@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Nicholas E. Bofferding <bofferdn@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>
* Support parsing call home informational eSELs with debug toolingNick Bofferding2018-12-061-5/+9
| | | | | | | | | | | | | | | | | Modifies eSEL.pl to support decoding call home informational eSELs which have the 'dd' type vs. the 'df' type for other eSELs. Hostboot sends call home informational eSELs when it finds an unacknowledged error from a previous boot. Change-Id: I0c4c8eabe2f104796b5bde05fd493958ddc662c7 CQ: SW452532 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/69474 Reviewed-by: Ilya Smirnov <ismirno@us.ibm.com> Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com> 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>
* Increase HBI partition to 12 MB (w/o ECC), up from 11Nick Bofferding2018-11-292-36/+36
| | | | | | | | | | | | | | HBI partition size is too small to fit all the code, so bumping it by logical 1 MB to provide breathing room Change-Id: I78cf10d033aee2ff904db101dde6d27fb3c00fca Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/69209 Reviewed-by: Ilya Smirnov <ismirno@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Matthew Raybuck <mraybuc@us.ibm.com> Reviewed-by: Roland Veloz <rveloz@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Fix HDAT PNOR Partition's OffsetIlya Smirnov2018-11-281-1/+1
| | | | | | | | | | | | | | | HDAT PNOR partition was overlapping with OCMBFW after the increase of HBRT to 7MB. This commit correctly spaces the HDAT partition. Change-Id: I1d7b1f07b39505fd9e360b0e4e1dfd16cda92eb1 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/69153 Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com> Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com> 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>
* Make TOC partion Read-Only in all PNOR LayoutsMike Baiocchi2018-11-271-6/+12
| | | | | | | | | | | | | | | | | | | This commit ensures that the TOC(s) are all set to "read-only" via the FFS flags for all PNOR files that are created. It also does the same for the "BACKUP_PART" and "OTHER_SIDE" partitions, if they exist, as well. Change-Id: I5698045b27630e2cb83f470193526ceea02c61d3 CQ:SW423354 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/68828 Reviewed-by: Marshall J. Wilks <mjwilks@us.ibm.com> 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: Nicholas E. Bofferding <bofferdn@us.ibm.com> Reviewed-by: Ilya Smirnov <ismirno@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* increase HBRT PNOR sizeZane Shelley2018-11-271-17/+17
| | | | | | | | | | Change-Id: Iaeb839c3f482904bc07118c5a862c2f59b0cad76 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/68862 Reviewed-by: Caleb N. Palmer <cnpalmer@us.ibm.com> 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> Reviewed-by: Zane C. Shelley <zshelle@us.ibm.com>
* all scripts for hdat pnor partitionSampa Misra2018-11-272-0/+12
| | | | | | | | | | | Change-Id: Ied9c154d544d65eebfe5cfb0185ccb26545ee130 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/67144 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: Jayashankar Padath <jayashankar.padath@in.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Update bbuild to b1114a_1846.930Christian Geddes2018-11-201-1/+1
| | | | | | | | | | | | | | | Backing Fips build was getting old, was about to add pre-req from three weeks ago then I figured I would just move the bbuild up instead. Change-Id: I986c13771c584725667d6d2c7887ba7e7e582147 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/68937 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: William G. Hoffa <wghoffa@us.ibm.com> Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com>
* Serialize the Check For Prologs in CIIlya Smirnov2018-11-191-12/+10
| | | | | | | | | | | | | | | | | | | | | Prolog checks are performed in parallel with build in hostboot CI process, and the return code from the prolog checking script is checked after the build job completes, which introduces significant delays before the developer is informed of incorrect prologs. This change serializes the check for prolog correctness and forces the check to run before the build process, so that the developer is informed about bad prologs as soon as possible. Change-Id: Ib84802f14022712268db1b0b539396e59d25792f Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/68678 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com> Reviewed-by: Nicholas E. Bofferding <bofferdn@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* PRD: Make room for Axone prf files in HostbootZane Shelley2018-11-161-1/+1
| | | | | | | | | | | | | Change-Id: Ic0f4fda42326f714538b392c4599398d9dfc65e5 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/68811 Reviewed-by: Brian J. Stegmiller <bjs@us.ibm.com> Reviewed-by: Caleb N. Palmer <cnpalmer@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Zane C. Shelley <zshelle@us.ibm.com>
* Pushing HB plugins and related files for building errltoolVenkatesh Sainath2018-11-153-7/+55
| | | | | | | | | | | | Change-Id: I91d432a901fb4ee6de469ca8076637eaa2e1315d Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/60027 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-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> Reviewed-by: Sampa Misra <sampmisr@in.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Add OCMBFW partition to FSP and Hostboot PNOR layoutsNick Bofferding2018-11-012-0/+20
| | | | | | | | | | | | | | | | Added new Open CAPI Memory Buffer Firmware partition to the FSP and Hostboot PNOR layouts, which will initially contain the explorer firmware. Change-Id: I56c14dfaf7a0e1e06ec733bbf1dfe018975b9742 RTC: 193924 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/68059 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: Ilya Smirnov <ismirno@us.ibm.com> Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Disable automatic collection of SIM logs with MAGIC_SET_LOG_LEVELChristian Geddes2018-11-011-4/+6
| | | | | | | | | | | | | | | | | | | | While we were trying to trace down some PSU simics bugs we added a new feature where HB can elevate log levels of simics components and copy out simics traces to a file which can be used for FFDC. We found the bug which prompted this and we are seeing issues where attempting to set the log level on procs in node2 is problematic. This commit makes it so no logs are captured unless the env var ENABLE_HB_SIMICS_LOGS is set in simics env. Change-Id: Icc049b86daf89ff037edb1b5a1cf99c9ce551a2d Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/68140 Reviewed-by: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: CALEB SCHLOSSIN <calebs@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Fix incorrect syntax in addimgid shell scriptArtem Senichev2018-10-291-2/+2
| | | | | | | | | | | | | | | | | | | Construction '>&' is a bash specific redirection. Make-based build system uses posix shell (sh), not bash. As a result, during the call to addimgid from Makefile we got 'Syntax error: Bad fd number' and empty image name. Anyway, it is a bad idea to hide build logs. Resolves #149 Signed-off-by: Artem Senichev <a.senichev@yadro.com> Change-Id: I77102484f41e4198cca8906a6698fc42b7574380 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/67624 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: Corey V. Swenson <cswenson@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Make Boootloader trace tool follow HRMORDan Crowell2018-10-291-2/+2
| | | | | | | | | | | | | | | | Got rid of a hardcoded value inside BlTrace.pm so that it will follow the HRMOR like all of the other debug tools we have. Change-Id: I7ca6c656962031223078403b86897db0e5d62285 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/68019 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: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: Corey V. Swenson <cswenson@us.ibm.com> Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com> Reviewed-by: Nicholas E. Bofferding <bofferdn@us.ibm.com>
* Break down Dump requests into 1MB chunksDan Crowell2018-10-291-15/+27
| | | | | | | | | | | | | | | | | I have observed situations where it either takes several hours for a large getmem call to return or the call just hung. This commit will break down large reads into smaller 1MB pieces and also display the progress. Change-Id: I31ea4c231ee89a065e53f160d8506f8c8cf607c7 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/67503 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: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com> Reviewed-by: Nicholas E. Bofferding <bofferdn@us.ibm.com>
* Add way for developers to disable hb sim logging via env variableChristian Geddes2018-10-251-1/+8
| | | | | | | | | | | | | | | | | | | | There was a request to add a way to disable a new feature that was added to allow hostboot to temporarily elevate and capture simics component logs. When hostboot puts the magic instructions to enable log capture and elevate log levels it causes log spam and adds to IPL time in simics. Simics devs can now set DISABLE_HB_SIMICS_LOGS = 1 and the magic instruction will look for that env var and skip collecting or elevating the logs. Change-Id: Ib31bad3485833f642533dfd52af9cf941a2772d6 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/67882 Reviewed-by: CALEB SCHLOSSIN <calebs@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Reviewed-by: Ilya Smirnov <ismirno@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: William G. Hoffa <wghoffa@us.ibm.com>
* Remove auto-release & hbRelease toolsCamVan Nguyen2018-10-232-3519/+0
| | | | | | | | | | | | | | These tools have been ported to pfd_infra_tools Change-Id: I5bc519689b8f13b48eb97a194b9b9f8ad0cb2840 RTC: 200487 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/67707 Reviewed-by: Sameer R Veer <sveer@us.ibm.com> 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>
* Adding cumulus model to ci testingElizabeth Liner2018-10-191-0/+4
| | | | | | | | | | | | | Change-Id: I6b7ad221bdc3862ecaa2d9e0a386332499f2be29 RTC:190805 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/58663 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Camvan T. Nguyen <ctnguyen@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: Sameer R. Veer <sveer@us.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com>
* Elevate log levels for simics during PSU opsChristian Geddes2018-10-181-0/+60
| | | | | | | | | | | | | | | | | | | | This commit introduces some new magic instructions one which will temporarily elevate the log levels for given components and another which will start and stop collection of these simics logs. This was added so we can temporarily increase log levels during PSU operations in hopes of catching a timeout we have been seeing in simics and getting more info for the simics teams. Change-Id: I990a4b5413f7ff14796dee36e39199f785aef458 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/67359 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: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: Ilya Smirnov <ismirno@us.ibm.com> Reviewed-by: Hieu C. Nguyen <hieu.nguyen@us.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com>
* genPnorImages: Add VERSION to signed partitionsSamuel Mendoza-Jonas2018-10-161-0/+1
| | | | | | | | | | | | | | | | | Mark VERSION as a signed partition and also check for it in the IPMI FRU setup. Change-Id: I6e8690c4f34d1f5ce92750960e6c73afcbf0a8e9 CQ:SW416159 Signed-off-by: Samuel Mendoza-Jonas <sjonas@au1.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/61439 Reviewed-by: Nicholas E. Bofferding <bofferdn@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Fix bugs in debug tools introduced when adding Axone targetsChristian Geddes2018-10-162-1/+2
| | | | | | | | | | | | | | Found some bugs that was causing simics to fail loading the tools Change-Id: I2fe1b02a6661975ea297141a4bb9bf81e026ed79 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/67374 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: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: Corey V. Swenson <cswenson@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Customize LPC console policy in SBESachin Gupta2018-10-141-1/+1
| | | | | | | | | | | | | | | | | | | SBE has started supporting IPMI console messages on openpower systems. All systems does not support console as its needs LPC/BMC support. HB will customise this policy via ATTR_LPC_CONSOLE_CNFG attribute in SBE Change-Id: Ia10e082d4d3a5bf107b9b813a48ff042a5f40803 CMVC-prereq: 1069051 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/66785 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: HWSV CI <hwsv-ci+hostboot@us.ibm.com> Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Reviewed-by: RAJA DAS <rajadas2@in.ibm.com> Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/66794 Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
* Axone PNOR GenerationBill Hoffa2018-10-114-3/+39
| | | | | | | | | | | | | | | | | - Makefile updates to generate Axone image - Initial Axone Simics XML file created with sys/node/proc targets (Several future commits will add the rest of the targets piece by piece) Change-Id: I3030265a5d81a8aeed527924bf0afd1215ac9740 RTC: 197037 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/64240 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Christian R. Geddes <crgeddes@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> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* ipmi: Split into ipmibase and ipmiext modulesAndrew Jeffery2018-10-101-1/+2
| | | | | | | | | | | | | | | | | | Split the IPMI module into base and ext portions, with the BT interface, device driver and resource provider in the base portion, and all remaining IPMI functionality in the ext portion. The split is in preparation for moving the base functionality in the hostboot base image. Change-Id: Iec864f96240d79f4fadd5519d2ef46437d07c1fd Signed-off-by: Andrew Jeffery <andrewrj@au1.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/66792 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Reviewed-by: Corey V. Swenson <cswenson@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* new targeting sorting tools removes newline from prologMatt Raybuck2018-10-081-1/+1
| | | | | | | | | | | | | | | | | | There was a defect with the new targeting sorting tools where newer versions of xsltproc wouldn't handle the prolog correctly on some systems. This commit resolves the issue. Change-Id: I95ffe09ef865954add1afe4c75c441cecd595221 CQ:SW447229 CMVC-Prereq: 1067370 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/67073 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Reviewed-by: Zane C. Shelley <zshelle@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: Ilya Smirnov <ismirno@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Adding prereqs to fix simics failsDan Crowell2018-10-071-0/+1
| | | | | | | | | | | | | Change-Id: I184cd19ce0cb2ed013103e69fe6e7684b8a3444f CMVC-Prereq: 1067370 CMVC-Prereq: 1067512 CQ: SW447854 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/67123 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>
* Fix cscope segfault problemMatt Derksen2018-10-061-2/+5
| | | | | | | | | | | | | | Remove -q option as this option leads to a segfault on obj/modules/testfapi2/testfapi2.C User will see this: /bin/sh: line 2: 10942 Segmentation fault cscope -bqk Change-Id: If4d6264351605e2313ce481a07f501e5cdec4fdf Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/67113 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Use Simics CPU Object passed in when executing hap handler codeBill Hoffa2018-09-271-29/+39
| | | | | | | | | | | | | Change-Id: I589529f3550e7c9a11f0e0027a75edd0eec48e25 CQ: SW445281 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/66677 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Ilya Smirnov <ismirno@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: Christian R. Geddes <crgeddes@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* SW427193 / HW461448: Enable memory controller watJenny Huynh2018-09-141-0/+3
| | | | | | | | | | | | | | | | | | | | Change-Id: I2fc4cf0dda43d4eba543024605c8f358e22e1bae CQ:SW427193 CQ:HW461448 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/65476 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: HWSV CI <hwsv-ci+hostboot@us.ibm.com> Reviewed-by: Adam S. Hale <adam.samuel.hale@ibm.com> Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com> Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com> Reviewed-by: Thi N. Tran <thi@us.ibm.com> Reviewed-by: Joseph J. McGill <jmcgill@us.ibm.com> Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/65510 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: Christian R. Geddes <crgeddes@us.ibm.com>
* Added portable shebang to applyTargetingStyle toolZane Shelley2018-09-141-1/+25
| | | | | | | | | | | | Change-Id: Iab2877114c8b34d3406bae61f726101515564a20 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/66079 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> Reviewed-by: Matthew Raybuck <mraybuc@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Reviewed-by: Zane C. Shelley <zshelle@us.ibm.com>
* Add commit hook to sort the attribute xml filesMatt Raybuck2018-09-115-1/+129
| | | | | | | | | | | | | | | | | | When xml files were changed it would produce an instant merge conflict. To resolve this issue a pre-commit hook was added that will sort the xml attributes by id preventing instant conflicts. Change-Id: Iffea1abfab459c542fe4f961616e0b5b4062804f RTC: 180350 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/65894 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Nicholas E. Bofferding <bofferdn@us.ibm.com> Reviewed-by: Ilya Smirnov <ismirno@us.ibm.com> Reviewed-by: Roland Veloz <rveloz@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>
* Enable CUMULUS config to use Zeppelin's SPDX overrideDan Crowell2018-08-311-2/+2
| | | | | | | | | | | | | Once the Zeppelin image is building, pull it into our standalone config to test more code paths. Change-Id: Ib7b9a2cbef782781c68ae74c5cc4f27300b8cce6 CMVC-Prereq: 1059608 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/64078 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-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>
* Another fix to symsmode parms for debug modeDan Crowell2018-08-302-3/+3
| | | | | | | | | | | | | | Fixed the mismatched default parm between hb-dump-debug and the framework. Change-Id: Id3af424ed65ed371d9c9dfc09be00212bfe87acb Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/65367 Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com> 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: William G. Hoffa <wghoffa@us.ibm.com>
* Fix some shortname issuesDan Crowell2018-08-292-29/+29
| | | | | | | | | | | | | | | | MEMPORT is using mp Made all processor-level names generic Change-Id: I50ac5742b20645574ce5bd4636fa23ec39357b96 RTC: 172969 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/64065 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: Christian R. Geddes <crgeddes@us.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Move bbuild to b0813a_1832.930Dan Crowell2018-08-284-55/+3
| | | | | | | Change-Id: I26cda4212af7aaee0d85e38faa268625df0922ef Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/65382 Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Tested-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Use Cumulus DD1.3 SBE image instead of DD1.0Dan Crowell2018-08-205-7/+19
| | | | | | | | | | | | | | | Adding support for Cumulus DD1.3 SBE images in the FSP PNOR Dropping support for Cumulus DD1.0 SBE image from FSP PNOR CMVC-Prereq: 1060757 CMVC-Prereq: 1063108 CMVC-Prereq: 1061378 Change-Id: Ia315c6f786ef6c522acf4e580ad435b0180513b3 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/62636 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-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>
* Update ecmd debug scripts and fapi_utils script with Axone targetsChristian Geddes2018-08-082-2/+43
| | | | | | | | | | | | | | | | | In my first pass to get things working I missed updating these scripts. I intentionally didn't update the mrw parsing scripts that work will be done in another story. Change-Id: Ieb11062911a841e83eb0a7fc7fcf6f861e12e241 RTC:172969 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/63917 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: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com>
* Make processing of hrmor value in MemStateInfo consistentChristian Geddes2018-08-011-5/+1
| | | | | | | | | | | | | | | | There was some confusion on whether the value in core scratch 1 that told us where Host HRMOR lives was in MB or just Bytes. Going forward this value will be in bytes. CQ: SW440425 Change-Id: I5e7e5efdd584c3a8c050091b32d5443dff3cd417 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/63627 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Nicholas E. Bofferding <bofferdn@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Avoid SIMICS exception if SBE tracMERG not availableAndrew Geissler2018-07-271-1/+4
| | | | | | | | | | | | | | | | It's not clear what's causing the failure to retrieve SBE traces but do not want to cause SIMICS to fail when this happens. Best to allow SIMICS to continue so further FFDC can be gathered. Change-Id: Icbde1065769a3755e0a4276a06ce627af0edcb3a CQ: SW437459 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/63172 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: Prachi Gupta <pragupta@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Add cumulus cdimm support to auto-release -m pathCamVan Nguyen2018-07-251-1/+12
| | | | | | | | | | | | Change-Id: I643f7605c51ed285567dfd6b532c307e7f61dca0 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/61786 Reviewed-by: Sameer R. Veer <sveer@us.ibm.com> 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> Disable-CI: Camvan T. Nguyen <ctnguyen@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Fix symsmode check to allow in-memory lookupDan Crowell2018-07-242-3/+4
| | | | | | | | | | | | | | Switched the default value from empty string to 'default' and fix the associated checks. Change-Id: If8379ad02854fd46b800e5dbfdfef41558ee745e Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/63154 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Secure Boot: Don't override user setting of SB_KEEP_CACHEDave Heller2018-07-241-1/+7
| | | | | | | | | | | | | | | | | | | The previous patch set SB_KEEP_CACHE=true in genPnorImages.pl in a way that would override the user setting. This was a bad idea. The desired behavior is to set SB_KEEP_CACHE=true but only if it has not already been set (through environment) by the user. Signed-off-by: Dave Heller <hellerda@us.ibm.com> Resolves #144 Change-Id: I5bf04bd1c8501695eccbb5e7229101cb9dab7078 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/62362 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: Nicholas E. Bofferding <bofferdn@us.ibm.com> Reviewed-by: Marshall J. Wilks <mjwilks@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Fixing copyright prolog to 2018 after Gerrit server move from RTP to RCHLANDSameer Veer2018-07-191-1/+2
| | | | | | | | | | | | Change-Id: Ic1dbf6534e26126d7667a0c9fdfdff75c6963b2b Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/62889 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> Reviewed-by: Corey V. Swenson <cswenson@us.ibm.com> Reviewed-by: Prachi Gupta <pragupta@us.ibm.com> Reviewed-by: Camvan T. Nguyen <ctnguyen@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* linker: fix compiler warningsJan Hlavac2018-07-191-9/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In Fedora 28 (x86_64) the linker binary, produced by GCC 8.1.1, always ended in an infinite loop in which it repeatedly appended some text to *.lnkout.bz2 file and *.bin.modinfo file. Example : .hbicore.lnkout.bz2: Error writing to output. Success hbicore.bin.modinfo: hbicore.elf,0x0 The GCC was also displaying warnings about missing return statements (-Wreturn-type). When these warnings were fixed, the problem with infinite loop disappeared. This problem also never appeared when the -O0 flag was used. While being there, also fix some warnings that happen with the -Wall flag. Resolves #143 Change-Id: Ia1ddfa07d3b22b151922de2df1db91ffc2aad2ca Signed-off-by: Jan Hlavac <jhlavac@redhat.com> Signed-off-by: Joel Stanley <joel.stanley@au1.ibm.com> Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/61862 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> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/62830 Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com>
OpenPOWER on IntegriCloud