summaryrefslogtreecommitdiffstats
path: root/src/usr/ipmiext
Commit message (Collapse)AuthorAgeFilesLines
* Automatically include config.hDan Crowell2019-12-061-2/+1
| | | | | | | | | | | | | | | | | | 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 not crash if we have no functional nodesDan Crowell2019-10-301-43/+50
| | | | | | | | | | | | | | | | | | | We hit a case where i2c errors caused the node target to get marked as non-present. This led to an assert crashing Hostboot because it assumed we would always have a functional node0. The change here is to allow this case to flow through so that we end up with more recognizable error handling for the real failures during boot. Change-Id: Iaaaa4e531b677bda442da892cfbd6094e0897beb Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85169 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: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: Corey V Swenson <cswenson@us.ibm.com> Reviewed-by: William G Hoffa <wghoffa@us.ibm.com>
* Verify manufacture date (MB) from PVPD before usinglixg2019-10-031-35/+57
| | | | | | | | | | | | | | | | | | If the contents of the MB keyword is invalid, we could end up doing some illegal math and causing a crash. This change will add some simple verification of the data before we use it. Resolves #183 Change-Id: I2f3ead87be24e909a671f54f6005f3fffdbafd52 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84441 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: William G Hoffa <wghoffa@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Skip OCMB targets while decidng what targets to process for IpmiFruInvChristian Geddes2019-09-161-2/+7
| | | | | | | | | | | | | | | Some targets types will not have IPMI fru inventory classes associated with them, attempting to process them will result in an assert. Change-Id: I34cb080c981ac5d0008eb937c127e043ff077a27 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/83433 Reviewed-by: William G Hoffa <wghoffa@us.ibm.com> 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: 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>
* ipmidcmi: Get DCMI capabilities info to check PM supportMaxim Polyakov2019-07-231-1/+130
| | | | | | | | | | | | | | | | | | | According to section 6.6 Power management of DCMI spec, the DCMI Get Power Limit command isn`t supported if Get DCMI Capabilities Info command indicates no Power Management support. Commit adds code to get DCMI capabilities information from BMC to check this Tested on the Vesnin P8 server Change-Id: I3c4216e2ee4f5fa98ffbcfa95c4af6771f81b43c RTC: 123256 Resolves #176 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/80681 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: Martha Broyles <mbroyles@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* HB Improvements: Compiler Issues with Different Config FilesLuis Fernandez2019-04-051-1/+6
| | | | | | | | | | | | | | | 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>
* Constants for future DIMMsDan Crowell2019-04-021-2/+4
| | | | | | | | | | | | | | Just adding a few constants to indentify future DIMMs. Change-Id: Ib3bdf01c918162f0753458236c0c45a4b6e55274 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/70075 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: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: Matthew Raybuck <matthew.raybuck@ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com>
* Move watchdog timer to prevent timeoutMatthew Raybuck2019-02-262-206/+1
| | | | | | | | | | | | | | | | | | | | The watchdog used to be set in istepdispatcher.C but due to various changes to PNOR that would cause a watchdog timeout. This commit moves the watchdog out of impiext code and into impibase which allows the watchdog to be setup as soon as the ipmi resource provider comes up. Then once the code reaches istepdispatcher the watchdog is adjusted as needed. Change-Id: Ia5f13800db22cd17b0feabc5257f8f6409f54376 CQ:SW453071 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/72269 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> Reviewed-by: Ilya Smirnov <ismirno@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>
* Calculate capacity for 3DS DIMMs in IPMI FRU inventoryMaxim Polyakov2019-01-041-3/+26
| | | | | | | | | | | | | | | | | | This commit fixes a bug in formula for calculating total capacity of DIMM module that has memory ICs package with 3DS structure. In the case of 3DS modules, it is necessary to take into account the die count (byte 6 [bits 6: 4] in SPD) in the formula. In addition, the duplicate entry in the Manufacturer list has been removed. Resolves #158 Signed-off-by: Maxim Polyakov <m.polyakov@yadro.com> Change-Id: I3527684f14d833a203056464493f2dbfcdeec8ea Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/70055 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>
* Add TPM FRU Inventory Record DataJaymes Wilks2018-12-132-10/+154
| | | | | | | | | | | | | | | | | | This adds a FRU Inventory Record entry for TPMs. Due to the current lack of a VPD chip (for now) on any supported board, the manufacturer is assumed to be IBM and the part number is all zeros. Change-Id: I080be7d6bcebcf36e5eec5b3ac111a0e37a78587 RTC:189819 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/69257 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: Christian R. Geddes <crgeddes@us.ibm.com>
* Fix shutdown race condition and task start error in IPMI SEL libraryNick Bofferding2018-11-271-0/+4
| | | | | | | | | | | | | | | | | | | | | | | - Load libipmiext.so with START_TASK option, in order to notify error logger that errors can now propagate down to BMC via IPMI. Previously and erroneously, the library was loaded with the INIT_TASK option, preventing any SELs from flowing down to BMC. - Force IpmiSEL constructor to run when libipmiext.so is loaded, in order to register the shutdown handler with the init service. Without this, it's possible for the IpmiSEL constructor to run during shutdown itself, which down not handle new shutdown handler registrations properly. Change-Id: If333a8a6e1ce1ad8ca405dc2bb80fca3905ae674 CQ: SW451419 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/69094 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: Michael Baiocchi <mbaiocch@us.ibm.com> Reviewed-by: Marshall J. Wilks <mjwilks@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Zero out TPM FRU ID to avoid hostboot crashJaymes Wilks2018-11-121-0/+10
| | | | | | | | | | | | | | | | | This change allows us to update the MRW with the TPM FRU ID while ignoring its value in hostboot to avoid crashing. In a later commit we will handle the FRU ID properly. Change-Id: Iefef07f9db36f22581423e8a0cf61f1b0a034d1f RTC:194318 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/68543 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: Michael Baiocchi <mbaiocch@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Enable IPMI errl after targeting is initializedCorey Swenson2018-11-051-0/+14
| | | | | | | | | | | | | | | | | A recent change split IPMI into base and extended. Move the enable of IPMI errl from base to extended to avoid targeting errors. Change-Id: Ia833dd8178ff407dff1fa514b32468c5fc70e945 CQ:SW450286 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/68265 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: ANDREW R. JEFFERY <andrewrj@au1.ibm.com> Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Allow stopIPL to be called before istepdispatcher is loadedMatt Derksen2018-11-021-1/+1
| | | | | | | | | | | | | | | Hostboot was crashing because a graceful shutdown was requested before libistepdisp.so loaded. Change-Id: Id9206961bcbb72094ff359f26db720b92480f5fb CQ: SW445973 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/68110 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: Daniel M. Crowell <dcrowell@us.ibm.com>
* genPnorImages: Add VERSION to signed partitionsSamuel Mendoza-Jonas2018-10-161-1/+46
| | | | | | | | | | | | | | | | | 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>
* ipmi: Split into ipmibase and ipmiext modulesAndrew Jeffery2018-10-1018-0/+7253
Split the IPMI module into base and ext portions, with the BT interface, device driver and resource provider in the base portion, and all remaining IPMI functionality in the ext portion. The split is in preparation for moving the base functionality in the hostboot base image. Change-Id: Iec864f96240d79f4fadd5519d2ef46437d07c1fd Signed-off-by: Andrew Jeffery <andrewrj@au1.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/66792 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: 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>
OpenPOWER on IntegriCloud