summaryrefslogtreecommitdiffstats
path: root/src/usr
Commit message (Collapse)AuthorAgeFilesLines
* Add support for NVDIMM secure erase verifyCorey Swenson2020-01-175-104/+509
| | | | | | | | | | | | | | | | | | Add nvdimm operations to interface Add wrapper for factory default function Add secure erase verify start function Add secure erase verify status function Change-Id: I84774e679593e7df1907c1a442c831b2f4df88d9 CQ:SW475562 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84301 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: Roland Veloz <rveloz@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Remove workaround added to increment MCS timeout for gemini'sChristian Geddes2020-01-171-51/+1
| | | | | | | | | | | | | | | | We are no longer using or supporting gemini cards so we can remove this workaround. Change-Id: I76303832cff93f8a8b5adec1d204ccb5ca827bc2 RTC: 248023 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/89267 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: Yun Pan <yun.pan@ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* NVDIMM : Catch save error during MPIPLCorey Swenson2020-01-171-0/+4
| | | | | | | | | | | | | | | | During MPIPL, call nvdimm_init() from nvdimmRestore() to catch errors skipped during SAVE step. Change-Id: I5a4177af59b0a5909ece27700cc3d51840e29440 CQ:SW476125 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/89682 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: Roland Veloz <rveloz@us.ibm.com> Reviewed-by: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: Nicholas E Bofferding <bofferdn@us.ibm.com>
* Fix for missing HBRT error log after error injectedCorey Swenson2020-01-151-2/+8
| | | | | | | | | | | | | | | | | | An HBRT error log larger than 4K sent to the host will be truncated to 4K before it reaches HWSV, where a size mismatch prevents processing of the log. To avoid this, truncate the HBRT log to 4K before sending across interface to host. Change-Id: Ic2c354da99c0a4ccec00ce58cf6ef34bc02e7996 CQ:SW480758 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/88724 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: Roland Veloz <rveloz@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Add more rc values to vmmpagetestDan Crowell2020-01-131-18/+20
| | | | | | | | | | | | | | Adds a little more ffdc to some testcases Change-Id: If6a69506c0c88e1c3c3d3affc764cad72ff3418a Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/87576 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: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: Nicholas E Bofferding <bofferdn@us.ibm.com>
* Add obj tree to EXTRAINCDIR for istep10Dan Crowell2020-01-101-1/+2
| | | | | | | | | | | | Some generated files will be needed by some HWPs in this istep. Change-Id: Ia8cbb87a4a9dfc5ede40261854df7ce3f22d9d8a Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/89403 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>
* NVDIMM: Mask mbacalfir at the beginning of restore step during mpiplMatt Derksen2020-01-102-3/+41
| | | | | | | | | | | | | | | | | Mask MBACALFIR[8]: EVENT_N: MBACALFIR_MASK_OR = 0x0080000000000000 at the beginning of nvdimm_restore() to avoid PRD errors. This mask is normal for non-mpipl path, so just needed for mpipl. CQ: SW477028 Change-Id: I3368b89af67f58edecb42dd0a99ca7799679e84d Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84190 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Reviewed-by: TSUNG K YEUNG <tyeung@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>
* Correct target hiearchy in axone system xmlChristian Geddes2020-01-101-2/+2
| | | | | | | | | | | | | | | Previously the comment at the top of this xml file stated that a MCC has 4 OMIs when in reality it only has 2. Change-Id: I442beceefcec5ad3601d7b2ec96f89ab52096b46 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/89268 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: Ilya Smirnov <ismirno@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Handle automatic interleaving between OMI sub-channelsDan Crowell2020-01-101-17/+17
| | | | | | | | | | | | | | | | | | | The memory channel is defined as a single MCC, and all of the memory on that channel is part of a single address space. That means that both OCMBs on the same MCC share an address space. While this isn't the same mechanism as true interleaving, it looks the same to the code consuming HDAT, so we need to set the sharing flags appropriately. Change-Id: Ic270582ced92614cf21ff7be0ef6f97c93ab2514 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/89340 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: Jayashankar Padath <jayashankar.padath@in.ibm.com> Reviewed-by: Christian R Geddes <crgeddes@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Move p9a_omi_setup_bars from 12.4 to 12.3Dan Crowell2020-01-102-49/+52
| | | | | | | | | | | | | | | The code wasn't matching the IPL Spec. There is really no change to the order things ran, but now it matches the doc. Change-Id: I35e3186d30bc946bf882e4657900a200ee1aa1b9 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/88949 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: Christian R Geddes <crgeddes@us.ibm.com> Reviewed-by: William G Hoffa <wghoffa@us.ibm.com>
* Add SBE Arch dump area for both OPAL & PHYPDean Sanner2020-01-085-74/+141
| | | | | | | | | | | | | | | Currently Hostboot only sets up the architected dump capture on the SBE for OPAL based systems. This commit adds the setup to PHYP based systems as well. Change-Id: I96271a57f7c15c571495ed56ea72dbeeebab8d94 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/88095 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: MURULIDHAR NATARAJU <murulidhar@in.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Support direct load of POWERVM from BOOTKERNEL partitionDean Sanner2020-01-072-1/+9
| | | | | | | | | | | | | | | | | PowerVM doesn't fit in the PAYLOAD section of PNOR. To enable a common PNOR layout instead put PowerVM into BOOTKERNEL partition, and if PHYP payload, load BOOTKERNEL instead of PAYLOAD PNOR partition Change-Id: Id19f3ce708b537a19256c22aab3cc1b3e83d4ab1 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/71677 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> Reviewed-by: MURULIDHAR NATARAJU <murulidhar@in.ibm.com> Reviewed-by: William G Hoffa <wghoffa@us.ibm.com>
* Enable copying of architected registers for OPALMurulidhar Nataraju2020-01-071-2/+2
| | | | | | | | | | | | based systems. Modified the check to be based on the hypervisor type. Change-Id: I2df21339160e0658feb1797dbcd7e3c4ae33c2f3 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/89285 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>
* Enhancements to default console outputDan Crowell2020-01-075-6/+158
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added the following pieces of extra information to the default console output: -- Attribute Overrides -- 3.49165|**Found 24 attribute overrides in Tank FAPI(1) 3.49169|- PEC:n0:p0:c0 3.49171| ATTR 0EB604DF [1] = 00 3.49174| ATTR 085F1C7E [1] = 00 3.49177| ATTR 048A8902 [1] = 01 3.49180| ATTR 04D8DF8D [2] = 0000 3.49182|- PEC:n0:p0:c1 3.49185| ATTR 0EB604DF [1] = 00 3.49188| ATTR 085F1C7E [1] = 00 3.49191| ATTR 048A8902 [1] = 03 -- New part detection -- 4.97449|Detected new part : 00030002 (Physical:/Sys0/Node0/DIMM2) -- Applying GARD records -- 8.78767|HWAS|Applying GARD record for HUID=0x000E0000 (Physical:/Sys0/Node0/Pr oc0/XBUS0) due to 0x90000135 8.78912|HWAS|Deconfig HUID 0x000E0000, Physical:/Sys0/Node0/Proc0/XBUS0 Change-Id: Ic2ae04515453dbb49e359abff952c87c83e23b72 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/88388 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Nicholas E Bofferding <bofferdn@us.ibm.com> Reviewed-by: Roland Veloz <rveloz@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>
* Call p9a_throttle_sync inside mss_scominitDan Crowell2020-01-074-7/+63
| | | | | | | | | | | | | | | | | | | | | | | | We need to setup the memory throttles for worstcase mode for a portion of the boot before we have the real values later in mss_thermal_init. Changes to IPL flow - 13.8 mss_scominit - exp_scominit() - p9a_throttle_sync() <<< new 14.2 mss_thermal_init - exp_mss_thermal_init() - p9a_throttle_sync() <<< current place, stays here but also called earlier Change-Id: Iebf4ccb0f8b3f84ca146314b2a5d94ccbe88fc06 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/88118 Reviewed-by: Christian R Geddes <crgeddes@us.ibm.com> Reviewed-by: Corey V Swenson <cswenson@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: William G Hoffa <wghoffa@us.ibm.com>
* Add current istep into TI SRCDan Crowell2019-12-181-0/+4
| | | | | | | | | | | | | | | | | | Word4 of the SRC is defined to be the 'last progress code' for FSP SRCs. For Hostboot TIs, we currently leave that word zero. This change will add the same data that we put into the scratch register 5 into word4 so that we will have the failing istep for any TI we encounter. Change-Id: Iaf5ec835d45b1ea3a6ced20b5b7f7d07216c548e Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/88813 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: Ilya Smirnov <ismirno@us.ibm.com> Reviewed-by: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: Nicholas E Bofferding <bofferdn@us.ibm.com>
* Switch DECONFIG over to DELAYED_DECONFIGDan Crowell2019-12-173-5/+5
| | | | | | | | | | | | | | | | | | There is a race condition with the HWSV code on the FSP related to how deconfigurations are happened. A full solution on the FSP side would be difficult so we're making a few targeted fixes on the Hostboot side instead. Change-Id: I57a6ec80df77464131020499e43181c1869d2514 CQ: SW481625 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/87928 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>
* Replace descriptions with JEDEC register namesArtem Senichev2019-12-161-88/+39
| | | | | | | | | | | | | | | | | | | | | | For some NVDIMM errors, Hostboot generates a PEL record, that contains a dump of NVDIMM controller registers. The values of these registers can be decoded and analyzed using the JEDEC specification JESD245, but it is difficult to do without standard register names. This changes replaces human readable descriptions with JEDEC specific names in the errl plugin implementation. Signed-off-by: Artem Senichev <a.senichev@yadro.com> Resolves #189 Change-Id: If7190772485530514e3ba21f48a04143de5a08d7 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/87583 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: Roland Veloz <rveloz@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* PRD: NVDIMM only update status bits for predictive logsCaleb Palmer2019-12-121-14/+21
| | | | | | | | | | | | | | Change-Id: I55251bdf32a32ecccebe1645067668ea3328cec9 CQ: SW482090 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/88167 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Benjamen G Tyner <ben.tyner@ibm.com> Reviewed-by: Brian J Stegmiller <bjs@us.ibm.com> Reviewed-by: Zane C Shelley <zshelle@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/88470 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>
* Always send NV_STATUS to Opal when opal-prd startsCorey Swenson2019-12-122-0/+36
| | | | | | | | | | | | | | | Opal does not persist NVDIMM status state so HBRT must send this info to Opal when opal-prd starts. Change-Id: I0771c63ff2857f9b70fbfad5760267e3da463009 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/87541 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: Roland Veloz <rveloz@us.ibm.com> Reviewed-by: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* PRD: Uncomment Axone HWP codeCaleb Palmer2019-12-0612-100/+165
| | | | | | | | | | | | | | Change-Id: I1d60cd40e93974a9a71bee35d27b2a2b7f3be510 RTC: 207273 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/86516 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Benjamen G Tyner <ben.tyner@ibm.com> Reviewed-by: Brian J Stegmiller <bjs@us.ibm.com> Reviewed-by: Zane C Shelley <zshelle@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/88206 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>
* PRD: Axone Dynamic Mem Dealloc addr translate insert Grp IDCaleb Palmer2019-12-061-11/+114
| | | | | | | | | | | | | | Change-Id: I43130be2841643d200b2c92c1d7ba3455c479dd5 RTC: 199036 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/87694 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Benjamen G Tyner <ben.tyner@ibm.com> Reviewed-by: Brian J Stegmiller <bjs@us.ibm.com> Reviewed-by: Zane C Shelley <zshelle@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/88205 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>
* PRD: Axone Dynamic Mem Dealloc Addr Translate Port AddrCaleb Palmer2019-12-063-78/+357
| | | | | | | | | | | | | | Change-Id: Iefc25ed1fa063078b180d3d2fcf4e1d862481d40 RTC: 199036 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/86747 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Benjamen G Tyner <ben.tyner@ibm.com> Reviewed-by: Paul Greenwood <paul.greenwood@ibm.com> Reviewed-by: Zane C Shelley <zshelle@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/88204 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>
* PRD: Update CE/UE flood threshold to reset on new ranksCaleb Palmer2019-12-0610-47/+137
| | | | | | | | | | | | Change-Id: I89dce691642ebf4d753812bfae111d14c52753e3 CQ: SW480922 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/87032 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Zane C Shelley <zshelle@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/88203 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>
* Add calls to p9a_disable_ocmb_i2c HWP in istep 8.12Christian Geddes2019-12-062-0/+64
| | | | | | | | | | | | | | | | | When in secure mode, after we get the memory mapped io to the OCMBs set up we can disable i2c access as it is no longer needed and it presents a security hole. Change-Id: Ibfce02b7a6be6534be901dbca75e7c950ad1a9bc RTC: 209320 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/88083 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>
* Filter out i2c slaves that are not associated with a given masterChristian Geddes2019-12-061-12/+4
| | | | | | | | | | | | | | | | | | | During axone bringup a workaround was put in place to avoid some of the logic that was used in witherspoon systems. It turns out for axone we want the same behavior. This commit reverts the behavior of the hdatGetI2cDeviceInfo to what is was previous to the workaround. Change-Id: I64ab0a28fda22d1709c5658ea36ce69d88c1deba RTC: 213230 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/88178 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: Jayashankar Padath <jayashankar.padath@in.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com> Reviewed-by: Nicholas E Bofferding <bofferdn@us.ibm.com>
* Enable Serial TestcasesBill Hoffa2019-12-063-11/+104
| | | | | | | | | | | | | | | | - Provide the ability for test modules to be defined as serial tests - Execute the serial test jobs prior to launching all of the other test modules in parallel Change-Id: Iff10ad049f6bb0e5f0f3c168a1bb30736b1b983d Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/83110 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>
* Automatically include config.hDan Crowell2019-12-06141-214/+71
| | | | | | | | | | | | | | | | | | 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>
* Add missing target types for OCMBDean Sanner2019-12-051-2/+8
| | | | | | | | | | | | Need to add missing unit numbers for FAPI ATTR Dump Change-Id: Icb6a332e7f7e6a7177dc2efc2d5e87e552f06f2c Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/87392 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>
* For ATTR_MSS_OCMB_HALF_DIMM_MODE to full dimm modeDan Crowell2019-12-051-0/+8
| | | | | | | | | | | | pSeries always runs in full-dimm mode. Change-Id: I8515c7d600993330abfd94da4fd810e07f11e230 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/87998 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>
* Correct ptr math and force CI i/o in kernal for machchk escalationChristian Geddes2019-12-031-5/+2
| | | | | | | | | | | | | | | | | | | | | When we take machine checks in hostboot we will escalate them to a checkstop by setting bit 31 in the PB Fir reg 0x5012000. This is done via xscom in some kernel code. There were 2 bugs found in this path. The first was in how we were calculating the MMIO addressed used to perform the xscom. The other issue was that we were not forcing the kernel code to perform a cache-inhibited store. This commit addressed both of these problems. Change-Id: I41871fa754cbedf4cdb913711ce02a00f0213334 CQ: SW481030 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/87539 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> Reviewed-by: Nicholas E Bofferding <bofferdn@us.ibm.com> Reviewed-by: William G Hoffa <wghoffa@us.ibm.com>
* Add blank files for explorer plug rules libsMark Pizzutillo2019-11-261-0/+1
| | | | | | | | | | | | | | Change-Id: Idce86b2c9aff7d944bd5ad2daafff1261fd57809 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/87533 Reviewed-by: Louis Stermole <stermole@us.ibm.com> Reviewed-by: STEPHEN GLANCY <sglancy@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/87656 Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Christian R Geddes <crgeddes@us.ibm.com>
* Fix auto-arming NVDIMMs for ESS systemsCorey Swenson2019-11-194-825/+841
| | | | | | | | | | | | | | | | | | | ESS requires HB to arm the NVDIMMs during IPL after PM complex is started. Requires moving the nvdimmArm() and associated functions from nvdimm_rt.C to nvdimm.C. Change-Id: Ie7fd08b27ad7959b21086f0f291764c2156e5bd9 CQ:SW479909 CMVC-Prereq:1096403 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/86489 Reviewed-by: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: Roland Veloz <rveloz@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 DIMM_BAD_DQ_DATA field for DDR4 DDIMM SPD and bad dq testsCaleb Palmer2019-11-193-0/+229
| | | | | | | | | | | | | | | Change-Id: I1242a9f8733679afb0fee632e985ccf28096fbd9 RTC: 245516 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/87098 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: Zane C Shelley <zshelle@us.ibm.com> Reviewed-by: Paul Greenwood <paul.greenwood@ibm.com> Reviewed-by: Brian J Stegmiller <bjs@us.ibm.com> Reviewed-by: Benjamen G Tyner <ben.tyner@ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Add NVDIMM opertions to rt_cmdsCorey Swenson2019-11-181-1/+44
| | | | | | | | | | | | Change-Id: I7b7fbe5c0248d749e80bfc3b8eaa7daa7dbf7a65 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/86759 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: Roland Veloz <rveloz@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Add support for NULL char in vendor logCorey Swenson2019-11-181-2/+51
| | | | | | | | | | | | | | | | | | | SMART is adding a NULL char to indicate the last entry added to the log. This is useful when the log wraps. A single NULL char indicates the log has wrapped. More than one NULL char in a row indicates the log has not wrapped. Display the log in chronological order. Change-Id: Ic0404b60ad617b7c0c6a09b710e54cb1736bd9d4 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/86630 Reviewed-by: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: TSUNG K YEUNG <tyeung@us.ibm.com> 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: Daniel M Crowell <dcrowell@us.ibm.com>
* Allow HWPs that send BOOT_CONFIG command to ocbm to run in simicsChristian Geddes2019-11-172-6/+0
| | | | | | | | | | | | | Previously simics was not responding correctly to this command but the latest simics versions have updates so they will give us what we expect. Change-Id: I1c009b834eb0081e3b806414d48409e23b6280ef Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/86871 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>
* Move KEY_CLEAR_REQUEST attribute from node level to system levelMike Baiocchi2019-11-161-3/+3
| | | | | | | | | | | | | Change-Id: I2de1bcbfb7a8d8adfa47a6e62bdbad3ab2514a55 RTC:210301 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/87081 Reviewed-by: Christopher J Engel <cjengel@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: Nicholas E Bofferding <bofferdn@us.ibm.com>
* Add Physical Presence Check and Window Open FeaturesMike Baiocchi2019-11-1411-11/+615
| | | | | | | | | | | | | | | | | | | | | | | | This commit does the following: - Adds an interface to detect if physical presence has been asserted -- This happens in istep 6 -- If the window is open to detect this, it is then closed here - Adds an interface to possibly open the window to look for physical presence -- This happens in istep 10 -- It first checks to see if the window should be opened -- If the window is opened then the system shuts down to wait for physical presence to be asserted on the next power on - Adds the necessary attributes to support and test this functionality RTC:211220 Change-Id: I05a26ebad581875a4b9f2a51eb1ca3062f36c5fb Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84656 Reviewed-by: Ilya Smirnov <ismirno@us.ibm.com> Reviewed-by: Christopher J Engel <cjengel@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: Nicholas E Bofferding <bofferdn@us.ibm.com>
* Support GPIO Physical Presence Detect DevicesMike Baiocchi2019-11-146-16/+465
| | | | | | | | | | | | | | | | | | | | This commit makes the necessary changes to support the GPIO devices used for physical presence detection: - New attribute GPIO_INFO_PHYS_PRES - Enable GPIO Device Driver by default - GPIO Device Driver updated to support new device type - Added new GPIO PCA9551 layer to sit on top of GPIO Device Driver Change-Id: I952007503eb5d9fe873adea2eaaeccff21ae18a6 RTC:211220 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84655 Reviewed-by: Ilya Smirnov <ismirno@us.ibm.com> Reviewed-by: Christopher J Engel <cjengel@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: Nicholas E Bofferding <bofferdn@us.ibm.com>
* MDIA: Uncomment HWP callsCaleb Palmer2019-11-123-27/+46
| | | | | | | | | | | | | | Change-Id: I23f0abb7b6d26c2fa6d197ceaa0cc276ec461594 RTC: 201293 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/86476 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Brian J Stegmiller <bjs@us.ibm.com> Reviewed-by: Benjamen G Tyner <ben.tyner@ibm.com> Reviewed-by: Paul Greenwood <paul.greenwood@ibm.com> Reviewed-by: Zane C Shelley <zshelle@us.ibm.com>
* Improper element erase during loop causing post-restore being skipped on nvdimmTsung Yeung2019-11-111-18/+21
| | | | | | | | | | | | | Change-Id: Iec89586a9ac019960a11b2c4924046c17cec3b06 CQ:SW474826 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/86265 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>
* Add 100ms wait to disable write protection for BPMMatthew Raybuck2019-11-111-0/+2
| | | | | | | | | | | | | | | | | | There was an issue where without waiting after unlocking the BPM registers we'd fail out of the BPM update. This commit adds a 100ms delay after writing the unlock sequence before reading to verify that the registers were unlocked. Change-Id: Ic8cfeb8dba181d4071bb2d4dccaf367d88840cc2 CQ:SW480382 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/86707 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: TSUNG K YEUNG <tyeung@us.ibm.com> Reviewed-by: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Move HBRT reserved memory off of NVDIMMsDan Crowell2019-11-082-1/+178
| | | | | | | | | | | | | | | | | | On a NVDIMM system we need to make sure that we don't use the NV memory for the HOMER (or other reserved memory). Depending on the specific memory layout the NV memory could be placed at the top of memory where we would normally land. Change-Id: Ie4407c63a4944cc5be13d2573d06fbed09d6a11f CQ: SW470564 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/86480 Reviewed-by: Ilya Smirnov <ismirno@us.ibm.com> Reviewed-by: Corey V Swenson <cswenson@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: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: William G Hoffa <wghoffa@us.ibm.com>
* Always position HOMER in HV space for istep 16Dean Sanner2019-11-082-74/+93
| | | | | | | | | | | | | | | | | | | | | | | | P9N/P9C always inits to DD2.x mode for hostboot which doesn't support SMF. This means that the HCODE non secure transition to HCODE secure won't work. Because of that, we now force istep 16 to always place HOMER in non SMF memory (in UV mode if SMF active) and transition into UV there. Then in istep 21 the HOMER is rebuilt correctly for SMF. Note that all the changes are in istep 15 as istep 21 uses the ATTR setup there. Change-Id: I566903dc373667a410ddcc0a1d2cc91bcd94b08d Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/81202 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: Dean Sanner <dsanner@us.ibm.com> Reviewed-by: Christian R Geddes <crgeddes@us.ibm.com> Reviewed-by: William G Hoffa <wghoffa@us.ibm.com>
* Modify INTRP Initialization Order to route LSI interrupts over PSHIHBBill Hoffa2019-11-081-7/+7
| | | | | | | | | | | | | | | | | - Currently the code sets up routing over PSIHIB and then a second function call overwrites that setting which forces things back on the local wire Change-Id: I7d3b44d48c66f994e8a8758eb75568038492fdaf CQ: SW464174 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/86669 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: Ilya Smirnov <ismirno@us.ibm.com> Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Fixes to PMIC presence detectionChristian Geddes2019-11-073-20/+70
| | | | | | | | | | | | | | | | | | | | | While performing presence detection on the PMIC targets we need to first read the parent OCMB's SPD to see what device address the PMIC is on. There was a bug where we were attempting to read the parent OCMB's spd without first checking if the OCMB is present itself. This commit adds a check to ensure we dont attempt i2c reads on devices that are not present. Also this commit adds a check to make sure we do not attempt presence detection on GEMINI ocmbs Change-Id: I999189b3b97210bb37b7ba1fdb2d86658d770e36 CQ: SW480414 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/86564 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>
* Avoid commiting errors inside fapi2_i2c device driverChristian Geddes2019-11-071-13/+34
| | | | | | | | | | | | | | | | | | | | | | | | There are instances where we expect NACK errors returned from i2c requests, for example when waiting for the ocmbs to declare themselves "ready". In these cases we do not want to flood the error log with expected I2C nack errors. To avoid this, instead of committing retriable errors inside of the fapi2_i2c device driver when the MAX RETRY ATTEMPTS is reached we will instead just add traces to the err that gets returned to the caller that explain previous retryable errs occured. In the case where we hit retryable errors and eventually hit and non-retryable error the retryable error is committed in the driver and the non-retryable error is returned to the caller. Also this commit adds some timings to the error log so we can tell how long our attempts took. Change-Id: I9a5ee0bfe088444aa7fecd974f61514c40b320a6 CQ: SW480155 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/86419 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: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Fix fapi spd testcasesDan Crowell2019-11-062-170/+123
| | | | | | | | | | | | | | | Fixed a few places where the Axone path was getting skipped incorrectly, then fixed the resulting latent errors. Change-Id: I917b9a0b6f4ff1491ff384dc924e29f688548873 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85688 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: Zachary Clark <zach@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: William G Hoffa <wghoffa@us.ibm.com>
* Remove excessive NVDIMM tracing from normal pathMatt Derksen2019-11-061-20/+52
| | | | | | | | | | | | | | | | | | Tracing is causing a bottleneck when running NVDIMM updates. Should only trace the real failure and add what is needed for debug into the error log. Change-Id: Ifa75f85e1129efd9642a3fef0b842fc1f60cd182 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/75667 Tested-by: Jenkins Server <pfd-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: Christian R. Geddes <crgeddes@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85499 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>
OpenPOWER on IntegriCloud