summaryrefslogtreecommitdiffstats
path: root/src/build
Commit message (Collapse)AuthorAgeFilesLines
* Remove workaround setting EFUSE3 power-on valueChristian Geddes2020-01-141-7/+1
| | | | | | | | | | | | | | | | We can remove this workaround as it is now correct in the simics copy of the power on values. Change-Id: I7a37e17595b8932406dbc98342107a0a04d66c3d RTC: 244050 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/89264 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: Roland Veloz <rveloz@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Support for ATTR_TMP and GUARD in standalone simicsDan Crowell2020-01-105-3/+55
| | | | | | | | | | | | | | | | | If a file called ATTR_TMP or GUARD exists in the simics search path, they will be automatically loaded into pnor when simics starts. The offset is computed when the pnor is built by way of a generated python file that we then load into simics. Change-Id: I4d68215a589dcbdaefb2a67e65d1f6928ce639fa Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/88398 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Luis P Fernandez <luis.fernandez@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: Nicholas E Bofferding <bofferdn@us.ibm.com>
* ffdcExpander bash utility tool, written for December, 2019 HackathonRoland Veloz2020-01-101-0/+727
| | | | | | | | | | | | | | | | | | | | | | | | | | | | The ffdcExpander tool will extract the various debug elements from a gzipped SYSDUMP file, a SYSDUMP file or from a HB dump file with a given build tag. The given dump files can be located in the current local directory or in a directory elsewhere. The artifacts that are produced are: Trace, Printk, Errl, Errl with tool option 0X9000013E, Ps, Ps with backtrace, PageMgr and Bltrace. These artifacts can be produced in the current local directory or redirected to a directory elsewhere via option -d, default is the local directory. Caller has the option of adding the artifacts attached to a defect via the -a option. If caller uses the -a option followed by a defect number, then the caller will be prompted for their CQ user id and password. Passwords will not be displayed as they are typed. An artifact that is produced is a file with the name of the FSP build tag. Mainly as a FYI to the caller. The contents of this file is the build tag, FSP driver, FSP driver location and all the commands that ran to create the other artifacts: Trace, Printk, etc. Change-Id: Ib0dc202f4a7e77336ffb097b6647874dd7cebad0 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/89463 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>
* Force a Hostboot dump on any TI in SimicsDan Crowell2020-01-072-4/+28
| | | | | | | | | | | | | | | Execute a magic instruction in the TI path to force a hostboot dump to be collected on any TI while running in Simics. Change-Id: I8aeffb2b646bbe8480568e8af33a658400fa01a5 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/88831 Reviewed-by: Nicholas E Bofferding <bofferdn@us.ibm.com> Reviewed-by: Matt Derksen <mderkse1@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 reporting eecache if no record specifiedMatt Derksen2019-12-191-4/+11
| | | | | | | | | | | | | | | | | eecache_editor.pl failed to run if no valid matching record was specified to the script. We want it to just dump the parsed version of the entire cache in that case. Change-Id: Ic010d3540aa71c8104de0e1945b91bab8f456d7d Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/88559 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: Chen Du <duchen@us.ibm.com> Reviewed-by: Christian R Geddes <crgeddes@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Introduce eecache_editor.pl toolChristian Geddes2019-12-121-0/+795
| | | | | | | | | | | | | | | | | This commit introduces the eecache_editor tool which can be used to take an ecc-less EECACHE section from pnor and provide human readable information about its contents, read specific cache entries out, and to clear/overwite exists entries in a given cache. Change-Id: I4c74f5172e7a681d34b3b654158dc8d76332107a Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/80622 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: Chen Du <duchen@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Automatically include config.hDan Crowell2019-12-061-0/+2
| | | | | | | | | | | | | | | | | | Rather than having to remember to include config.h anywhere we reference a CONFIG variable (and usually forgetting), this adds it to the default compiler flags so that it gets included in every source file we build. Change-Id: I53622ab4d46c55d942e98cae6ec03049fd5b3d08 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/87475 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: Zachary Clark <zach@ibm.com> Reviewed-by: Roland Veloz <rveloz@us.ibm.com> Reviewed-by: Christian R Geddes <crgeddes@us.ibm.com> Reviewed-by: Nicholas E Bofferding <bofferdn@us.ibm.com>
* Add missing trace point to BlTrace.pm toolDan Crowell2019-11-161-0/+1
| | | | | | | | | | | | | | | We were missing the FD error path. Change-Id: Ib52b2ee7dd3ebe7583224b8ef57e85fce561641e Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/86960 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: Chen Du <duchen@us.ibm.com> Reviewed-by: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Update simics level to f0c3530f4fefad5236391e61ae29b1a2582636caDan Crowell2019-11-141-1/+1
| | | | | | | | | | | | | | | Fixes for - 2 proc axone model - BOOT_CONFIG responses for Explorer Change-Id: I30273f3b64ec5cfc28331916883c1dfdce744d0e Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/86870 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: Daniel M Crowell <dcrowell@us.ibm.com>
* Fix timestamp parser for OpenBMC ESELsDan Crowell2019-11-081-10/+34
| | | | | | | | | | | | | | | | | | | | Evidently the OpenBMC code changed their timestamp format at some point so our parser isn't grabbing it anymore. It will now handle data in a format like this: "raweSEL": "00 00 df 00 00 00 00 20 00 04 ... "timestamp": "2019-10-31 10:20:50" Change-Id: Ieeae63505ce0497a3b3fcf1fd0a92455c8319e29 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/86538 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: Corey V Swenson <cswenson@us.ibm.com> Reviewed-by: William G Hoffa <wghoffa@us.ibm.com>
* New Optional Argument to PnorUtils.pm’s loadPnorLayoutLuis Fernandez2019-10-294-21/+43
| | | | | | | | | | | | | | | | | | New Optional Argument to PnorUtils.pm’s loadPnorLayout to declare where to place the newly created layout file. If argument is not given, file will not be saved. Edited scripts and makefiles that use this function. Change-Id: I7b9ee0f7ad33ad88be58598ddf594765eabe29ea Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85916 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: Christian R Geddes <crgeddes@us.ibm.com> Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Fix linker Object initCorey Swenson2019-10-291-2/+4
| | | | | | | | | | | | | | | | Builds on Ubuntu are failing with hbibl.bin size error. Traced back to linker struct Object constructor missing the init of sfpr section. Change-Id: I9ad7ab90f02d403dc0237831736cc615e73738e6 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/86066 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Reviewed-by: Nicholas E Bofferding <bofferdn@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Zachary Clark <zach@ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: William G Hoffa <wghoffa@us.ibm.com>
* Automatically set VPD cache flags based on EEPROM cache settingDan Crowell2019-10-171-26/+4
| | | | | | | | | | | | | | | | Added more logic to the vpd/HBconfig file to force the correct (use hardware) values for the VPD code if the EECACHE flag (SUPPORT_EEPROM_CACHING) is set. This allows the system config files to not include the unused VPD flags anymore. Change-Id: I87f7c5f3e51e3121c081b3007164dbf21cbafba8 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84757 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> Reviewed-by: Corey V Swenson <cswenson@us.ibm.com> Reviewed-by: William G Hoffa <wghoffa@us.ibm.com>
* HCBI-239 Enable static analysisDan Larson2019-10-092-21/+27
| | | | | | | | | | | | | Change-Id: Id07e25aeb7337060baab8eaa2bbf4e3e8df238cd Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84660 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: Luis P Fernandez <luis.fernandez@ibm.com> Reviewed-by: Camvan T Nguyen <ctnguyen@us.ibm.com> Reviewed-by: Nicholas E Bofferding <bofferdn@us.ibm.com> Reviewed-by: William G Hoffa <wghoffa@us.ibm.com>
* Update dimmspd used by explorers in axone simicsChristian Geddes2019-10-072-2/+2
| | | | | | | | | | | | | | We want the explorer's SPD to contain valid i2c addresses for the PMIC for simics. Currently simics has these as 0x90 and 0xC0. Normally we ould just update simics directly and then update the simics relase but there has been issues in the last few weeks getting onto a newer simics release and we need these changes to test functionality that is being added to aid with hardware bringup. Change-Id: I1e29205cef59da986092bc8b609ebe27a89c5611 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84682 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Add call to fapi wrapper of exp_getidec into hwasPlat codeChristian Geddes2019-10-031-1/+8
| | | | | | | | | | | | | | | | | | | | | | The file hwasPlat.C contains the code we use to lookup and compare the IDEC information of a given OCMB when read from the card's SPD versus what is read from the card's defined registers. Recently the code that looks up the IDEC from the card's registers was made into a HWP so it could be leveraged by multiple platforms. A new wrapper has been created on the hostboot side in src/usr/fapiwrap/ that provides an interface for the hostboot platform to call in order to leverage the new EKB hwp. Change-Id: I9155767b199c9b197c010521f33cc0d418901628 RTC: 214627 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/83109 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: Daniel M Crowell <dcrowell@us.ibm.com> Reviewed-by: William G Hoffa <wghoffa@us.ibm.com>
* Compile Hostboot with -OsDean Sanner2019-10-032-1/+17
| | | | | | | | | | | | | | | | | | | Hostboot currently spends a significant percentage of time loading data from PNOR across a relatively slow LPC bus. This commit enables compilation of Hostboot with -Os, optimized for space to reduce the amount of data required to be transferred across the LPC bus Rough measurements show a ~30% size reduction of the hostboot image and a ~10% reduction (improvement) in boot time Change-Id: Ib73863019595a24cc7d10e924bc5c92cd64af1ed Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84436 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>
* Fixed portabiliy issue in pre-commit hookZane Shelley2019-10-021-1/+1
| | | | | | | | | | | | | | A change was previously applied to pre-commit via 1d4530dac35 that included a bashism that is not portable with other shells. Change-Id: I0ab536005973627953e1a3b4fa41f51963a953ea Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84413 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Luis P Fernandez <luis.fernandez@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>
* Increase hostboot standalone timeout to be 75 minutesChristian Geddes2019-10-011-2/+8
| | | | | | | | | | | | | | | | We were seeing normal CIs take 56 minutes which was exceeding the existing 50 minute timeout. This increase is due the adding a 2nd proc to the axone simics xml and the addition of axone related tests. Change-Id: I0fa1f2ce93efab47e4ccb7d81a13cf293b3ed5b0 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84549 Reviewed-by: DANIEL J LARSON <dlarson@us.ibm.com> Reviewed-by: Matt Derksen <mderkse1@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: Daniel M Crowell <dcrowell@us.ibm.com>
* Patch For: Implement Static Analysis with cppcheckLuis Fernandez2019-09-181-38/+33
| | | | | | | | | | | | | | | CPPCHECK commands fail in machines that do not use bin/sh. Wrap commands inside an if-statement to only be read when User explicitly run the tool. Change-Id: Ifad56b0978c8190efa033203f661daf9259a4d43 RTC: 201451 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/83938 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>
* Update axone pnor layout to move eecache to the beginingChristian Geddes2019-09-162-9/+11
| | | | | | | | | | | | | | | | | Because we rely on having a stable offset for the EECACHE section of pnor we will move it to the start of the pnor layout. This will make it less likely to move around as now we are auto-adjusting offsets if sections like HBI happen to exceed to allotted space. Change-Id: I4463c9d1cfb6ba86c777c976f911b14be5a483e3 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/83791 Reviewed-by: Zachary Clark <zach@ibm.com> 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: William G Hoffa <wghoffa@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Update startup.simics To Disable SBE UpdatesIlya Smirnov2019-09-131-3/+3
| | | | | | | | | | | | | | | Update the startup.simics command to disable SBE update using new simics UI. Change-Id: I80ab844fe5dcb976b9d41907ac3803e12edb801a Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/83736 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: Michael Baiocchi <mbaiocch@us.ibm.com> Reviewed-by: Matthew Raybuck <matthew.raybuck@ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Do not allow PNOR partition adjustments when not in test modeDan Crowell2019-09-121-1/+1
| | | | | | | | | | | | | | | | | | In some cases we were seeing confusing error messages while building pnor images in op-build that pointed at the wrong partition being the problem. This was caused by some logic that automatically adjusts the size of the HBI partition to handle our testcase environment. Change-Id: Ib6089ca9a0c68d4c8814fe7a36404096b42837b7 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82952 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: Zachary Clark <zach@ibm.com> Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com> Reviewed-by: William G Hoffa <wghoffa@us.ibm.com>
* Fix GCOV build errors for AXONE configurationZach Clark2019-09-112-24/+3
| | | | | | | | | | | | | | | | | | | | | | | | This commit fixes two problems that were preventing GCOV instrumentation from building: 1. There was an uninitialized variable in fapi2GetChildrenTest.H 2. The PNOR layout for AXONE had physical offsets for each partition specified manually, which was thwarting the automatic offset calculation required to be able to successfully adjust the partition layout when the size of HBI exceeds its initial allotment due to GCOV instrumentation There is another uninitialized-variable warning in the import tree which will be fixed by another commit. Change-Id: Ibcd5e9d62a93589836cb10697e9a963428571131 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/83524 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: 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>
* Implement Static Analysis with cppcheckMegan2019-09-1113-9/+244
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Cppcheck static analysis tool can now be run when compiling by running: make cppcheck And it will always run when committing without stopping the commit if an error is found. This commit suppresses some false positives found in the code. A false positive can be suppressed inline by placing “// cppcheck-suppress syntaxError” in a line before the false positive., e.g.: char arr[5]; // cppcheck-suppress arrayIndexOutOfBounds arr[10] = 0; Change-Id: I9e833a153d6b04df2f8a72cd994e43922c61c2fe RTC: 201451 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/79629 Reviewed-by: Zachary Clark <zach@ibm.com> 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: 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>
* Account for Explorer's DMB_REVISION format in its SPDChristian Geddes2019-09-113-4/+7
| | | | | | | | | | | | | | | | | | | | | | We have been told that the field we are reading to figure out the EC level of the OCMB via its SPD (offset 0x200) is going to say 0xA0 for the first version of the card, at IBM we would call this EC 0x10. The Explorer manufacture has stated that they will increment the 1st nibble when they do match card revisions, and the 2nd nibble on minor revisions. This commit adds the logic to convert the manufacture's revision format into IBM's EC level format. It also updates the prebuilt cache we use in simics for axone to have the expected 0xA0 value for the explorer SPD stored in the prebuilt cache Change-Id: I10e9523a8fbea847e3124c256fab7bee850cb29d RTC: 214627 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82873 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>
* (Fix for:) Alert when a partition's space utilization exceeds a thresholdLuis Fernandez2019-09-041-2/+2
| | | | | | | | | | | | | | | | Fix for an earlier commit that is failing with a newer version of the PERL interpreter. Change-Id: I193e4712d5ac95f18bb2fe67efc11a955234b579 RTC: 212220 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/83204 Tested-by: Jenkins Server <pfd-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> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Nicholas E Bofferding <bofferdn@us.ibm.com> Reviewed-by: William G Hoffa <wghoffa@us.ibm.com>
* Axone 2-Proc Standalone Sim EnablementBill Hoffa2019-08-302-2/+2
| | | | | | | | | | | | | | | - XML Additions for all targets on 2nd proc Change-Id: Ib8a860e4679e08253076abbbade98f4ba172e81d RTC: 208448 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82685 Reviewed-by: Christian R Geddes <crgeddes@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: Nicholas E Bofferding <bofferdn@us.ibm.com>
* Alert when a partition's space utilization exceeds a thresholdMegan2019-08-282-2/+37
| | | | | | | | | | | | | | | | | Calculate the percentage utilization of a pnor partition by using binary file size + header size + ecc overhead (if used for partition). User is warned when a pnor partition exceeds a specific utilization threshold Change-Id: I00b0b69a039ffefdf086dc470fd0f76b72ee267d RTC: 212220 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/81990 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: William G Hoffa <wghoffa@us.ibm.com>
* Allow defining a partition with automatic starting offsetMegan2019-08-273-47/+90
| | | | | | | | | | | | | | | | | | If section does not have a starting offset, calculate starting offset using previous section's offset/size, generate a new output file with offsets and feed into existing tools, throw error if any sections collide Change-Id: I9adf3357b3751087535854a89fc634a415b870d0 RTC: 212219 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/80098 Reviewed-by: Nicholas E Bofferding <bofferdn@us.ibm.com> Reviewed-by: Glenn Miles <milesg@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: William G Hoffa <wghoffa@us.ibm.com>
* Update Sim level to 08_23_19_f5e686_simics.tar.gzBill Hoffa2019-08-261-1/+1
| | | | | | | | | | | | Change-Id: I5f912de4fd98914e6b9f4fd9ff2cbbb2134575ad Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82772 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: William G Hoffa <wghoffa@us.ibm.com>
* Update sim scripts to use $proc_chip_type to lookup chip for AxoneChristian Geddes2019-08-202-4/+4
| | | | | | | | | | | | | | | Update the simics scripts to use variables provided by simics Change-Id: I5c01760630911d723286fcb1450a78e7e44c56a1 RTC: 204842 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82306 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: Corey V Swenson <cswenson@us.ibm.com> Reviewed-by: Roland Veloz <rveloz@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Fix BPM flash image script to properly calculate line lengthMatthew Raybuck2019-08-091-12/+25
| | | | | | | | | | | | | | | | | | | | | Before this commit, the flash image script made certain assumptions about the flash image file when calculating the number of bytes on a line. Those assumptions aren't always true. Therefore, this commit adds a function to properly determine the number of bytes on a line for any arbitrary amount of extraneous characters on the line by stripping them all off prior to calculation. Change-Id: Ibeea1f62177109a512920be1434557ac55510743 RTC:212448 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/81947 Reviewed-by: Christian R Geddes <crgeddes@us.ibm.com> Reviewed-by: William G Hoffa <wghoffa@us.ibm.com> Reviewed-by: Roland Veloz <rveloz@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> 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>
* Revert "Compile Hostboot with -Os"Daniel M Crowell2019-08-082-17/+1
| | | | | | | | | | | | This reverts commit cd1e5c91e4a15ca933c699553ece9b841825ac03. The previous change seems to be causing problems with thread local storage (TLS) access. CQ: SW471547 Change-Id: Ic6f43fbba2a3d5abe94ea73e8e5fd03702b84365 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/81652 Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com> Tested-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Fix BPM script to put temp files into obj treeDan Crowell2019-08-071-3/+3
| | | | | | | | | | | | | | | | | Depending on where the input files and scripts live and if we are compiling in a backing build or a sandbox, the temporary (.crc) files can sometimes be created inside a readonly tree. This change will place the temp files in the same (writeable) output directory as the final binary images. Change-Id: Ib5e6ed5ffeb825d46681eda89d93311931365663 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/81787 Reviewed-by: Matthew Raybuck <matthew.raybuck@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>
* Enable secure boot in Axone simicsMatt Derksen2019-08-021-4/+19
| | | | | | | | | | | | | | | | | | SECUREBOOT::enabled() was returning 0. secure_jumper access was changed in Axone. Updating startup.simics to use new method. Change-Id: I55baac73172dd43257b14850575b61241e9d8083 RTC:201738 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/81456 Reviewed-by: Hieu C Nguyen <hieu.nguyen@us.ibm.com> Tested-by: Jenkins Server <pfd-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> Reviewed-by: Roland Veloz <rveloz@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>
* Remove MVPD,MEMD and CENHWIMG section from Axone pnor layoutChristian Geddes2019-07-303-64/+35
| | | | | | | | | | | | | | | | | This commit removes the MVPD, MEMD and CENHWIMG sections from the axone pnor layout as they are not being used and pnor space is limited. Along with removing the defintion in the pnor layout xml some code changes were required to no longer attempt to load the MVPD/MEMD sections. Change-Id: I20739e30ad497737c0a30b66cc36052c08c11de2 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/80295 Reviewed-by: Glenn Miles <milesg@ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Matt Derksen <mderkse1@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>
* re-enable CRC generation for BPM firmware binaryMatthew Raybuck2019-07-261-11/+7
| | | | | | | | | | | | | | | | | | | Turns back on the functionality to generate a BPM firmware image with calculated CRC included in it. This was turned off temporarily to allow CI to pass on the primary commit and be used as a pre-req to a CMVC feature. Change-Id: Ibe382fc6adf3da820caa668ffc45c33cb30b26ae RTC: 212446 CMVC-Prereq:1087492 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/80765 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Reviewed-by: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: Roland Veloz <rveloz@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>
* Compile Hostboot with -OsDean Sanner2019-07-252-1/+17
| | | | | | | | | | | | | | | | | | | | | | Hostboot currently spends a significant percentage of time loading data from PNOR across a relatively slow LPC bus. This commit enables compilation of Hostboot with -Os, optimized for space to reduce the amount of data required to be transferred across the LPC bus Rough measurements show a ~30% size reduction of the hostboot image and a ~10% reduction (improvement) in boot time Change-Id: Ib140e6f07b112ae95a901f49b75416ef91c94a9d Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/79059 Reviewed-by: Nicholas E Bofferding <bofferdn@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> 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: Zachary Clark <zach@ibm.com> Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com> Reviewed-by: William G Hoffa <wghoffa@us.ibm.com>
* Add BPM firmware update procedureMatthew Raybuck2019-07-231-3/+7
| | | | | | | | | | | | | | | | | | Adds the necessary code to update the firmware data of the BPM. This commit doesn't support the config data portion of the update. The BPM code hasn't been hooked into the existing NVDIMM code yet either. These things, along with error paths, will be added in future commits. Change-Id: I11aa8b8afda26a093682765e488a4de0e31c0f6c RTC:210367 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/79277 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Matt Derksen <mderkse1@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: Daniel M Crowell <dcrowell@us.ibm.com>
* Update buildBpmFlashImages.pl to support configuration data binary generationMatthew Raybuck2019-07-234-94/+575
| | | | | | | | | | | | | | | | | | | Currently, buildBpmFlashImages.pl will only generate the firmware binaries for the BPM update for firmware versions less than 1.05. This commit adds support for the configuration portion of the update for version 1.05 and updates the script so that the generated firmware binaries are compliant with v1.05 requirements. Change-Id: I29b80b01fc8af85ffdefe8c4ad486206e557306e RTC:212446 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/80198 Reviewed-by: Roland Veloz <rveloz@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: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Cache objdump output for build performanceZach Clark2019-07-182-14/+96
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Hostboot uses the objdump tool to disassemble modules and the genlist program to create the listing files (*.list.bz2) for each binary blob generated by the build process (hbibl, hbicore, etc). The hundreds of objdump invocations account for a majority of the build time when recompiling hostboot after a small number of files have been changed because all the shared objects for a binary must be dumped again regardless of whether they have been modified. This commit causes the genlist program and the makefiles to cache the (compressed) disassembly when the BUILD_FAST flag is set so that only modified ELF files need to be redumped. Also several binaries include the same modules, which means that a given module may be dumped multiple times needlessly. Benchmarks for recompilation after modifying small numbers of files show at least a 5x speedup as compared with the original build system. Using BUILD_FAST comes with a cost of about 35 MB of disk space. When compiling all of hostboot (i.e. all libraries must be dumped), using BUILD_FAST is not slower than a vanilla build. The cached objdump output is stored in the img/objdump/ folder. Example: make BUILD_FAST=1 -j32 Change-Id: I65be7b29cec953950bfa664a5ee04e3a711007b5 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/80115 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: Luis P Fernandez <luis.fernandez@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>
* Add BPM utility scripts to support additional firmware update requirementsMatthew Raybuck2019-07-183-0/+702
| | | | | | | | | | | | | | | | | | | Due to the additional requirements with BPM firmware update 1.05, SMART provided several utility scripts to meet these new requirements. Two of those are of use to hostboot. insertBpmFwCrc.py and imageCrc.c, which will be used to calculate and add a CRC to the flash image prior to pre-processing the flash image into a binary. Change-Id: I9cd751f286b039b67e29ea8f5c7b69b1afa85cb9 RTC:212446 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/80197 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: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Force sbe update loop if a change in OMI freq is detectedChristian Geddes2019-07-171-0/+3
| | | | | | | | | | | | | | After we parse the SPD to determine correct frequency settings we need to check if the optimal settings found differ from the original settings we booted with. This commit adds a check for OMI frequency changes in addition to the existing nest frequency and mc sync mode checks. Change-Id: Icaf64eda225df3aab82a033866663e3103cef55f RTC: 207596 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/78739 Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com> Tested-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Add optional params to build-script to skips some stepsChristian Geddes2019-07-101-55/+112
| | | | | | | | | | | | | | | | | | build-script not only compiles Hostboot and primes the ODE sandbox it also performs a copyright check and runs cxx tests. There is a desire to run this script to generate a populated sandbox but we dont need to run the copyright check or test cases so we need flags to avoid that. Change-Id: I93b698a0b8d634bedba43e25361771a0e08875fa Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/79692 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: Roland Veloz <rveloz@us.ibm.com> Reviewed-by: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Start simics in vexec for Axone by defaultChristian Geddes2019-07-101-4/+12
| | | | | | | | | | | | | | | | | | | | | Previously when you called 'hb startsimics' with the axone config we were not running in a vexec shell. This was causing the pool machines to crater if people were running 1 or 2 simics sessions. This commit will check if the '-nre' flag has been passed in, or if we are in a vexec shell already. If niether of those are true then we will boot simics in a vexec shell, if they are -nre has been passed or we are already inside a vexec then we will not boot simics inside a vexec. Change-Id: I17895bb109f18bbcc4b0daafe6c864531bf5994a Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/79573 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: Chen Du <duchen@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Add command echo to sandbox executorDan Crowell2019-07-091-2/+1
| | | | | | | | | | | | | | | Print out the command being run inside the ode sandbox whenever execute_in_sandbox is called. Change-Id: I6c1a78a93860eddaf7f99d88cb402609d2b4e166 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/79549 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: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Revert "Remove references to prebuilt eecache"Christian R. Geddes2019-07-011-0/+3
| | | | | | | | | This reverts commit 1b06c4f813fd7a8395ccfc93c9dea9b22dd488f1. Change-Id: Ibbbd4a25a7374a001333b18aa863e9d5fd04293e Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/79849 Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com> Tested-by: Christian R. Geddes <crgeddes@us.ibm.com>
* Fix errludattribute changes that broke the buildDan Crowell2019-07-011-2/+2
| | | | | | | | | | | | | | | | Split the plugin file into a separate file and renamed it to make it obvious which file (runtime or parser) is which. Change-Id: I60d1b9cf7e5816e2d03ce56f6529c0b88178c489 CQ: SW469233 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/79762 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: Nicholas E. Bofferding <bofferdn@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* HBCI-177: Enable Code Coverage in CIDan Larson2019-06-272-21/+67
| | | | | | | | | | | | | | | | To enable code coverage set in the environment export HOSTBOOT_PROFILE=1 Report is generated under obj/gcov_report Change-Id: I9548c742f5d3bb83aab6d6e4ffd857cccb48b5f1 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/78736 Reviewed-by: Camvan T. Nguyen <ctnguyen@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> 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: Zachary Clark <zach@ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
OpenPOWER on IntegriCloud