summaryrefslogtreecommitdiffstats
path: root/src/usr/sbeio
Commit message (Collapse)AuthorAgeFilesLines
...
* Moving SBE error and threshold handler to sbeioElizabeth Liner2017-10-102-1/+1104
| | | | | | | | | | | | Change-Id: I15438b21692f45899efb75afcc1db31f9caece15 RTC:178801 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/46843 Reviewed-by: Martin Gloff <mgloff@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Prachi Gupta <pragupta@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Update set_sbe_error macro exceeds line lengthRichard J. Knight2017-10-031-0/+1
| | | | | | | | | | | | | | | | | | | | | -Undate macro to call a function for each RC value instead of inlining of all FFDC collection -Update parseErrorInfo.pl script to generate a new file which contains functions for use in new macro layout Change-Id: I40a02fef07b3618b137a8e0c2bfa10c9756da600 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/46719 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: ARAVIND T. NAIR <aravindnair@in.ibm.com> Reviewed-by: Sachin Gupta <sgupta2m@in.ibm.com> Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com> Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/46722 Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Empty interface for SBE vital attention handlingDan Crowell2017-09-283-0/+62
| | | | | | | | | | | | | | | | | | Creating a NOOP function for PRD to code against for SBE error handling. Change-Id: Ic16bb8ed2dca7f8a77a74d4fb7a60b2859621d39 RTC: 180239 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/46711 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: Martin Gloff <mgloff@us.ibm.com> Reviewed-by: Elizabeth K. Liner <eliner@us.ibm.com> Tested-by: Daniel M. Crowell <dcrowell@us.ibm.com> Reviewed-by: Brian J. Stegmiller <bjs@us.ibm.com> Reviewed-by: Zane C. Shelley <zshelle@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Set variables to nullptr after they are deletedChristian Geddes2017-09-251-0/+3
| | | | | | | | | | | | | | | | | | While updating some LPC logic a developer noticed a bug where an errlog handle wasnt getting set to nullptr after it was deleted this caused a confusing bug that took awhile to track. I noticed that this bug was all over our code so this commit fixes a lot of cases where variables get deleted but not set to nullptr Change-Id: I103b5c71b93686c3c89b04d1d565a24d4de74e6f Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/45890 Tested-by: Jenkins Server <pfd-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> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Reviewed-by: Matt Derksen <mderkse1@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Move targ info getter func for sbechipOp into common codeChristian Geddes2017-09-212-0/+125
| | | | | | | | | | | | | | | | Before we only needed to get the target info for the scan chipop, now more sbe chip ops require this functionality so we are moving it to common code Change-Id: Ifbb680db27c5975ee216d5d21ff78192c9ea2d34 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/46345 Reviewed-by: Martin Gloff <mgloff@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: 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>
* cumulus: changes to boot simics all the way to test casesPrachi Gupta2017-08-292-3/+5
| | | | | | | | Change-Id: Ibc62f59635db28e50197139276b9b4c6b59ccc3e Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/44768 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Implement getSbeFfdc FIFO requestMarty Gloff2017-08-278-17/+542
| | | | | | | | | | | | | | | | Add support for sending getSbeFfdc FIFO requests to SBE and add a test case for the getSbeFfdc request. Change-Id: I19dbbd92cb2d25679218528d199cfbd09fd9f995 RTC: 178022 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/44288 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: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Implemented SBE Passthrough for Attribute Override.Donald Washburn2017-08-247-5/+1778
| | | | | | | | | | | | | | | | | | | | | | | | | *Wrote a runtime SBE pass-through function to invoke the runtime apply_attribute_overrides function. The caller is required to place the request data in the SBE COMMS reserved memory area and invoke the sbe_message_passing function. The request data consists of an sbeMessage_t header followed by binary override data which can be obtained from the attributeOverride tool. Upon completion of the sbe_message_passing function, the respose data will be written to the SBE COMMS AREA. *Added support for accessing system attributes at runtime. RTC: 152409 Change-Id: Ieed3e026350cc272e6268e7802971551d3f39ed4 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/44311 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: Roland Veloz <rveloz@us.ibm.com> Reviewed-by: Martin Gloff <mgloff@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Use keyAddr stash chipop to pass rsv mem addr through SBE for MPIPLcrgeddes2017-08-232-2/+26
| | | | | | | | | | | | | | | | | | Previously we were relying on the reservedMem section to be a certain address for the attrrp to find after an MPIPL. With this change the address is passed through the SBE so the reservedMem data can be anywhere RTC: 165369 Change-Id: I06521c049088c4a53a8c0a51fa07e5200da09483 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/44624 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: Martin Gloff <mgloff@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 SBEIO PSU calls to Open/Close Unsecure Memory RegionsMike Baiocchi2017-08-044-71/+313
| | | | | | | | | | | | | | | | | | | | This commit enables the SBEIO PSU calls to the SBE to open and close Unsecure Memory Regions now that the SBE team supports that API. Their support, however, does not actually whitelist/blacklist the memory at this time. This commit also gracefully handles the errors from the SBE saying this support is not there. Change-Id: Ia9367b17e403bf759d778de5f4bc7fca3544397b RTC:174970 CMVC-Prereq: 1030098 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/44018 Reviewed-by: Marshall J. Wilks <mjwilks@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Stephen M. Cprek <smcprek@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>
* Added parser for scom FFDC data and a SBE messaging buffer class.Donald Washburn2017-08-0412-112/+3684
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The FFDC data from an SBE Scom read/write needed to be parsed and used to invoke PIB::addFruCallouts during both FIFO and PSU operations. During task research it was discovered that the error code path for reading FIFO request responses had several errors. These errors related to manipulating a duel buffer system. Fixed these errors and encapsulated the duel buffer handling in order to facilitate unit testing. * Created FFDC parsing classes FfdcParsedPackage and FfdcScomPibErrorPackage in order to validate and process FFDC data based upon the return code contained within the FFDC. * Created the SbeFifoRespBuffer class for encapsulating the handling for the duel buffers used in the fifo readResponse method. * Modified the SbeFifo::readResponse method to use the SbeFifoRespBuffer class for messaging and the new FFDC parsing classes for handling PIB addFruCallouts. * Modified the SbePsu::readResponse method to use the FFDC parser classes to invoke PIB::addFruCallouts. * Added Unit Tests for SbeFifoRespBuffer, FfdcParsedPackage and FfdcScomPibErrorPackage. Change-Id: I195fa036dfa6a0d66d9a3dc15aeb8b0f01cf798c RTC: 175891 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/43212 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: Martin Gloff <mgloff@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Dzuy Nguyen <dzuy@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Fix race condition between INTR and SBEIODan Crowell2017-07-312-30/+101
| | | | | | | | | | | | | | | | | | | | | Fixed a race condition in clearing out the PSU interrupt register that existed between the INTR and SBEIO code. We can sometimes lose interrupts for SBE PSU operations which leads to a timeout. Also added code to look for SBE errors if a PSU operation times out Change-Id: I8cdcdcc08956b038bcc65ad7e00a34719bf14c61 CQ: SW396057 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/43339 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Elizabeth K. Liner <eliner@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: William G. Hoffa <wghoffa@us.ibm.com>
* Secureboot: Add IPL calls to Open/Close Memory Regions via SBEMike Baiocchi2017-07-192-48/+86
| | | | | | | | | | | | | | | | | | | This commit adds the open/close SBE memory region calls throughout the IPL, as needed. The underlying SBE calls are still commented out, however, as we await SBE support. Change-Id: I87685dca95e5cb7d10997eb6eb414a1927862d64 RTC:145686 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/42873 Reviewed-by: Nicholas E. Bofferding <bofferdn@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Stephen M. Cprek <smcprek@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: Marshall J. Wilks <mjwilks@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Secureboot: Base Support of Opening/Closing Memory Regions via SBEMike Baiocchi2017-07-194-0/+943
| | | | | | | | | | | | | | | | | | | | | This commit provides the base support of Opening and Closing Unsecure Memory Regions via PSU commands to the SBE. It introduces external interfaces, a Memory Region Manager class, and a testcase that covers the expected use of the interfaces. However, it does not enable the actual PSU commands to the SBE since their support is not yet available. Change-Id: I086011fcbd485abf2db574a82d43cfdea120ffc8 RTC:145686 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/42834 Reviewed-by: Nicholas E. Bofferding <bofferdn@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Stephen M. Cprek <smcprek@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: Marshall J. Wilks <mjwilks@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Add in PSU infrastructure for stashKeyAddr sbe chip opcrgeddes2017-07-173-1/+87
| | | | | | | | | | | | | | | | | | For mpipl we need to pass an address through the SBE. To do this the SBE is writing functionality that takes in uint8_t key and a uint64_t value and from this chipOp and stores it in the bootloader image (before 12k exception vector but after magic keyword). I also cleaned up some prior naming inconsitencies in this commit RTC: 173362 Change-Id: Id6dc47ed0e34f50f74fc894007dd144b2f7bfe81 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/41844 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: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Correctly handle sbe ranges for multiple chipsDan Crowell2017-07-131-1/+9
| | | | | | | | | | | | | | | | | Fixing a bug where we did not use unique instance ids for each SBE's comm and ffdc area. Also reconciled a couple related mismatches and added some traces for easier debug. CQ: SW395465 Change-Id: Ib2a50644c114e5b08222b9fb19e92923466fa24c Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/42994 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Martin Gloff <mgloff@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Reviewed-by: Matt Derksen <mderkse1@us.ibm.com> Tested-by: 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 some more FFDC for SBEDan Crowell2017-07-121-5/+9
| | | | | | | | | | | | | Add the response and the target processor to the SRC for PSU timeouts. Change-Id: I4c739a66ab5bee8e35a514f6851e4c55e09d18e8 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/42369 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>
* handle pibrc correctly from sbe fifoPrachi Gupta2017-06-302-1/+63
| | | | | | | | | | | | | Change-Id: I3f0ba875a47fdd00758c65b7982f300f0e05cd14 RTC: 170433 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/41847 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: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Sachin Gupta <sgupta2m@in.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Matthew A. Ploetz <maploetz@us.ibm.com>
* Finalize SBE PSU FFDC collection - Handle setFFDCAddress failedDzuy Nguyen2017-06-231-12/+27
| | | | | | | | | | | Change-Id: I9d9ac10df9eb44f63203abca14db3bcf8a7c9d16 RTC: 164405 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/41608 Reviewed-by: Martin Gloff <mgloff@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Shakeeb A. Pasha B K <shakeebbk@in.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Finalize SBE PSU FFDC collection - Set FFDC address in PSUDzuy Nguyen2017-06-061-82/+123
| | | | | | | | | | | | Change-Id: Ia2b1012cf2938f34cc6acdbb888b5a489aac98f6 RTC: 164405 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/40453 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: Martin Gloff <mgloff@us.ibm.com> Reviewed-by: Shakeeb A. Pasha B K <shakeebbk@in.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Full Path of SBE message passing Add HTMGT default path testingMarty Gloff2017-05-181-3/+94
| | | | | | | | | | | | | | | Add a test that calls the default HTMGT processing path when HTMGT is configured. Change-Id: I6cde3e9fbf35370118e715c19fb4f63c2ee21edd RTC:173806 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/40099 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>
* Fix rt_sbeio.C include and add TS_FAIL infoMarty Gloff2017-05-102-2/+1
| | | | | | | | | | | | | | Commit 39894 causes compile fails because of a deleted header file. Fix includes in rt_sbeio.C and also make a recommended test case change from review of the same commit. Change-Id: Iac3d90ee80f483c695da42122013aef3c1c72017 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/40351 Reviewed-by: Daniel M. Crowell <dcrowell@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: Christian R. Geddes <crgeddes@us.ibm.com>
* Include HUID of failed Proc in TI data for certain SBE errorsBill Hoffa2017-05-102-2/+4
| | | | | | | | | | | | | | | - Include a generic error pipleline for other future error scenarios to leverage this functionality Change-Id: Icc1399ee93157c7106d394944a3355285a8cd830 RTC: 171865 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/39072 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: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Prachi Gupta <pragupta@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Full Path of SBE message passing Design updatesMarty Gloff2017-05-102-57/+291
| | | | | | | | | | | | | | | | | | | | Several design changes have been made and need to be reflected in the HBRT code. Increase size of SBE Communication buffer Changes to definition of Command Class / Command Add a wrapper for existing HTMGT function Set Generic HTMGT Message to be processed via wrapper Add erase of process command function entry from map Also address minor review comments from prior reviews. Change-Id: Ibe450507233a39dc976042296ce5ea0314a4af78 RTC:173486 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/39894 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>
* SBE FIFO FFDC parser pluginDzuy Nguyen2017-05-076-25/+587
| | | | | | | | | | | RTC: 168620 Change-Id: If1e099411f5b5b12be1a3832ad818c2019f4dadb Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/39432 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: Martin Gloff <mgloff@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* SBE message passing interface - TestingMarty Gloff2017-04-304-3/+1348
| | | | | | | | | | | | | | Create testcase to test SBE message passing. Change-Id: I596115d96d989dbee41070c4b9e419d2043494ae RTC: 170764 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/38170 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: Daniel M. Crowell <dcrowell@us.ibm.com>
* SBE message passing interface - assert CFAM registerMarty Gloff2017-04-262-74/+150
| | | | | | | | | | | | | | | | Turn on bits in CFAM 0x283B PERV_SCRATCH_REGISTER_4_FSI when SBE Message processing is in progress and when it is complete. Change-Id: I33fd30793e5d8724dc6addc3a7066bc9a8be7326 RTC: 170763 Depends-on: I220a864a072bcc60cc873cde243a28cc899e2722 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/38996 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: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com>
* sbe_psudd.C: don't error out if sbe command is not supportedPrachi Gupta2017-04-241-1/+17
| | | | | | | | | Change-Id: I35062b18794622d7adc0114340593bf2ad7aae70 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/39605 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: William G. Hoffa <wghoffa@us.ibm.com>
* SBE message passing - Set FFDC Address command and set HDAT RHB entryMarty Gloff2017-04-182-0/+106
| | | | | | | | | | | | | | | | | Create space for SBE Communications buffer in HB reserved memory and send Set FFDC Address PSU command to SBE. Change-Id: Ibccdf087d0416edaf6e1e2c5993a10adcddf67fd RTC: 170758 Depends-on: Id14471b1e6f036c278fd5ae1950b942290282c1e Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/38251 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: Daniel M. Crowell <dcrowell@us.ibm.com> Reviewed-by: Corey V. Swenson <cswenson@us.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com>
* SBE message passing interface - process SBE message write responseMarty Gloff2017-04-141-2/+91
| | | | | | | | | | | | | | | | Add prossessing to check the response that was written to the SBE Communication buffer for the processed Pass-through command. Change-Id: I2a898af8209159a28bcde172d6d85c99feaf82ad RTC: 170762 Depends-on: I220a864a072bcc60cc873cde243a28cc899e2722 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/37871 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: Corey V. Swenson <cswenson@us.ibm.com> Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* SBE message passing interface - process SBE message read commandMarty Gloff2017-04-101-1/+223
| | | | | | | | | | | | | | | Add prossessing to read the Pass-through command from the SBE Communication buffer. Change-Id: I14723347fbd362566d16a04438e1efd661969e8e RTC: 170760 Depends-on: I220a864a072bcc60cc873cde243a28cc899e2722 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/37854 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: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Send system configuration to all Procs, not just mastercrgeddes2017-04-101-7/+7
| | | | | | | | | | | | | | | | Previously there was no way to send chipOps to specific Procs, Now that it is possible we want to send the system configuration to all SBEs, not just the master. This way the SBE doesnt need to worry about propagating the system config to slave SBEs Change-Id: I471c60a294fb2e4dd43772322fb5c6c80ccdda1f Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/38544 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Martin Gloff <mgloff@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: Daniel M. Crowell <dcrowell@us.ibm.com>
* SBE message passing interface - call appropriate command processorMarty Gloff2017-04-101-49/+250
| | | | | | | | | | | | | | | Add definitions of SBE message and pass-through command. Create mechanism to call functions to process commands. Change-Id: Id14471b1e6f036c278fd5ae1950b942290282c1e RTC: 170761 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/38167 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: Daniel M. Crowell <dcrowell@us.ibm.com>
* Add Support for sbe_continueMPIPL FIFO Chip opcrgeddes2017-04-043-0/+158
| | | | | | | | | | | | | | Hostboot needs an interface to request for the continueMPIPL procedure on each slave proc. Change-Id: I6d43985aa4c2657c60ed30aac7bd2e9b98e4bd8d Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/38540 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: Dzuy Nguyen <dzuy@us.ibm.com> Reviewed-by: Martin Gloff <mgloff@us.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com>
* SBE message passing interface - call of HBRT process SBE messageMarty Gloff2017-03-213-2/+190
| | | | | | | | | | | | | | Add the new Host interface for SBE message passing. Change-Id: I220a864a072bcc60cc873cde243a28cc899e2722 RTC:170759 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/37694 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: Christian R. Geddes <crgeddes@us.ibm.com> Reviewed-by: Corey V. Swenson <cswenson@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Null errl handle used in upstream timeout path for sbe fifoRichard J. Knight2017-03-171-0/+5
| | | | | | | | | | Change-Id: If2aa6c64a8d106c2746e7e726df03d60537ec495 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/37598 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: Daniel M. Crowell <dcrowell@us.ibm.com>
* Update SBE FIFO to the latest SBE HB spec (1.3a)Dzuy Nguyen2017-03-172-48/+7
| | | | | | | | | | | | | | Add sbe_sp_intf.H from SBE repository to include SBE variables from SBE Change-Id: I49aa987ab4b81b94939791571f005aa29338b36e RTC: 169436 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/36560 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: William G. Hoffa <wghoffa@us.ibm.com> Reviewed-by: Prachi Gupta <pragupta@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Data Storage exception during proc_check_slave_sbe_seeprom_completeRichard J. Knight2017-02-281-0/+8
| | | | | | | | | | | | | -Fixed null errl handle usage in timeout path Change-Id: I6e8233ab82b3054aa3090e519003d3d78ba3642a CQ:SW377830 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/37026 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: Dzuy Nguyen <dzuy@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Quiesce SBE before writing SEEPROMMarty Gloff2017-02-104-27/+46
| | | | | | | | | | | | | | Call sendPsuQuiesceSbe before doing the deviceWrite of the SBE SEEPROM with the SBE image. Change-Id: I6d3cec5b0430b8083acabc30bb7ff14ba5e1b56e RTC:158899 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/34994 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: Michael Baiocchi <mbaiocch@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Ensure sbefifo is clean on slave chips before startingDean Sanner2017-02-073-3/+60
| | | | | | | | | | Change-Id: I440d6f5bd6aa1c3c72b3d8df581262b01765214c Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/35447 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> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com>
* Debug improvements for interrupt and sbe psu failsDan Crowell2017-02-071-9/+12
| | | | | | | | | | | Change-Id: Id71da5d29d24d1386be684f4e7594c9e38e0aa84 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/35172 Reviewed-by: Martin Gloff <mgloff@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> Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Increase SBE timeout to 90 secondsDan Crowell2017-02-011-11/+8
| | | | | | | | | | | | | | There is at least 1 scenario where the FSP and HB are both accessing the SBE at the same time. The FSP operation can take a very long time so HB is increasing our timeout so that we don't fail if we happen to hit that window. Change-Id: I8908608a1db4ac8c27ded59eced8f0ad71f70314 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/35612 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: William G. Hoffa <wghoffa@us.ibm.com>
* SBE FIFO device driver FFDC and error recoveryDzuy Nguyen2017-01-197-617/+1019
| | | | | | | | | | | | | | Add SBE FFDC error handling to FIFO driver Change-Id: I6ae81062eac1d5362c5fa5651c8ca5a2043b5c73 RTC: 149454 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/32969 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: Shakeeb A. Pasha B K <shakeebbk@in.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Add intrastructure for sending psuQuiesce cmd to SBEcrgeddes2016-12-203-2/+82
| | | | | | | | | | | | | | | | The SBE provides an interfaces for the HOST to issue a psu cmd that will quiesce the SBE. This commit adds the ability for HB code to call one function and notify the SBE to quiesce Change-Id: Icb153875b797f107891b05fd26dccbc413fd8f93 RTC:158899 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/33610 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: Martin Gloff <mgloff@us.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Ignore CHIP_OFFLINE pib error when scom is multicastcrgeddes2016-11-141-5/+2
| | | | | | | | | | | | | | When a scom returns an error that says the chip was offline, if it was a multicasted scom we expect this to happen so we can ignore the error and continue RTC:163898 Change-Id: Id9753eab70f78573f0c8d27a74aef62e5715e85c Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/32455 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Zane C. Shelley <zshelle@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Add sbe_psu FFDC error handling.Dzuy Nguyen2016-11-146-50/+487
| | | | | | | | | | | | | | Turn sbe_psudd into singleton. Add handle FFDCError function Change-Id: If84bb2bafcca685c8d31c664f7005de056e96c4c RTC: 144313 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/31468 Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Prachi Gupta <pragupta@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 infrastructure for sending setSystemConfig cmd to SBEcrgeddes2016-10-192-0/+88
| | | | | | | | | | | | | | | | | The SBE needs to know about other PROCs in the sytem. To do this we are sending a 64 bit string that encodes the system configuration down to the SBE at the end of the IPL. This commit just sets up some of the infrastructure that will be needed to send the message. Change-Id: I1a4dfb4e54e043d94697fa809acf1eee8ba9f726 RTC: 160666 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/31069 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com> Reviewed-by: Matthew A. Ploetz <maploetz@us.ibm.com>
* Log SBE Traces on error (simics only)Dan Crowell2016-09-282-0/+32
| | | | | | | | | | | | | | | A quick change to add a new magic instruction that collects the SBE traces any time we hit an error from the SBE. If multiple errors occur they will all be appended to the same file. Data will be saved to $sb/simics/sbetrace.hb.txt. Change-Id: I27575c1565c0089e847e19c3e51cb2926833e387 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/30206 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Martin Gloff <mgloff@us.ibm.com> Reviewed-by: Sachin Gupta <sgupta2m@in.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Added support for 'Put Ring from Image' command on HB for SBEVitaly Marin2016-09-183-422/+20
| | | | | | | | | | | | | Change-Id: I4930633f87bdb61acb19e01f17a8006277f7868d RTC:132654 CMVC-Prereq: 1004971 CMVC-Prereq: 1005024 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/28887 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Vitaly Marin <mvitaly@us.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Temporarily force terminal RC on SBE errorsDan Crowell2016-09-082-0/+62
| | | | | | | | | | | | | | | Until Hostboot has the infrastructure in place to collect all of the FFDC from a SBE failure, we will rely on the FSP code to collect the data. This requires Hostboot to terminate with a special RC to force this behavior. Change-Id: I94fba157826664b680a51790104932f97b671950 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/29253 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Martin Gloff <mgloff@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Deepak Kodihalli <dkodihal@in.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com>
OpenPOWER on IntegriCloud