summaryrefslogtreecommitdiffstats
path: root/src/usr/errl/errlmanager.C
Commit message (Collapse)AuthorAgeFilesLines
* Automatically include config.hDan Crowell2019-12-061-1/+0
| | | | | | | | | | | | | | | | | | Rather than having to remember to include config.h anywhere we reference a CONFIG variable (and usually forgetting), this adds it to the default compiler flags so that it gets included in every source file we build. Change-Id: I53622ab4d46c55d942e98cae6ec03049fd5b3d08 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/87475 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Zachary Clark <zach@ibm.com> Reviewed-by: Roland Veloz <rveloz@us.ibm.com> Reviewed-by: Christian R Geddes <crgeddes@us.ibm.com> Reviewed-by: Nicholas E Bofferding <bofferdn@us.ibm.com>
* Do nothing with error logs after shutdown event is received.Glenn Miles2019-10-151-0/+28
| | | | | | | | | | | | | | | | | | Disable processing of error logs by the error log server after receiving the shutdown event. This is to prevent the possibility of bad things from happening once other services are shutdown. CQ: SW465848 Change-Id: I59c4951182d43a78f5704b4b4c3379f10dd03182 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85278 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: Roland Veloz <rveloz@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Cache VERSION PartitionIlya Smirnov2019-08-291-2/+136
| | | | | | | | | | | | | | | | | | | | | | This commit adds logic to cache the VERSION partition so that errl commit function is able to use the cached data instead of sending messages to SPnorRP queue. Said messages also cause deadlocks if secureboot verifications fail, since the code attempts to send a synch message to the SPnorRP queue while it's handling another synch message. These changes only impact OP systems and code paths. Change-Id: I9eeb05878d98fb2891fe76adc06ad376edc77fbe CQ: SW465038 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82304 Reviewed-by: Nicholas E Bofferding <bofferdn@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Glenn Miles <milesg@ibm.com> Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* errl: Mark errlogMsgHandler() as detachedAndrew Jeffery2018-10-121-0/+2
| | | | | | | | | | | | | | | This ensures we crash if we can't process error log messages properly. Currently the boot will just hang without any obvious reason if the task crashes. Change-Id: I939f720b04274ec6718690b7126db7103f9f1003 Signed-off-by: Andrew Jeffery <andrewrj@au1.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/67274 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>
* Additional tracing for shutdown eventsMatt Derksen2018-09-121-2/+4
| | | | | | | | | | | | | | | | | Hit an issue where a shutdown was called but all that is traced is a msgQ pointer. These traces should help associate the msgQ pointer to an actual task being shutdown. Change-Id: Ic22fdde1a146eec90f26d557e8af1ddec3c85c4e CQ:SW443508 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/65464 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: Corey V. Swenson <cswenson@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Modify debug framework to be build-independentDan Crowell2018-06-261-0/+5
| | | | | | | | | | | | | | | | | During boot, Hostboot will push key pointers into memory. This allows the debug tools to find the pointers (using a known static memory address) to base memory accesses on. This replaces the existing symbol lookup that we use now. That means we don't need to have the exact symbol file for the build we're debugging against. Change-Id: I4618e15a3dc90acc3a89520a502eb818c1b4258c Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/56097 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 memory leaks associated with various msg_sendrecv callsNick Bofferding2018-06-041-1/+5
| | | | | | | | | | | Change-Id: I30bb6627610391aea248d4a3c4d722ef1695b02f Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/59677 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: ILYA SMIRNOV <ismirno@us.ibm.com> Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com> Reviewed-by: Nicholas E. Bofferding <bofferdn@us.ibm.com>
* Get Hostboot Dump after Hostboot CrashMarty Gloff2018-04-091-1/+9
| | | | | | | | | | | | | | | | | | | | | | 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>
* Fix rollover of PLID numbersMatt Derksen2018-02-211-3/+4
| | | | | | | | | | | | | | | | | | | | | Need to avoid incrementing into the processor node field of the PLID. Decided to use the highest bit in the last 3 bytes as a dividing bit. [0x##800000 - 0x##FFFFFF] -- for pre-boot setup errors. [0x##000000 - 0x##7FFFFF] -- for post setup errors When setting up from PNOR, we will look for the max EID in the post-setup range. If no max is found, restart at 0. Change-Id: I4a8811d9c93039090504ca9cf8c0eebc6812a4ca CQ:SW416592 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/54362 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>
* Ensure AbaPtr is not used in runtime codeDan Crowell2017-10-191-0/+1
| | | | | | | | | | | | | | | | | The AbaPtr makes assumptions that we have pointer values that are only 32-bits long. In our runtime (HBRT) environment that assumption is not true so we need to make sure none of the code that runs there is trying to use the AbaPtr code. Change-Id: I0a26558f305fada723bf3cb3447a2bdfddb194d0 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/48248 Reviewed-by: Martin Gloff <mgloff@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>
* Clear ECC sections marked "clearOnEccErr" on errorDean Sanner2017-09-141-1/+33
| | | | | | | | | | | | | | | | | | | | | | - Add the capability for Hostboot to recover (with reboot) when it consumes an ECC error - PNOR layout needs to be updated to flag the recoverable sections (generally cached or throw away data like *VPD HBEL, and GUARD partitions) - Upon bad ECC detection, Hostboot will check partition flag and if set, it will clear and write good ECC to PNOR. It will then throw the normal error and terminate, waiting for the BMC to issue a reboot Change-Id: Ie4f4c0637d3962e9d4871e84a0bda8c256a74440 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/44608 Reviewed-by: Stephen M. Cprek <smcprek@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: Daniel M. Crowell <dcrowell@us.ibm.com>
* Re-Initialize Targeting Variables after Attribute Override ProcessingBill Hoffa2017-09-111-1/+0
| | | | | | | | | | | | | Change-Id: Iebbc85cc75116f45e78b5b2d4bc5b1622c7f68f9 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/45525 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Christian R. Geddes <crgeddes@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: Prachi Gupta <pragupta@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* HB CI should not fail with committed RECOVERED error logThi Tran2017-06-191-2/+3
| | | | | | | | | | | Change-Id: I78fcc3e47359e013987c3e863ccd9b9dab92ae95 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/41836 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: Richard J. Knight <rjknight@us.ibm.com> Reviewed-by: Thi N. Tran <thi@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Increment reboot count when gard records are writtenMatt Ploetz2016-05-111-0/+43
| | | | | | | | | | | | | | | Change-Id: Iaf9a4d4be19877e564ca1871564467441b472253 CQ:SW347568 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/23186 Reviewed-by: A. P. Williams III <iawillia@us.ibm.com> Tested-by: Jenkins Server Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW Tested-by: FSP CI Jenkins Reviewed-by: Matthew A. Ploetz <maploetz@us.ibm.com> Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/24340
* Update constants and comments for P9 PIR formatDan Crowell2016-02-291-6/+6
| | | | | | | | | | | | | | | Implemented a set of macros and constants that can be used everywhere to translate a PIR into its component parts and pull out individual pieces of data from a complete PIR. Also added and updated the references to the old ATTR_FABRIC_NODE_ID with ATTR_FABRIC_GROUP_ID. Change-Id: If9735f53940e5849a648729e4bf8ca0cfbb09f6e RTC: 88055 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/706 Tested-by: Jenkins Server Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* P9 - FakePNOR scripts for Nimbus simicsMarty Gloff2015-12-111-3/+1
| | | | | | | | | | | | Tools/scripts to support a fake pnor configuration in Simics. Change-Id: Idf060cbfb589f3ccabde96d56bbe84b6cc9902a9 RTC:127337 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/19176 Tested-by: Jenkins Server Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Reviewed-by: Christian Geddes <crgeddes@us.ibm.com> Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* Base kernel changes for Nimbus/CumulusCorey Swenson2015-12-111-2/+3
| | | | | | | | | Change-Id: Ic5dfde1e975453d760631335bab674919e1109e7 RTC: 126637 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/18321 Tested-by: Jenkins Server Reviewed-by: Christian Geddes <crgeddes@us.ibm.com> Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* adding error message to console only when error existsManali Kumar2015-12-011-3/+4
| | | | | | | | | | | | | | Resolves open-power/hostboot#34 Change-Id: I5febe4f4e8fe6b8a6d0272bf40fce7f284762645 RTC:142083 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/22301 Tested-by: Jenkins Server Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com> Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW Tested-by: FSP CI Jenkins Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* fix issues with dup sel/eselBrian Horton2015-09-031-39/+11
| | | | | | | | | | | | | | | | | | . call errlmanager ack inline/synchronously, so that we don't have dup eSELs due to shutdown . don't resend sensor SEL events for errorlogs read out of PNOR Depends-On: Ia2b211d8ebf489e92f2bc3d281d92175a6b61f30 Change-Id: I32c8e49ef2e00567987cdf030a225b61e7077cf6 CQ: SW312094 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/19993 Tested-by: Jenkins Server Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW Tested-by: FSP CI Jenkins Reviewed-by: WILLIAM G. HOFFA <wghoffa@us.ibm.com> Reviewed-by: Richard J. Knight <rjknight@us.ibm.com> Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* add support to write errlog to PNOR during runtimeBrian Horton2015-09-031-435/+1
| | | | | | | | | | | | | | | | Change-Id: Ia2b211d8ebf489e92f2bc3d281d92175a6b61f30 Depends-On: I170d6586062b6e4684a55782df83a1b19ed7d937 Depends-On: Id8c44b0dcdb5714c163ac6485a53216786f9fbcf Depends-On: I626bce5c8c0c8b7d0a44408280de178c7a86a83f RTC: 123419 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/18233 Tested-by: Jenkins Server Tested-by: Jenkins OP Build CI Reviewed-by: STEPHEN M. CPREK <smcprek@us.ibm.com> Tested-by: Jenkins OP HW Reviewed-by: WILLIAM G. HOFFA <wghoffa@us.ibm.com> Tested-by: FSP CI Jenkins Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* No SEL Callouts for Memory injects at runtimeRichard J. Knight2015-08-101-3/+6
| | | | | | | | | | | -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-48/+65
| | | | | | | | | | | | | | | . 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-9/+68
| | | | | | | | | | | 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>
* Runtime error log support in OpenPOWERStephen Cprek2015-02-241-110/+0
| | | | | | | | | | | Change-Id: I1bb13fede249ee09a65541cd04c6449b1bb555b2 RTC: 122885 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/15712 Tested-by: Jenkins Server Reviewed-by: Brian H. Horton <brianh@linux.ibm.com> Reviewed-by: Corey V. Swenson <cswenson@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* Update to use uint32_t sensor numbersRichard J. Knight2015-02-191-3/+3
| | | | | | | | | | | | | | | | -Add frequency sensor type/numbers -Add getTargetFromSensor interface -Fix missing gaps in DIMM Temp sensor range -Add Centaur Temp sensors -Move getSensorNumber to targeting Change-Id: Id788a60b4dc0171402817408e8c6d6dcd637048c RTC:123523 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/15653 Tested-by: Jenkins Server Reviewed-by: Brian Silver <bsilver@us.ibm.com> Reviewed-by: Brian H. Horton <brianh@linux.ibm.com> Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* error log to esel integrationBrian Horton2015-02-161-78/+107
| | | | | | | | | | Change-Id: I2f134d73b114e4177e3e285defd85fbc21153913 RTC: 117454 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/15466 Tested-by: Jenkins Server Reviewed-by: STEPHEN M. CPREK <smcprek@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* add SEL/eSEL supportBrian Horton2015-01-221-37/+200
| | | | | | | | | | | in errlmanager, convert errlog to SEL/eSEL format, prepare to send to BMC via IPMI Change-Id: I11e246264840ff756e92b97673e4742b9d03daad RTC: 114908 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/14238 Tested-by: Jenkins Server Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* Send un-ACK'ed error logs in PNOR down to the BMCBrian Horton2015-01-161-4/+39
| | | | | | | | | | | | | | | when the errlmanager thread starts, it looks at all error logs in PNOR; any that were not ACK'ed during the previous IPL (meaning they were not confirmed as to have been processed by the BMC) will be put into the queue to be sent down once BMC contact has been setup. Change-Id: Ifef5a797eb53a1f28deefc90177dd958648e0210 RTC: 116461 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/14738 Tested-by: Jenkins Server Reviewed-by: STEPHEN M. CPREK <smcprek@us.ibm.com> Reviewed-by: Elizabeth Liner <eliner@us.ibm.com> Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* Fix spelling mistakes using codespell.Patrick Williams2015-01-151-3/+3
| | | | | | | | | | | | - See https://github.com/lucasdemarchi/codespell Change-Id: I03e102d1ebb9473b6226fa9b6edb684fa0218a2f Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/15031 Tested-by: Jenkins Server Reviewed-by: STEPHEN M. CPREK <smcprek@us.ibm.com> Reviewed-by: Zane Shelley <zshelle@us.ibm.com> Reviewed-by: Brian Silver <bsilver@us.ibm.com> Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* Fix errldipaly compTable to be sorted by numerical compId valueStephen Cprek2014-10-221-1/+1
| | | | | | | | | | | | Removed '\n' from errldisplay console prints as they add an extra new line. Don't see a need to print out label, more useful target info will be printed out in displayTarget Change-Id: I8c392f3c84d8acd4bda544e5e5b9668aebbcc0ed Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/14082 Reviewed-by: Brian H. Horton <brianh@linux.ibm.com> Tested-by: Jenkins Server Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* Add break in errlmanager to fix errldisplayStephen Cprek2014-10-171-0/+2
| | | | | | | | Change-Id: I52736d6c0e6d544cd20d7f5aebe38a7af4a8ef12 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/14060 Tested-by: Jenkins Server Reviewed-by: Brian H. Horton <brianh@linux.ibm.com> Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* Add human-readable dumps of error log entries as they are committed.Stephen Cprek2014-10-161-163/+205
| | | | | | | | | | Origin: Google Shared Technology Change-Id: I1d0a30faa27c0b8bd1b76418706e24378a5da7eb RTC: 97491 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/11266 Tested-by: Jenkins Server Reviewed-by: Brian Silver <bsilver@us.ibm.com> Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* Throttle mailbox operations to avoid out-of-memory conditionDoug Gilbert2014-08-121-3/+14
| | | | | | | | | | Change-Id: I18f13ec742efe196e6a2bc3714051e2489567158 RTC: 102852 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/11586 Tested-by: Jenkins Server Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Reviewed-by: Corey V. Swenson <cswenson@us.ibm.com> Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* Change copyright prolog for all files to Apache.Patrick Williams2014-05-211-10/+10
| | | | | | | Change-Id: I5664587b4f889099290ef50d50fa9ce5e580e1eb Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/11167 Tested-by: Jenkins Server Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* Look at eid and plid when going through errlogsStephen Cprek2014-03-201-4/+22
| | | | | | | | | | | | | | | | HB needs to look for the case where a special REF code is provided by the FSP and only look at its PLID Change-Id: I299bf6056bcfaca37bc60449a745a9bbeaa754c2 RTC: 99598 CQ: SW251889 Backport: release-fips810 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/9703 Reviewed-by: MIKE J. JONES <mjjones@us.ibm.com> Reviewed-by: Brian H. Horton <brianh@linux.ibm.com> Tested-by: Jenkins Server Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* Ensure error logs during IPL are caught as CxxTest fail.Patrick Williams2014-03-071-1/+16
| | | | | | | | | | | RTC: 39871 Change-Id: I6a3045609d696f5d7d6f3d240da17a150ad70af7 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/9362 Reviewed-by: Brian H. Horton <brianh@linux.ibm.com> Tested-by: Jenkins Server Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Reviewed-by: Andrew J. Geissler <andrewg@us.ibm.com> Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* don't write INFORMATIONAL errorlogs to PNORBrian Horton2014-02-061-39/+54
| | | | | | | | | | Change-Id: I63e82c349e5e5dec8357034e38b7adfa01fcd4a0 RTC: 96089 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/8370 Tested-by: Jenkins Server Reviewed-by: Donald E. Dahle <dedahle@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* Handle Multiple TOCs and removed side codeStephen Cprek2013-12-121-3/+1
| | | | | | | | | | | | | Also added Hostboot Base image version header Change-Id: I0fc878a48b9449e5d4875fd14525faefe01b1ace RTC: 34764 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/7276 Tested-by: Jenkins Server Reviewed-by: Brian H. Horton <brianh@linux.ibm.com> Reviewed-by: ADAM R. MUHLE <armuhle@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* fix minor issues in errlmanager codeBrian Horton2013-12-121-17/+41
| | | | | | | | | | | | | . on ack from FSP, check ToSave list; remove if there; . remove checks that don't need to be there; Change-Id: Id518625fd5efb4a8d6666d80e6310e8e02869874 RTC: 92123 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/7554 Reviewed-by: Donald E. Dahle <dedahle@us.ibm.com> Tested-by: Jenkins Server Reviewed-by: MIKE J. JONES <mjjones@us.ibm.com> Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* fix bug that caused double free in errlmanagerBrian Horton2013-12-041-2/+8
| | | | | | | | | | | | | code was incorrectly returning the pointer to a msg that was successfully sent to the FSP; result was that the shutdown code would try to free the msg, when it had already been processed. Change-Id: I37a02187db69d5c49f486b6a0d7c94a33db66973 CQ: SW235512 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/7537 Reviewed-by: Andrew J. Geissler <andrewg@us.ibm.com> Tested-by: Jenkins Server Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* store error logs in PNORBrian Horton2013-11-141-172/+644
| | | | | | | | | | | | | | . store error logs in PNOR, truncating to 4K . set and clear ACKnowledge bit as appropriate . move PNOR and MBOX init code in errlmanager until after PNOR and MBOX services are actually up and running Change-Id: I57c5bacae8ffd283577164a0706c081a8ef5af30 RTC: 47517 Reviewed-on: http://gfw160.austin.ibm.com:8080/gerrit/6884 Tested-by: Jenkins Server Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* Fix race conditions in initservice shutdown path.Patrick Williams2013-10-151-3/+8
| | | | | | | | | | Change-Id: I0da3c2050d5d64d20975031e093dd10978684e2b Reviewed-on: http://gfw160.austin.ibm.com:8080/gerrit/6663 Tested-by: Jenkins Server Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Reviewed-by: Brian H. Horton <brianh@linux.ibm.com> Reviewed-by: Andrea Y. Ma <ayma@us.ibm.com> Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* Resolve race condition in errl shutdown.Patrick Williams2013-10-101-2/+3
| | | | | | | | | | | | | | The errorlog manager is responding to the shutdown message prior to fully doing its own shutdown. This is bad because it hasn't actually push the error log content to PNOR yet and the system could shutdown before this is complete. For some reason, this is also causing pervasive failures in Jenkins (due to manipulating the shutdown-callback list while other code is using it). Change-Id: I25377b1bfbd4503b10a18444fb67eca3912e3d11 Reviewed-on: http://gfw160.austin.ibm.com:8080/gerrit/6627 Tested-by: Jenkins Server Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* Implement termination from fatal error log.Richard J. Knight2013-10-081-25/+46
| | | | | | | | | Change-Id: Ie83f0876887ee0465cd4d430fa4a335f6aa396ec RTC:35268 Reviewed-on: http://gfw160.austin.ibm.com:8080/gerrit/6403 Tested-by: Jenkins Server Reviewed-by: Brian H. Horton <brianh@linux.ibm.com> Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* Determine starting error log id.Brian Horton2013-09-131-38/+75
| | | | | | | | | | | | | errlManager will set starting log id based on the node (0x9#) and the highest id of the error logs found in memory. Change-Id: I1867b83d069e048fcb411bb243b6eaf62ab578c7 RTC: 68687 Reviewed-on: http://gfw160.austin.ibm.com:8080/gerrit/6062 Tested-by: Jenkins Server Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Reviewed-by: Andrea Y. Ma <ayma@us.ibm.com> Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* implement deferred deconfigure supportBrian Horton2013-06-141-12/+4
| | | | | | | | | | | | | deferred deconfigure record is created when an error log with a DELAYED_DECONFIGURE callout is commited; at the end of the istep worker function, all outstanding deferred deconfigure records are processed. Change-Id: I3b6623c4f1e49f362d5b3302a964823810ea18ec RTC: 45781 Reviewed-on: http://gfw160.austin.ibm.com:8080/gerrit/4786 Tested-by: Jenkins Server Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* handle MASTER_PROCESSOR_CHIP_TARGET_SENTINEL as approporiateBrian Horton2013-04-091-6/+28
| | | | | | | | | | | | | | | | | | . create new errl->callout function to handle calling hwas::processCallout, so that can be done in proper order (after commit, save to PNOR and send to FSP of errl) . in callout code, convert SENTINEL target to masterProc . a deconfigure of the masterProc means that we can't continue, so return flag so that doShutdown() will be called to halt the IPL (TI). Change-Id: Ie43b8caf28d3aea20c15827e035dd890378dbcee RTC: 45780 Reviewed-on: http://gfw160.austin.ibm.com:8080/gerrit/3695 Tested-by: Jenkins Server Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* Triggering Hostboot Shutdown when PNOR is badAdam Muhle2013-02-041-3/+2
| | | | | | | | | | | | | Updating the doShutdown path to support receiving a reason code as input. Then changing PNOR RP to issue a shutdown when problems are detected with the PNOR Partition table. RTC: 44146 Change-Id: Ib4111d0a91f53d90fa100422a1463539897598e6 Reviewed-on: http://gfw160.austin.ibm.com:8080/gerrit/3024 Tested-by: Jenkins Server Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* Report host boot error log to FSPPaul Nguyen2012-10-301-24/+334
| | | | | | | | | | Change-Id: Iad4be4f0a45a607f7b56fc5d194f1812f74fbfe8 RTC: 34235 Reviewed-on: http://gfw160.austin.ibm.com:8080/gerrit/1949 Tested-by: Jenkins Server Reviewed-by: MIKE J. JONES <mjjones@us.ibm.com> Reviewed-by: Brian H. Horton <brianh@linux.ibm.com> Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* Support code coverage in extended modules.Patrick Williams2012-10-091-10/+6
| | | | | | | | | | | | | | | | | - Reduce optimization (to -Os) to fit when doing coverage profile. - Remove errl storage area from base image. - Add GCC function attributes to sys library functions. RTC: 36933 Change-Id: Ic83011a2444ef5b735db0446a14a0af34187eebf Reviewed-on: http://gfw160.austin.ibm.com:8080/gerrit/1908 Tested-by: Jenkins Server Reviewed-by: Brian H. Horton <brianh@linux.ibm.com> Reviewed-by: ADAM R. MUHLE <armuhle@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Reviewed-by: Melissa J. Connell <missyc@us.ibm.com> Reviewed-by: Paul Nguyen <nguyenp@us.ibm.com> Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
OpenPOWER on IntegriCloud