summaryrefslogtreecommitdiffstats
path: root/src
Commit message (Collapse)AuthorAgeFilesLines
* Enable platSpecialWakeup during IPL time and add supportChristian Geddes2018-02-1711-44/+64
| | | | | | | | | | | | | | | | | | Previously we only supported platSpecialWakeup at runtime because we were not worried about needed special wakeup as the cores would not go into stop state during the IPL. However, now PM code is expecting bits to be set by the p9 special wakeup hwp so we need to call it during the IPL. This code makes the handleSpecialWakeup.C/H files compatible with IPL time build environment and enables the call on the IPL time path CQ: SW415949 Change-Id: I94cca8a84946bea464b21cb95cdbbb075f766c3d Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/52858 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: Christian R. Geddes <crgeddes@us.ibm.com> Tested-by: Christian R. Geddes <crgeddes@us.ibm.com>
* Fix bad dq wiring data translationCaleb Palmer2018-02-171-13/+15
| | | | | | | | | | | | | | Change-Id: I6b7bc977d89a518cc9c152b9e6bff98d2789a5fd CQ: SW416910 Backport: release-fips910 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53896 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Reviewed-by: Benjamin J. Weisenbeck <bweisenb@us.ibm.com> Reviewed-by: Brian J. Stegmiller <bjs@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>
* Add SECBOOT Partition Back to PNORIlya Smirnov2018-02-162-2/+20
| | | | | | | | | | | | | | | | Add SECBOOT partition back to the bottom of PNOR. The partition is the copy of that in Witherspoon xml. Change-Id: Iaa12f5457b7839d10ffde42909678d6b627d8830 RTC: 170687 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53898 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Michael Baiocchi <mbaiocch@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: Daniel M. Crowell <dcrowell@us.ibm.com>
* Update TCE TestcasesMike Baiocchi2018-02-164-270/+177
| | | | | | | | | | | | | | | | | This commit updates and enables the TCE testcases used in HB standalone simics configurations. It also does some minor cleanup to address all outstanding TODOs for this story. Change-Id: I6b1ca049633a8c70b53bb35c776403dc86fec86b RTC:168745 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/54085 Reviewed-by: Nicholas E. Bofferding <bofferdn@us.ibm.com> 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> 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>
* Enable Scom + Targeting unit tests for Cumulus modelBill Hoffa2018-02-164-17/+10
| | | | | | | | | | | | | Change-Id: Ib02c0af30d6b06418f2ed6a2356cee93162aaffb RTC: 178802 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/54136 Reviewed-by: Corey V. Swenson <cswenson@us.ibm.com> Reviewed-by: Prachi Gupta <pragupta@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>
* Remove all explicit calls to special_wakeup from hostboot codeChristian Geddes2018-02-169-385/+3
| | | | | | | | | | | | | | The only time that special wakeup should be called on the hostboot platform is when it is called by other HWPs. PM_RESET will only enable special wakeup if the SKIP_WAKEUP attribute is set to 0 Change-Id: Ib9939eae857fa5113a876a717f8169e150b78318 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/54148 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>
* Add checks to PnorUtils.pm to look for errors in layout filesMike Baiocchi2018-02-161-2/+43
| | | | | | | | | | | | | | | | This commit adds a check that 2 PNOR sections do not start at the same offset. It also checks that adjacent PNOR sections do not have overlapping regions. Change-Id: I57d3a545190e6c9cb47b58977b2b0ec002bf0452 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/54090 Reviewed-by: ILYA SMIRNOV <ismirno@us.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: 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>
* Add TCE-related attributes to the Node targetMike Baiocchi2018-02-161-0/+3
| | | | | | | | | | | | | | | | | | | This commit adds all of the TCE-releated attributes to the node target level such that FSP can use some of them for PSI Dianostics on each node and we have flexibility for TCE support on multinode systems. Change-Id: I85c6c986760f61a0dfb7f8c0de6362f6264596af RTC:187335 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/54035 Reviewed-by: ILYA SMIRNOV <ismirno@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Srikantha S. Meesala <srikantha@in.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>
* Reenable reconfig loop for new bad dq dataCaleb Palmer2018-02-161-32/+48
| | | | | | | | | | | | | | Change-Id: I47ae857062ffe7861d8b130b29dfc7b6baa4249e CQ: SW417224 Backport: release-fips910 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53944 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: Benjamin J. Weisenbeck <bweisenb@us.ibm.com> Reviewed-by: Brian J. Stegmiller <bjs@us.ibm.com> Reviewed-by: Zane C. Shelley <zshelle@us.ibm.com>
* enforce strict 512 GB per socket limit on Witherspoon memory mapJoe McGill2018-02-162-4/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | SW415901 exposed a problem with the current implementation of extended addressing for Witherspoon Coral systems. With fully configured memory present in the system (8x64GB=512GB per socket), GARDing a DIMM will currently result in: - group of 6 fullying occupying 0-512GB address space - group of 1 mapped at 8TB region (2nd extended addressing region) The single group mapping has RA bit 20 active, which is problematic for the NVIDIA device driver. p9_fbc_utils.H p9.trace.scan.initfile for HW423589 option 2, enable chip address extension for chip ID LSB RA bit 21 only. This creates only one 4TB extended addressing region per socket. indirectly, this limits DIMMs to map into the 512 GB region with RA bit 21=0 and should cause an IPL failure if more than 512 GB is plugged or the memory grouping algorithm attempts to spill beyond 512 GB on a given chip p9_mss_eff_grouping.C prohibit formation of group sizes 6 and 3 when HW423589 option2 WA is active Change-Id: I997c080a2821cf3c556a4f8b35d5e0fdb34da500 CQ: SW415901 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53406 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: HWSV CI <hwsv-ci+hostboot@us.ibm.com> Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com> Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Thi N. Tran <thi@us.ibm.com> Reviewed-by: Jenny Huynh <jhuynh@us.ibm.com> Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com> Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53412 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>
* Close race condition in multi instance IPCDean Sanner2018-02-162-3/+16
| | | | | | | | | | | | | | | | | | P9 Doorbells operate differently than P8 IPI mechanism, creating a race condition. Basically because the HB kernel turns a doorbell into a message, and then is ready for another one -- it can lose the next one since repeated messages with same key are dropped (IPC handling clears memory comm area, other HB instances notice, fill and drop a doorbell before getting back into kernel) Change-Id: I8eb6f010600afb9de365c8942fcd24227903436f Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/54118 Reviewed-by: Prachi Gupta <pragupta@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>
* Increase SBE FIFO delay timeoutDean Sanner2018-02-161-1/+1
| | | | | | | | | | | | | | | Architected value for worst case is 90 sec. Code was setting it to 10ms, which was occasionally causing timeouts on long running operations (MPIPL continue). Change-Id: I51b05ad4daa56901402a34dee4a24f73bfd479f2 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53573 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: Prachi Gupta <pragupta@us.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com>
* Send additional FSP progress messages during long isteps (e.g. memdiags)Rick Ward2018-02-156-90/+77
| | | | | | | | | | | | | | | | | | | Updated sendProgressCode() so that it would only output trace or console messages once per step/substep. Replaced most calls to IPMIWATCHDOG::resetWatchDogTimer() with calls to sendProgressCode() which calls resetWatchDogTimer() if communicating with a BMC, or updates the FSP with its progress on FSP-based systems. Change-Id: I29beb7ce5cdae467d26a0b2c5fee9e3cc4629161 RTC: 169682 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53995 Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com> Reviewed-by: Prachi Gupta <pragupta@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 VPD lookup in Firmware bootsDan Crowell2018-02-151-9/+23
| | | | | | | | | | | | | | | | | | | | | The bucket checks are not working in firmware boots because the VPD lookup isn't done until a valid pointer is passed in. Change-Id: I08d58a2cc77f706985ea49b90b1df74a9a863a29 CQ: SW415931 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/54091 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: ANDRE A. MARIN <aamarin@us.ibm.com> Dev-Ready: ANDRE A. MARIN <aamarin@us.ibm.com> Reviewed-by: Louis Stermole <stermole@us.ibm.com> Reviewed-by: STEPHEN GLANCY <sglancy@us.ibm.com> Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com> Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/54095 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>
* Enable FFDC logging for mss_freq for real VPD accessing failsAndre Marin2018-02-151-1/+11
| | | | | | | | | | | | | | | | Change-Id: Id999bb6d73fa98f9caf43a968d818a9300153a7f CQ: SW415931 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53568 Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@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: Louis Stermole <stermole@us.ibm.com> Reviewed-by: STEPHEN GLANCY <sglancy@us.ibm.com> Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com> Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53658 Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Tested-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Add VPD lookup to build a supported frequency list, and bin-down supportAndre Marin2018-02-1513-629/+1022
| | | | | | | | | | | | | | | | | | | | | | | | Bypassing FFDC collection in p9_get_mem_vpd_keyword.C for unsupported configs that appear while polling the VPD for supported frequencies in the memory procedure p9_mss_freq.C. This removes the fail trace during istep to avoid user confusion. Change-Id: Ic679fb2fd8357567059f87d689acc0e0c534cd9f CQ:SW415931 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/51836 Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Dev-Ready: Brent Wieman <bwieman@us.ibm.com> Reviewed-by: STEPHEN GLANCY <sglancy@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: HWSV CI <hwsv-ci+hostboot@us.ibm.com> Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com> Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com> Reviewed-by: Louis Stermole <stermole@us.ibm.com> Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com> Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/51842 Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Tested-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Pass Actual SecureRom size in HDAT to OPALIlya Smirnov2018-02-151-8/+2
| | | | | | | | | | | | | | | | | | | | | Up until now we aligned the SecureRom size for OPAL to consume, requiring different sizes of SecureRom to be used between OPAL and PHYP. However, it turned out that it is unnecessary to align the size for OPAL, so the code can be simplified to remove the distiction between the size for PHYP and OPAL. Note that the address of the SecureRom still needs to be aligned. Change-Id: I223c1af36755efe134ba496c36129f76b14d946d RTC: 183697 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53897 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: Marshall J. Wilks <mjwilks@us.ibm.com> Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com> Reviewed-by: Nicholas E. Bofferding <bofferdn@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Moving Sbe Retry Handler work to common file for runtimeElizabeth Liner2018-02-155-10/+39
| | | | | | | | | | | | | | | | We want to move the sbe_retry_handler.C and other files associated with it to a common directory and makefile. Change-Id: Ifc725709d23d9eec75d2f91b2be73728c91a8d86 RTC:180241 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53591 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: Martin Gloff <mgloff@us.ibm.com> Reviewed-by: Roland Veloz <rveloz@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Enable sbeio runtime testcasesBill Hoffa2018-02-153-13/+13
| | | | | | | | | | | | | | Change-Id: Id5d8ef64606cd808cc05e4ec4f0a317773317b67 RTC: 178802 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/54065 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: Corey V. Swenson <cswenson@us.ibm.com> Reviewed-by: Prachi Gupta <pragupta@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Enable fsidd testcases for Cumulus modelBill Hoffa2018-02-154-46/+32
| | | | | | | | | | | | | Change-Id: I3ad1335d734ec04e7cc68833662bf228302b838f RTC:178802 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/54037 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: Prachi Gupta <pragupta@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Increase poll wait time for SBE PSU operationsDan Crowell2018-02-151-2/+2
| | | | | | | | | | Adding an explicit sleep versus relying on a context switch to determine when PSU operations to the SBE have timed out. Change-Id: I411b45f3ce793eb60e7defe37cf4273abd5d066a Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/54048 Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Tested-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Add attribute to give platform more control over PM_RESETChristian Geddes2018-02-144-5/+37
| | | | | | | | | | | | | | | | | | | | | | The PM_RESET hwp calls special wakeup enable on all EX targets, then will clear auto-special wakeup bit on the core if special wakeup is done. In some cases hostboot does not want these steps of the PM_RESET. This attribute gives the platform the ability to decide if they want to enable special wakeup and clear autowakeup on the cores during PM_RESET CQ:SW412666 Change-Id: I8f2e40f4b122f3ff6a048fa6931a1e47f89d3e4f Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53953 Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: HWSV CI <hwsv-ci+hostboot@us.ibm.com> Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com> Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com> Reviewed-by: RANGANATHPRASAD G. BRAHMASAMUDRA <prasadbgr@in.ibm.com> Reviewed-by: Gregory S. Still <stillgs@us.ibm.com> Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53976
* Error when hb tries to access reserved memory past limitIlya Smirnov2018-02-146-19/+268
| | | | | | | | | | | | | | | | | | | | | | An issue has recently occurred when hb reserved memory past the allowed limit (256M-4K) and overwrote some of the PHYP memory, which caused bugs that were difficult to debug. This change is to check the memory address we are trying to reserve and throw an error if the address is at or above the allowed limit. We only execute this check if PHYP is running and only on components not belonging to PHYP (PHYP components may be placed outside of the limit). Change-Id: Ic62a7b724abc3b29b7872d0af47de8c68cde2ea8 RTC:186332 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/52850 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> Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Fixing SBE PSU error messagesElizabeth Liner2018-02-141-8/+59
| | | | | | | | | | | | | | | | When we added support for SBE rebooting, an error was incorrectly removed - the error when there is a correct PSU timeout that is not associated with the SBE. This commit adds that error back in. Change-Id: Ie289f5ff5312d0b1415a98531fd4a8b94cef37ae Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/52844 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: Daniel M. Crowell <dcrowell@us.ibm.com>
* Remove extraneous error messages for multicast scomsDan Crowell2018-02-141-3/+3
| | | | | | | | | | | | | | | | | | | Fixed two bugs in the code that was added to avoid extra error logging in the multicast workaround code: - Fixed bad double-negative check for MC sync mode - Fixed incorrect EC chiplet range Change-Id: I8a6747ee66206e0df519ad9b58cdc0a37a4aa7d6 CQ: SW417057 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53857 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> Reviewed-by: Corey V. Swenson <cswenson@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Prachi Gupta <pragupta@us.ibm.com> Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com>
* Add duplibs.sh tool to our hostboot repoMatt Derksen2018-02-141-0/+45
| | | | | | | | | | | | | This tool helps detect if object files are being included in multiple libraries Change-Id: Ie01a079560ee785802e9245e8804db6587893191 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53592 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Martin Gloff <mgloff@us.ibm.com> Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Allow --with-backtrace for hb-Ps on Simics Command LineBill Hoffa2018-02-141-2/+2
| | | | | | | | | | | | Change-Id: If1426ec93cf14f6fb9d6ce7fc5132492b03987ca Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/54001 Reviewed-by: Michael Baiocchi <mbaiocch@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: Prachi Gupta <pragupta@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Secure Boot: Adapt workbooks' TPM target to Hostboot'sNick Bofferding2018-02-141-0/+8
| | | | | | | | | | | | | | | | Adds a chip-NPCT501 target adapter which adapts various workbooks' TPM target to Hostboot's TPM target Change-Id: I6387e9c5a9c5f3d4520fba88e8a9367bcfc35d70 RTC: 187287 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53902 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: Michael Baiocchi <mbaiocch@us.ibm.com> Reviewed-by: Marshall J. Wilks <mjwilks@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Properly handle INTRP unmask error handlingBill Hoffa2018-02-141-2/+12
| | | | | | | | | | | | | Change-Id: Ic81b3b2bb73e714b0609fa14b5b308db399b88b6 CQ: SW416916 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53945 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: Prachi Gupta <pragupta@us.ibm.com> Reviewed-by: Elizabeth K. Liner <eliner@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Bug fix to TOR API to not check TOR header version against TOR_VERSIONClaus Michael Olsen2018-02-141-1/+0
| | | | | | | | | | | | | | | | | | | | I'm removing this check because it is not a valid check for making a decision that the TOR header is invalid (it's still valid). CQ: SW416424 Change-Id: I2cdb78bc18a1fcc718038be41b2965255ba5d0de Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/54017 Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: HWSV CI <hwsv-ci+hostboot@us.ibm.com> Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com> Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com> Reviewed-by: Sachin Gupta <sgupta2m@in.ibm.com> Reviewed-by: Thi N. Tran <thi@us.ibm.com> Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com> Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/54023 Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com> Tested-by: Christian R. Geddes <crgeddes@us.ibm.com>
* Fix to TOR API failure on 32b systemsClaus Michael Olsen2018-02-141-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | This fixes a bug wrt incorrect use of sizeof() on a pointer that only shows up on 32b system (but is luckily successful on 64b systems). Key_Cronus_Test=XIP_REGRESS Change-Id: I9f33c5728cb68acaeee55580c9f1c1b8743cbd8d CQ: SW412437 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53875 Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: HWSV CI <hwsv-ci+hostboot@us.ibm.com> Reviewed-by: Sachin Gupta <sgupta2m@in.ibm.com> Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com> Tested-by: Cronus HW CI <cronushw-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: Jennifer A. Stofer <stofer@us.ibm.com> Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53905 Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com> Tested-by: Christian R. Geddes <crgeddes@us.ibm.com>
* Re-order INTR/IPC shutdown message handlingBill Hoffa2018-02-142-17/+18
| | | | | | | | | | | | | | | | | - The mbox msg_handler already subscribes to the INITSERVICE shutdown message so perform the IPC shutdown logic during that Change-Id: I2402e6ac17212051029de3474e24b8e91dce9544 CQ: SW417278 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53984 Reviewed-by: Richard J. Knight <rjknight@us.ibm.com> Reviewed-by: Prachi Gupta <pragupta@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: Dean Sanner <dsanner@us.ibm.com>
* Update utilDeallocateTces and other TCE CleanupMike Baiocchi2018-02-136-49/+35
| | | | | | | | | | | | | | | | | | This commit removes the size parameter from utilDeallocateTces() as we should be able to use the token parameter to get back the saved size of the region. This commit also cleans up some other minor TCE issues and even fixes one I2C trace. Change-Id: I9ed5695d5ea13d89a22ace9593b5d9dc31a16736 RTC:168745 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53908 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> Reviewed-by: Marshall J. Wilks <mjwilks@us.ibm.com> Reviewed-by: ILYA SMIRNOV <ismirno@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Callout DIMM if bad dq set in mnfg modeCaleb Palmer2018-02-131-1/+4
| | | | | | | | | | | | | | Change-Id: I0213e5b716e155f8ed36b7c31eb716dca2875a46 CQ: SW416914 Backport: release-fips910 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53789 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Benjamin J. Weisenbeck <bweisenb@us.ibm.com> Reviewed-by: Brian J. Stegmiller <bjs@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>
* Check TYPE_NODE Targets Twice for Serial/Part Number UpdatesIlya Smirnov2018-02-131-1/+77
| | | | | | | | | | | | | | | | | | | | | | There was an issue where the system serial number was not updating properly in hostboot. The problem is that changing just the system serial/part number does not trigger FRU update on our side. The fix is to add an explicit check for SN/PN in the code path that compares SEEPROM and PNOR records/keywords for TYPE_NODE targets only. Change-Id: I3c4d43e2ca9ade0e03c8f37198b67dd7e7efbbde CQ:SW414411 Backport:release-op910 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53754 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> Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com> Reviewed-by: Marshall J. Wilks <mjwilks@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>
* PRD: Move MC_ADDR_TRANS register definition to MCABenjamin Weisenbeck2018-02-133-24/+10
| | | | | | | | | | | | | | | | Change-Id: Ibd54afa60b46ad064344a0aac6050262d64b9336 CQ: SW417058 Backport: release-fips910 Backport: release-op910 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53726 Reviewed-by: Benjamin J. Weisenbeck <bweisenb@us.ibm.com> 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: Zane C. Shelley <zshelle@us.ibm.com> Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53889 Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
* PRD: VPD not getting cleared when all repairs used up during memdiagZane Shelley2018-02-134-69/+81
| | | | | | | | | | | | | | | | Change-Id: I5eb3fd052f0bbd3958379a6020592d8ea272d7e4 CQ: SW416691 Backport: release-fips910 Backport: release-op910 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53797 Reviewed-by: Caleb N. Palmer <cnpalmer@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Benjamin J. Weisenbeck <bweisenb@us.ibm.com> Reviewed-by: Brian J. Stegmiller <bjs@us.ibm.com> Reviewed-by: Zane C. Shelley <zshelle@us.ibm.com> Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53888 Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
* PRD: NPUFIR updates per request from hardware teamZane Shelley2018-02-132-18/+18
| | | | | | | | | | | | Change-Id: I370a6f98e6074df083b76da508833e58a7b4df28 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53553 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Caleb N. Palmer <cnpalmer@us.ibm.com> Reviewed-by: Benjamin J. Weisenbeck <bweisenb@us.ibm.com> Reviewed-by: Zane C. Shelley <zshelle@us.ibm.com> Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53887 Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
* Updates to make multinode IPC workDean Sanner2018-02-1310-44/+42
| | | | | | | | | | | | | | | | | | -Fixed node/group id calculations -Reduced dbell printk to prevent printk overflow -Fixed architectual hole in how internode IPC works Workitems won't work, instead just always check for IPC on any doorbells to master thread -Changed PIR tracing to print out in hex Change-Id: I25eb7f87fd812a90f98a7724b1ac1100f764fe7b Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53187 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Prachi Gupta <pragupta@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: Dean Sanner <dsanner@us.ibm.com>
* Fixing processor ordinal id for multinode systemsVenkatesh Sainath2018-02-131-5/+47
| | | | | | | | | | | Change-Id: Id1553c64c76b64d07647d863682880aabc638bb1 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53555 Reviewed-by: Prachi Gupta <pragupta@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> Reviewed-by: NAGENDRA K. GURRAM <nagendra.g@in.ibm.com> Reviewed-by: Dean Sanner <dsanner@us.ibm.com>
* PRD: Additional target support for Predictive Dyn Mem DeconfigZane Shelley2018-02-122-12/+23
| | | | | | | | | | | | | | | Change-Id: I860502c950d06b876974feb0fbfeb3cd56bf6e6e CQ: SW416352 Backport: release-fips910 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53607 Reviewed-by: Caleb N. Palmer <cnpalmer@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Brian J. Stegmiller <bjs@us.ibm.com> Reviewed-by: Zane C. Shelley <zshelle@us.ibm.com> Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53651 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>
* HTMGT: Bias values need to be applied to frequenciesChris Cain2018-02-124-34/+116
| | | | | | | | | | | | | Change-Id: Ia36af860f907a0cc8916a549bcd2d6160f46faee CQ: SW414845 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53236 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Martha Broyles <mbroyles@us.ibm.com> Reviewed-by: Sheldon R. Bailey <baileysh@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 and codify how system and node targets are handled by attribute overridesBrian Bakke2018-02-122-477/+647
| | | | | | | | | | | | | | | | | | This change documents the syntax for attribute override text files and fixes implementation misses. The API is documented in attrTextToBinaryBlog.H . Change-Id: Iab34365eadd908e7293942152c5140733d39e9e8 RTC: 187338 CQ: SW416716 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53746 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: Christian R. Geddes <crgeddes@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Bad dq bitmap vpd write fixesCaleb Palmer2018-02-121-3/+25
| | | | | | | | | | | | | | Change-Id: I4fa184b35b5d28feb837c3fa5b72b667c184877f CQ: SW414262 Backport: release-fips910 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53676 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: Brian J. Stegmiller <bjs@us.ibm.com> Reviewed-by: Benjamin J. Weisenbeck <bweisenb@us.ibm.com> Reviewed-by: Zane C. Shelley <zshelle@us.ibm.com>
* Populate the Severity of PLAT ErrorsIlya Smirnov2018-02-122-0/+222
| | | | | | | | | | | | | | | | | | | | An issue was observed where recovered PLAT error logs were not being properly hidden/skipped and incorrect severity was assigned to those error logs. The reason is that the path of code that PLAT errors take fails to overwrite the default severity (unrecoverable). A fix is to explicitly set the severity of all errors at the end of rcToErrl. Change-Id: Ib0aff72a800e0ed7a5b48e753049d2f3422406b4 CQ:SW410923 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53221 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: Prachi Gupta <pragupta@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Switch polarity of return data for perst commandsDan Crowell2018-02-121-4/+4
| | | | | | | | | | | | | | | HWSV and Hostboot had backwards interpretations of fail/success. Changing Hostboot to return zero for success. Change-Id: Ia21b2b7c78eb9c64f06a690a50740357441b23c1 CQ: SW415796 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53653 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: Brian E. Bakke <bbakke@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 attributes based on message from the FSPMatt Derksen2018-02-114-36/+169
| | | | | | | | | | | | | | | | | | At least one usecase exists where code on the FSP needs to update an attribute and have that value reflected inside of HBRT. Change-Id: Iad5057ecbb182f6f58879ba694d4b8adea485248 RTC:186991 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53486 Reviewed-by: Martin Gloff <mgloff@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: Thi N. Tran <thi@us.ibm.com> Reviewed-by: Christian R. Geddes <crgeddes@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>
* Additional risk level support - (step 1) Backward compatibilityClaus Michael Olsen2018-02-119-67/+306
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The purpose of this commit is to avoid a coreq situation by ensuring this commit is fully propagated through our repos and test drivers before introducing the change to the new HW image with two RLs. The commit enables simultaneous support for producing a HW image and retrieving rings from an image that has either one or two risk level (RL) rings in the .rings section. The commit however does NOT actually, yet, make any changes to the image which is the aim of the (step 2) commit 53292. Nor does this commit generate any raw ring files or process any RL2 level rings yet. Again this will happen in 53292. The commit also includes, - various related cleanups in data naming and ring file processing, - some data and invironment specific parts in ring_apply.C have been moved to common_ringId.C. Key_Cronus_Test=XIP_REGRESS HW-Image-Prereq=53292 - This commit (52659) must be fully merged before merging 53292. Change-Id: I402d53c4a3ca6a084c958321069cc6f60e04ad24 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53019 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>
* Set ATTR_RISK_LEVEL based on scratch reg bitsDan Crowell2018-02-112-6/+26
| | | | | | | | | | | | | | | | The risk level value has expanded to 2 extra bits to handle new modified settings. Change-Id: Ic3794066659fe5ebe82b330c9530a04f7eadee94 RTC: 187088 Backport: fips910 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53023 Reviewed-by: Martin Gloff <mgloff@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>
* Add callout for unresponsive TPMsJaymes Wilks2018-02-101-4/+40
| | | | | | | | | | | | | | | | | | | Adds a callout for TPMs that are unresponsive to i2c requests and lowers the processor callout priority to avoid recommending a processor replacement when it was the TPM that was unresponsive. Change-Id: Ia213183f2e93e3bd7e56e93c831ef52c28af31c4 CQ:SW415447 Backport:release-fips910 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53387 Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com> Reviewed-by: Nicholas E. Bofferding <bofferdn@us.ibm.com> Reviewed-by: ILYA SMIRNOV <ismirno@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>
OpenPOWER on IntegriCloud