summaryrefslogtreecommitdiffstats
path: root/src/usr/errl/errlentry.C
Commit message (Collapse)AuthorAgeFilesLines
* Add Axone target types to errl/hwas functionsDan Crowell2019-06-201-79/+118
| | | | | | | | | | | | | | | | | | | Filled in a couple spots where we needed to support the new Axone targets types: OCMB, PMIC. Cleaned up the way we handle adding PN/SN/ECID to our logs to make them more generic to handle new types in the future automatically. Change-Id: I12173f762f92923347f7215c4e590436fcfb9ed7 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/79061 Reviewed-by: Christian R. Geddes <crgeddes@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: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Remove procedure callout EPUB_PRC_NVDIMM_ERCorey Swenson2019-05-131-19/+0
| | | | | | | | | | | | | | This PRC is no longer needed, removed the procedure callout from errlEntry.C. Leaving the enum in place as a PRD function is still using it. Change-Id: I80c0b1820e10823b61d909acf427378693e2a74c Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/77058 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>
* HB Improvements: Compiler Issues with Different Config FilesLuis Fernandez2019-04-051-1/+7
| | | | | | | | | | | | | | | Fixed compiler issues with Witherspoon, Romulus, Zaius, Boston. Change-Id: Ic4046323eb391be6ec311bc408ef9d858ceff8cd RTC: 202716 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/75552 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>
* Add checks for invalid trace data in removeDuplicateTracesGlenn Miles2019-03-261-11/+53
| | | | | | | | | | | | | | | | | | | | | This change addresses defect SW457275 where hbrt was crashing while removing duplicate traces in an error log that had corrupt trace meta data. The corrupt meta data was causing code to write to an invalid memory location. This change adds some checking of the meta data and traces some debug data that might be useful in debugging future cases of corrupt meta data. CQ: SW457275 Change-Id: If4d85e156202050e1c3faa4d4d991e94c9159c3f Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/74898 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: Ilya Smirnov <ismirno@us.ibm.com> Reviewed-by: Roland Veloz <rveloz@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Add callouts for NVDIMM errorsCorey Swenson2019-03-111-0/+20
| | | | | | | | | | | | | | | | | | - Add part callout for NVDIMM controller - Add part callout for Backup Power Module - Call new procedure for NVDIMM errors Change-Id: I33d14b8e4220ced3c632c8174eaed5faca4f088d RTC:199645 CMVC-Prereq:1077602 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/71977 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: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: TSUNG K. YEUNG <tyeung@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Initialize and test alt-master PNOR access in all bootsBill Hoffa2019-03-011-1/+28
| | | | | | | | | | | | | | | | | | - Update to istep 10 (call_host_slave_sbe_update) to validate the alternate master pnor chip in all boots (no-op for systems with only one pnor chip) - Updates to pnor callout logic to callout as a PNOR part for several of the error paths Change-Id: I9218f9a14496444288ea7985e1fb080c25f7f201 RTC: 200449 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/71489 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>
* Add VERSION to all OpenPOWER HBRT error logsMatt Raybuck2018-11-271-2/+94
| | | | | | | | | | | | | | | | | Now that the VERSION partition is a secure section, in order to have the version info in all runtime error logs the VERSION section has been put into reserved memory which can then be loaded and added to all HBRT OpenPOWER error logs. Change-Id: Iaf74d19270f8221710f30834097e131f4dadeeba RTC:200439 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/68855 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> Reviewed-by: Nicholas E. Bofferding <bofferdn@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Support openpower-specific I2C device calloutsJaymes Wilks2018-11-121-1/+6
| | | | | | | | | | | | | | | | | | | | | This change supports the openpower path for adding I2C device callouts to error logs. The process works as follows: - Create an I2C device lookup table on first use of I2C callout - Use that table to map I2C info to the actual device to callout - Callout any other I2C devices on the bus with lesser priority - If no I2C match found, callout the I2C master instead - If an I2C device was found, callout the I2C master as low Change-Id: Ib7b248ae60e7e834d6165bbdf4bd9b776ea2421b RTC:94872 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/64833 Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Reviewed-by: Nicholas E. Bofferding <bofferdn@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>
* Check for targeting before adding version info in errl commitCorey Swenson2018-10-261-3/+3
| | | | | | | | | | | | | | | | | addVersionInfo function calls spBaseServicesEnabled, which accesses targeting data. Moved the function call after the check for targeting loaded. Change-Id: Iaee25d71ce046821b7805f9fdb5a80bfa452f3fe CQ:SW449232 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/67730 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* genPnorImages: Add VERSION to signed partitionsSamuel Mendoza-Jonas2018-10-161-8/+15
| | | | | | | | | | | | | | | | | Mark VERSION as a signed partition and also check for it in the IPMI FRU setup. Change-Id: I6e8690c4f34d1f5ce92750960e6c73afcbf0a8e9 CQ:SW416159 Signed-off-by: Samuel Mendoza-Jonas <sjonas@au1.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/61439 Reviewed-by: Nicholas E. Bofferding <bofferdn@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Fix duplicate trace section removal accounting in error log flattenNick Bofferding2018-09-281-10/+8
| | | | | | | | | | | | | | | | | Prior to this change, if trace sections were removed in order to jettison duplicate trace entries, the error log's section count could be miscomputed, causing the unflatten API to detect corrupt error log content. This change tracks the actual number of sections flattened into the log, and updates the final section count in the private header accordingly. Change-Id: Id63582b6c73e3344bdc80a7fcb277526d806201f CQ: SW446517 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/66547 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 VERSION to all OpenPOWER error logsMatt Raybuck2018-09-071-0/+100
| | | | | | | | | | | | | | | | | | | When an error log is created for OpenPOWER systems it didn't contain the VERSION info within it. This commit adds a function called AddVersionInfo() that appends the VERSION info to every error log for OpenPOWER. Change-Id: Ic259e10e942f6df0675055823e4945ef9696b659 RTC: 196794 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/65506 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: 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>
* Combine trace buffers in errlogMatt Raybuck2018-08-281-2/+233
| | | | | | | | | | | | | | | | | | | Whenever ErrlEntry::collectTrace() was called it would create a new trace UD section each time. This led to duplicate entries in the errl. A new function was added to ErrlEntry to prune duplicates called removeDuplicateTraces(). Change-Id: I3685ecf1368dad421bca2d1b6b612e08755e25ce RTC: 118128 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/64971 Reviewed-by: Roland Veloz <rveloz@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> Reviewed-by: Nicholas E. Bofferding <bofferdn@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Add flag to HWAS Callout for SMP repair indicatorDan Crowell2018-08-191-4/+7
| | | | | | | | | | | | | Used one of the reserved bytes of the callout structure to hold a flag that will indicate if the SMP bus being called out is completely down, and thus eligible for concurrent repair. Change-Id: I299e3393529e811aa006a087de51165b79038201 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/63946 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>
* Implement generic i2c device callouts for FSPJaymes Wilks2018-08-131-2/+61
| | | | | | | | | | | | | | | | | | | Creates a new type of callout for i2c devices which populates the error log user details section with the address, port, engine, and i2c master of the device being called out. The change only affects the FSP path. Change-Id: I92acc092f105ee21346594f5e3733957e15ad768 RTC:94872 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/63223 Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Nicholas E. Bofferding <bofferdn@us.ibm.com> Reviewed-by: MURULIDHAR NATARAJU <murulidhar@in.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Minor Error Log ImprovementsMike Baiocchi2018-07-301-2/+14
| | | | | | | | | | | | | | | | | This commit adds the error log severity to the trace saying that a new error log is being created. It also cleans up an unused variable in some error log parsing logic. Change-Id: Iff196993bdabf80ad65006fca195f0785221257f Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/63476 Reviewed-by: Nicholas E. Bofferding <bofferdn@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: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: ILYA SMIRNOV <ismirno@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Add error callouts and other improvements for Node CommunicationsMike Baiocchi2018-07-031-2/+71
| | | | | | | | | | | | | | | This code adds bus, hw, code, etc callouts to the appropriate errors. It also cleans up other TODOs, refactors some functions, and applies somes fixes found in testing. Change-Id: I462cdb2c4e313c277a3e0542f740e82ede3313f3 RTC:184518 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/61813 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com>
* Fix mismatch between logger and parser for Istep state infoBrian Bakke2018-05-021-9/+2
| | | | | | | | | | | | | | | | | The logging function that added the Istep state to all Error Logs created data that was formatted differently than the parser that attempted to display this data was expecting. Change-Id: I2bcf5d86381baf33169c5677a34a4a3c21d4a1fa Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/57909 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> 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>
* Log Hostboot state information in all error logsDan Crowell2018-04-131-0/+16
| | | | | | | | | | | | | | | | Add the Istep number and substep number to Hostboot Error Logs to assist in problem debug. These items will be added as a user data record to all Hostboot Error Logs prior to Runtime. Change-Id: Ib5d269f3c1853d6d6856669744630dc130fa7094 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/54764 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: 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>
* Get Hostboot Dump after Hostboot CrashMarty Gloff2018-04-091-2/+4
| | | | | | | | | | | | | | | | | | | | | | When Hostboot has a TI with a PLID rather than a Reason Code (RC), HWSV does not attempt the Hostboot Dump. Thought is that happens per the design as the error log related to the PLID sent by Hostboot should already have the required information. There are instances where the error log does not have sufficient data and a dump would be beneficial. A path is being added so an error log can be flagged as needing a Hostboot dump. Change-Id: I97972308c70e7210f578fb818563bb9b0cd940b4 CQ: SW420219 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/56600 CI-Ready: Daniel M. Crowell <dcrowell@us.ibm.com> CI-Ready: 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> Reviewed-by: Corey V. Swenson <cswenson@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Support multiple nodes in HBRT - Concurrent Code UpdateMarty Gloff2018-04-091-2/+2
| | | | | | | | | | | | | | | | | | Enable HBRT to prepare for a concurrent code update on a multiple node system. Modify preparation function called by Host to loop through all nodes on the system. Make additional updates to go from supporting only a single node to supporting multiple nodes. Change-Id: I2d72b26abf40812641815051b968d6b540d27b00 RTC: 186584 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/53952 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>
* Trace cleanup: do not look for parent chip on non-parent chip targetsMatt Derksen2018-03-041-3/+3
| | | | | | | | | | | | | | | Cleanup this type of bad errl traces E> Failed to find a parent chip target for huid=00050000 Change-Id: I46713f185fbb4dd22739ad5b4f36f740bcccbc41 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/54709 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Brian E. Bakke <bbakke@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Remove invalid access to TARGETING::EntityPath in base imageDan Crowell2017-11-201-71/+69
| | | | | | | | | | | | | | | | | The addPartCallout path was triggering a call to the EntityPath constructor which leads to an Inst Exception. I saw the same issue in the other callout paths so fixed those too. Change-Id: I348e09d4f19d50edb39aea9093fa3d5701a0be39 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/49840 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: Christian R. Geddes <crgeddes@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Richard J. Knight <rjknight@us.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com>
* Send down OCC info logs to BMC for call-homeMatt Derksen2017-11-091-2/+7
| | | | | | | | | | | | | | | | | | This creates a new eSEL type (dd) to display informational callhome logs. The OCC will send down telemetry information to track the general health of the system using this new log. Change-Id: I0319798554c4e48c287953bd2d0de8352bfc4909 RTC:180324 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/48776 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 E. Bakke <bbakke@us.ibm.com> Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com>
* Added Error log support for new GPU sensorsMatt Derksen2017-09-011-0/+49
| | | | | | | | | Change-Id: I8a0de390516fd02df07860b960db506899b13f14 RTC:178218 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/45116 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>
* Add fault sensor information into IPMI error logsMatt Derksen2017-07-281-1/+83
| | | | | | | | | | | | | | | | Fault sensor number, FRU ID, and priority added to hardware callouts. Change-Id: I8e4d815a4a3b26db29f852d6160dcf4b12452660 RTC:134069 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/43604 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: Christian R. Geddes <crgeddes@us.ibm.com> Reviewed-by: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com>
* Avoid callouts, deconfigs, and gards for all non-visible errorsDan Crowell2017-07-211-3/+44
| | | | | | | | | | | | | | | | Fixed the deferred deconfig path that was still applying callouts for info logs Simplified where the check is for regular deconfigs Change-Id: I49ff4b0fe4ee81c28fde594c15bfd8f38bea1afc Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/43289 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: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Matthew A. Ploetz <maploetz@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com>
* Allow deconfig without a callout in error log.Matt Derksen2017-06-231-3/+4
| | | | | | | | | | | | Change-Id: Ia2ce6f89b4fbc4bbafa355cb1da07bdbd360c317 RTC: 174701 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/42085 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: Christian R. Geddes <crgeddes@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Changing NV target to OBUS_BRICK and mark it a child of OBUSPrachi Gupta2017-05-231-1/+0
| | | | | | | | | | | | Change-Id: I17b52033d1e6811c320f201056708f31ef34d0e9 RTC:171597 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/38481 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: Matthew A. Ploetz <maploetz@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Enforce synchronized processor security stateJaymes Wilks2017-05-101-2/+2
| | | | | | | | | | | | | | | Ensure the secure state is consistent across all processors. Change-Id: I671253b99d5d87509909681a2cec2380ca6918b0 RTC:167775 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/38764 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> Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com> Reviewed-by: Stephen M. Cprek <smcprek@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Handle addHwCallout with null targetCorey Swenson2017-05-101-1/+8
| | | | | | | | | | | | Change-Id: I8eb42ef8bb967d1ecd8e2f5f0c002ddeae44f578 CQ:SW386628 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/39939 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: Martin Gloff <mgloff@us.ibm.com> Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* move all non-volatile/openpower attributes from _hb to _openpower filesPrachi Gupta2017-05-011-1/+4
| | | | | | | | | | | | | | | | | | | When pushing hb's attr/target type files to common-mrw-xml repo, we add tags to represent which attributes are for openpower (bmc based system) and which are for fsp based systems. All attributes in _hb files are considered common for both platforms and do not have any tag. All attributes in _openpower will get openpower tag and be only shown to a system engineer if they are building a bmc based system. Therefore, the source of the attribute file is important from now on. Change-Id: I213348f4c4f372b0610b07ad3453f08f75d02224 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/38161 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Matthew A. Ploetz <maploetz@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Fix and guarantee order of target to subsys translation tableDan Crowell2017-04-281-115/+1
| | | | | | | | | | | | Change-Id: Idef7e41e358965afc7e670f2f2108742d37532f5 CQ: SW384972 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/39262 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: Corey V. Swenson <cswenson@us.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com>
* Revert "Adding in support for PHYP SRC and all 9 words"Elizabeth K. Liner2017-03-231-21/+3
| | | | | | | | | | This reverts commit a0c2aa6132a0b49975327f5aa9238d1d3e2b750b. Change-Id: I6cfe36de59e6981ff23aa7d9aea1474f14729d2d Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/38362 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>
* Adding in support for PHYP SRC and all 9 wordsElizabeth Liner2017-03-131-3/+21
| | | | | | | | | | Change-Id: I82c5b42c2167970437dae1323aa4aabed62068c9 RTC:158762 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/37514 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: Daniel M. Crowell <dcrowell@us.ibm.com>
* Fix reversed logic in old errl trace entryDan Crowell2017-03-131-2/+2
| | | | | | | | | Change-Id: I35534486b09ba7b58707efbadb1cd72af69afa3c Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/36946 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: Daniel M. Crowell <dcrowell@us.ibm.com>
* Avoid potential pnor deadlock for xscom errorsDan Crowell2017-03-101-2/+12
| | | | | | | | | | | | | | | | | | The problem we have here is if we get an xscom fail in the act of loading one of the HWAS code pages after the point where HWAS has been loaded at least once, or even that an HWAS code page is not present in memory when we take an XSCOM error. Change-Id: I1acf6586886ecd5f05575769405808a2a590df27 RTC: 130367 Backport: yes Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/37191 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Reviewed-by: Richard J. Knight <rjknight@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>
* Allow garding of CORE targetsBill Hoffa2016-09-121-56/+1
| | | | | | | | | | | | Change-Id: I1d605593309eb132334b749b8bc2ba84d5177f20 RTC: 158588 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/29392 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: DHRUVARAJ SUBHASH CHANDRAN <dhruvaraj@in.ibm.com> Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com> Reviewed-by: Matthew A. Ploetz <maploetz@us.ibm.com>
* Fill in some missing callout info for new P9 typesDan Crowell2016-08-301-0/+13
| | | | | | | | | | | | | | Added a few target->subsys maps Added a new OBUS callout type Change-Id: Ic986db332395b6d8ad174f3f5b96baee651b711b Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/28789 Tested-by: Jenkins Server <pfd-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> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Support for GPU callout procedureDan Crowell2016-08-301-0/+1
| | | | | | | | | | | | | | | | | Added EPUB_PRC_GPU_ISOLATION_PROCEDURE = FSPSP94 This procedure should be used to refer customers to the isolation procedure to identify NVLINK/GPU faults. Change-Id: I32d6ea318d2699e9f309cc9a048cb4ea805457e8 ForwardPort: yes CMVC-Prereq: 1002330 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/28781 Reviewed-by: Benjamin J. Weisenbeck <bweisenb@us.ibm.com> Reviewed-by: Sampa Misra <sampmisr@in.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/28910 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
* Fix pib err mask to actually look at the Userdetails section datacrgeddes2016-08-241-4/+8
| | | | | | | | | | | | | | | | Previously there was a bug with pib_err_mask where we incorrectly tried to cast a userdetails object into an incorrect structure. This caused an issue when we tried to access members of the struct that we were casting to. This commit changes the function that provides lookup of userdetails sections so it only returns the user data that the user details section contains. Change-Id: I38d6bbef8525a177b3f1baf72ea042c8e756c34d Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/28105 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Matt Derksen <v2cibmd@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Refactor fapi2 scom interface to use pib_err_maskcrgeddes2016-07-281-0/+17
| | | | | | | | | | | | | | Up to this point we have been ignoring pib err mask. This commit adds the pib_err_mask support to to platform RTC: 144507 Change-Id: Ie810f0915d6ba8091ea4740e705ba550dbad6eae Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/26382 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Andrew J. Geissler <andrewg@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: Daniel M. Crowell <dcrowell@us.ibm.com>
* Support new verification failure procedure calloutNick Bofferding2016-07-041-0/+1
| | | | | | | | | | | | | | | | | | | - Added new procedure callout for verification failure - Mapped callout to firmware subsystem - Added procedure decoder to error parser Change-Id: Ie94790df68c15ba100723a9eb8a7688a7e7612a2 Forwardport: yes CMVC-Prereq: 998934 RTC: 93870 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/26481 Reviewed-by: Martin Gloff <mgloff@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Elizabeth K. Liner <eliner@us.ibm.com> Reviewed-by: Stephen M. Cprek <smcprek@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/26585 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
* Only collect part/serial numbers for error logs if Targeting is upAndres Lugo-Reyes2016-03-181-25/+36
| | | | | | | | | | | | | | | Change-Id: Ic1b86c44c807c3e1ee4809103ead09dc5730482a CQ:323693 ForwardPort: yes Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/22143 Tested-by: Jenkins Server Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Reviewed-by: Matthew A. Ploetz <maploetz@us.ibm.com> Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/22201 Tested-by: FSP CI Jenkins
* PRD: add support for cooling issue calloutZane Shelley2015-11-041-0/+1
| | | | | | | | | | | | | | | | | Change-Id: I8d039ce100dbcd3f9e225014a2a88e80ba977f79 CQ: SW323280 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/20948 Reviewed-by: Zane Shelley <zshelle@us.ibm.com> Tested-by: Zane Shelley <zshelle@us.ibm.com> Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/20952 CMVC-Prereq: 974682 CMVC-Prereq: 961895 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/20954 Tested-by: Jenkins Server Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* set subsystem for part calloutsJay Azurin2015-09-031-0/+7
| | | | | | | | | | | | | Change-Id: I47e80e97d918a2e148324706bfc01b75ff83a5ec RTC:128430 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/20187 Tested-by: Jenkins Server Tested-by: Jenkins OP Build CI Tested-by: FSP CI Jenkins Tested-by: Jenkins OP HW Reviewed-by: Richard J. Knight <rjknight@us.ibm.com> Reviewed-by: Brian H. Horton <brianh@linux.ibm.com> Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* No SEL Callouts for Memory injects at runtimeRichard J. Knight2015-08-101-11/+71
| | | | | | | | | | | -Fixed runtime resolution of the flag which determines if an error log will be visable to the customer. Change-Id: Id8c44b0dcdb5714c163ac6485a53216786f9fbcf RTC:133158 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/19658 Tested-by: Jenkins Server Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* fixes to allow error logs to be pulled from PNOR to be resentBrian Horton2015-08-101-3/+14
| | | | | | | | | | | | | | | . correct logic in errorlog unflatten . remove unconditional 'ack' in errlmanager Change-Id: I170d6586062b6e4684a55782df83a1b19ed7d937 CQ: SW312094 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/19652 Tested-by: Jenkins Server Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Reviewed-by: STEPHEN M. CPREK <smcprek@us.ibm.com> Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* Ignore info/recovered logs in OpenPower boxesAndres Lugo-Reyes2015-07-021-1/+2
| | | | | | | | | | | Change-Id: Icb6f8b39ee7ab299f4292ac2d775adaf9c16721b RTC:126449 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/17807 Tested-by: Jenkins Server Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW Reviewed-by: Brian H. Horton <brianh@linux.ibm.com> Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* Add serial numbers/part numbers to err logs w/ hwcalloutsAndres Lugo-Reyes2015-06-191-0/+103
| | | | | | | | | | | Change-Id: Ifef77c71f40e70136cdb5172ce653f7a5ebfdd10 RTC:122890 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/16870 Tested-by: Jenkins Server Tested-by: Jenkins OP Build CI Reviewed-by: Brian H. Horton <brianh@linux.ibm.com> Reviewed-by: Corey V. Swenson <cswenson@us.ibm.com> Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
OpenPOWER on IntegriCloud