summaryrefslogtreecommitdiffstats
path: root/src/usr
Commit message (Collapse)AuthorAgeFilesLines
...
* 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>
* 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 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>
* 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-155-89/+66
| | | | | | | | | | | | | | | | | | | 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>
* 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-153-45/+31
| | | | | | | | | | | | | 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-143-4/+20
| | | | | | | | | | | | | | | | | | | | | | 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-144-9/+235
| | | | | | | | | | | | | | | | | | | | | | 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>
* 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>
* 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-135-43/+34
| | | | | | | | | | | | | | | | | | 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-131-3/+9
| | | | | | | | | | | | | | | | | | -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-111-36/+110
| | | | | | | | | | | | | | | | | | 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>
* Set ATTR_RISK_LEVEL based on scratch reg bitsDan Crowell2018-02-111-2/+21
| | | | | | | | | | | | | | | | 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>
* Fix developer and customer descriptions for TPM required policyJaymes Wilks2018-02-101-1/+20
| | | | | | | | | | | | | | | | | | Adds more useful information to the developer and customer descriptions of error logs that are created when the TPM required policy is broken. Change-Id: Ib2c42c85cb5689b5a791ab96709dc40f4bbd85d4 CQ:SW415447 Backport:release-fips910 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/52848 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> 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> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Revert "Remove Attribute Check For TCE Settings"Nicholas E. Bofferding2018-02-101-6/+16
| | | | | | | | | | | This reverts commit 4f3656309da631e6a5fcf4d6cb8fe5241c61e1de. Change-Id: I4e9de83751d970af394e4a964c096c452a3efdcc Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53808 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: Nicholas E. Bofferding <bofferdn@us.ibm.com>
* Default to full cache usage during early HostbootDean Sanner2018-02-092-2/+16
| | | | | | | | | | | | | | Early hardware had to support a partial cache config which is inefficient. Placing this code behind a config option, defaulting to full cache usage Change-Id: I521a1d9644d754a2657993603e3e2b2aab90112a Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53606 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Reviewed-by: Prachi Gupta <pragupta@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Dean Sanner <dsanner@us.ibm.com>
* Remove Attribute Check For TCE SettingsMike Baiocchi2018-02-091-16/+6
| | | | | | | | | | | | | | | | | | | | This commit removes hostboot checking ATTR_USE_TCES_FOR_DMAS when determining whether to enable TCEs or not. The default now it to always use TCEs on FSP-based systems regardless of this attribute's setting. Change-Id: I08bcb3b7c2643b857b0b74732b1334403139c780 RTC:168745 Backport:release-fips910 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53611 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Marshall J. Wilks <mjwilks@us.ibm.com> Reviewed-by: ILYA SMIRNOV <ismirno@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: Daniel M. Crowell <dcrowell@us.ibm.com>
* Only access HB_VOLATILE on non-FSP systemsMatt Derksen2018-02-082-47/+56
| | | | | | | | | | | | | | | | | | This section of memory was added for semi-persistance of variables in a non-FSP environment. Since this section only pertains to non-FSP systems, it therefore should not be accessed by FSP systems. Change-Id: I0dc3181781f8ba55720facaeb2b1e5d67c2817c6 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53293 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> Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Find location of HDAT relocation dynamically from PAYLOADMike Baiocchi2018-02-082-11/+56
| | | | | | | | | | | | | | | | | | | This commit looks at the NACA structure in the Primary PAYLOAD lid to determine the final destination of the HDAT structure that the FSP loaded into a temporary memory space using TCEs. Change-Id: I82b94f5fc133e4222d4efdcc7444012d7bcb5166 RTC:168745 Backport:release-fips910 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53564 Reviewed-by: ILYA SMIRNOV <ismirno@us.ibm.com> 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> Reviewed-by: Nicholas E. Bofferding <bofferdn@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 Istep 10.3 From IPL FlowIlya Smirnov2018-02-083-879/+732
| | | | | | | | | | | | | | | | | | | | | | | Istep 10.3 (host_set_voltages) has been moved to Istep 8.12 and needs to be no-oped. Executing host_set_voltages twice causes crashes due to secureboot thinking we are trying to skip an istep. Note that validateSecuritySettings that was performed in 10.3 has now been moved to 10.4 (with the required sub-routines). Change-Id: I81284157dedebb3f4ee357ce28b29b1dd6a3fe8a CQ:SW416209 CMVC-Coreq:1044932 Backport: release-fips910 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53443 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: Michael Baiocchi <mbaiocch@us.ibm.com> Reviewed-by: Marshall J. Wilks <mjwilks@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Secure Boot: Remove utilmem from verify container fail pathNick Bofferding2018-02-071-20/+26
| | | | | | | | | | | | | | | | | | | When the Hostboot extended image verification fails, it calls FFDC collection routine which invokes UtilMem functionality that is contained in the extended image. This creates a circular loop of dependency that cannot be satisfied. The FFDC collection was fixed to remove use of UtilMem while providing same function Change-Id: Id7a15ae68ec316c5d6d6779143d1409f5333e816 Backport: release-fips910 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53456 Reviewed-by: ILYA SMIRNOV <ismirno@us.ibm.com> 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: Daniel M. Crowell <dcrowell@us.ibm.com>
* Set PAYLOAD_KIND attribute based on fused bits in scom registerChristian Geddes2018-02-071-1/+32
| | | | | | | | | | | | | | | | | | | | | | | | | The scom register 0xF000F, bit 57 tells us whether or not the fuse has been blown on this processor. If it is set, that means that it is up to hostboot to set the PAYLOAD_KIND attribute. We decide this based on if we are in SMT4 or SMT8 mode, which is known during istep 6. If we are in SMT4 and the fuse is blown, we can assume PAYLOAD_KIND is OPAL, if we are in SMT8 and the fuse is blown, we are booting PHYP. In a lab scenario on which a system has blown fuses, and we want to force OPAL boot on a system we are forcing to be SMT8, we would have to apply an attribute override to make ATTR_PAYLOAD_KIND = 0x2 . 0x2 is the enum for SAPHHIRE as the payload kind. Change-Id: Ia15dee4dd67130691440bf7fe76915acfb81a19d RTC: 186965 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/52957 Reviewed-by: Martin Gloff <mgloff@us.ibm.com> Reviewed-by: ILYA SMIRNOV <ismirno@us.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>
* LINK_TRAIN: HWSV needs to write this attr, so, marking it as writablePrachi Gupta2018-02-071-0/+4
| | | | | | | | | | | | Change-Id: If15d73de8ba92a822cfe3fb83dc48ad0a8f47ff9 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53378 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: Bhimsen G. Kulkarni <bhimseng.kulkarni@in.ibm.com> Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Initialize SBE message communication on HBRT startMatt Derksen2018-02-073-80/+200
| | | | | | | | | | | | | | | | | | | Need to clear first 2 bits in CFAM register 0x283B. This basically resets our SBE communication stream just incase HBRT restarts after processing SBE interrupt. Clearing these bits tells the sender to resend the command. Change-Id: I1a1c5b80cc99414649d38c06b93428c87299f242 RTC:186334 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53225 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> Reviewed-by: Brian E. Bakke <bbakke@us.ibm.com> Reviewed-by: Roland Veloz <rveloz@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Always set RISK_LEVEL regardless of valueDan Crowell2018-02-072-4/+4
| | | | | | | | | | | | | | | | | | | | Fixing a bug where we only set ATTR_RISK_LEVEL if the value is non-zero. This leads to cases where we cannot clear the value once we set it. I also changed the persistency of ATTR_RISK_LEVEL so that it doesn't persist across boots since it will always be set to a new value during each boot. Change-Id: I9df1e505755f860524a3b1af46c68f315f142066 CQ: SW415934 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53451 Reviewed-by: Dean Sanner <dsanner@us.ibm.com> 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>
* PRD: clear old symbol marks in TPS analysisZane Shelley2018-02-071-1/+26
| | | | | | | | | | | | | | | | Change-Id: I22b80bcdbd6a3e57303982351268a4a3c327c8b3 CQ: SW415366 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53351 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/53491 Reviewed-by: Zane C. Shelley <zshelle@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>
OpenPOWER on IntegriCloud