blackbird-hostboot/src/include/runtime, branch 07-25-2019

Add support for NVDIMM secure erase verify

2020-01-17T19:45:37+00:00

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 Tested-by: Jenkins Server Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW Tested-by: FSP CI Jenkins Reviewed-by: Roland Veloz Reviewed-by: Daniel M Crowell

Always send NV_STATUS to Opal when opal-prd starts

2019-12-12T21:13:53+00:00

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 Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW Tested-by: FSP CI Jenkins Reviewed-by: Roland Veloz Reviewed-by: Matt Derksen Reviewed-by: Daniel M Crowell

Revert "Remove ibm prefix from reserved memory spaces"

2019-10-23T13:10:22+00:00

Turns out OPAL wasn't quite ready... This reverts commit e9c4e6ad68f9ac55345180bd77a7dd97b364c384. Change-Id: Ie826083788af599cda5eae6506ac0c95a591a7f4 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85695 Reviewed-by: Roland Veloz Reviewed-by: Matt Derksen Tested-by: Jenkins Server Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW Tested-by: FSP CI Jenkins Reviewed-by: Daniel M Crowell

Remove ibm prefix from reserved memory spaces

2019-10-17T16:22:02+00:00

Removing the unnecessary "ibm," prefix from the HBRT reserved memory spaces. Change-Id: Ib044e9dd719061e32b7332c50b08d6f7de320cf3 RTC: 198923 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/83852 Reviewed-by: Corey V Swenson Reviewed-by: Matt Derksen Reviewed-by: Roland Veloz 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

Added code to support doing an NVM health check

2019-09-03T14:49:31+00:00

Added a method that will do an NVM (non-volatile memory) health check. In particular this method will check the flash error counts and does a predictive callout if the number of flash error counts exceeds the maximum allowed. This method also checks the bad block percentage and does a predictive callout if the number of bad block percentage exceeds the maximum allowed. A predictive callout is done if either or both fail the check. Added support in the runtime commands to make the nvm health check call as well. Also, when I did the ES (energy source) health check method, I was not very explicit in the method that it was doing an ES health check. So I updated the verbiage in the nvDimmCheckHealthStatus to add ES wherever appropriate as to make these two method explicit as to what health check is being performed. Change-Id: Ib9925fd2bb8430cf2121108329247d96072beb1b CQ: 473220 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82843 Tested-by: Jenkins Server Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW Tested-by: FSP CI Jenkins Reviewed-by: Glenn Miles Reviewed-by: Zachary Clark Reviewed-by: Christian R Geddes

Implement an HBRT interface to log a gard event from PHYP/OPAL

2019-08-28T21:50:27+00:00

Added a firmware notify interface, gard_event_t, to accept a gard event message, from PHYP/OPAL, to log that event. Change-Id: I9bcf684f0850c9a07ab7d46635aa07a2c1e9917c RTC: 210201 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82199 Tested-by: Jenkins Server Reviewed-by: Matt Derksen Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW Tested-by: FSP CI Jenkins Reviewed-by: Christian R Geddes Reviewed-by: Daniel M Crowell

Added support for checking on the health status of an NVDIMM

2019-07-31T21:13:03+00:00

* The check for the health status piggy backs off the current NVDIMM operation interface. * Added the method doNvDimmCheckHealthStatus to perform the health check status of the individual NVDIMMs. * Added wrapper methods nvDimmCheckHealthStatusOnSystem and nvdimmCheckHealthStatus that call the doNvDimmCheckHealthStatus * Added an interface in the runtime commands to call make the check health status call. Change-Id: Iefa1fcf5cb6a13c496fd776cdc34ade58ae0612b CQ:SW469962 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/80589 Tested-by: Jenkins Server Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW Tested-by: FSP CI Jenkins Reviewed-by: Corey V Swenson Reviewed-by: Matt Derksen Reviewed-by: Daniel M Crowell

Unit test for the firmware request sendAttributes

2019-07-02T18:44:57+00:00

Added a unit test for the firmware request sendAttributes. The unit test exercises and verifies the correctness of the sendAttribute firmware request method. Did some massive cleanup of the testing framework/harness. It was becoming unwieldy and cumbersome to update. I create methods for the individual firmware request test instead of having the test reside in over bloated method. Added a section, in the README file, on how to test new interfaces that are added for firmware request lineup. Change-Id: I0ff7c0e48211b12d81be1f179ad3ee08e4d254af RTC: 208343 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/78934 Tested-by: Jenkins Server Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW Tested-by: FSP CI Jenkins Reviewed-by: Glenn Miles Reviewed-by: Christian R. Geddes Reviewed-by: Daniel M. Crowell

Push ATTR_NVDIMM_ARMED down to FSP to handle MPIPL case

2019-07-01T22:07:18+00:00

If the NVDIMM is not armed when the save/flush call is made during a mpipl reboot, it will cause errors in the memory. To avoid this we need HBRT to push down the armed state to the FSP so that HWSV can skip the save call in the case where the NVDIMM is not armed. Change-Id: I16611e8fcb5274c3cb0aee6fec1300a95579b11b Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/79825 Tested-by: Jenkins Server Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW Tested-by: FSP CI Jenkins Reviewed-by: Daniel M. Crowell

Changed the generic message type from MSG_DECONFIG_TARGET to MSG_ATTR_WRITE_OP

2019-06-14T16:43:45+00:00

Was using the wrong type. MSG_ATTR_WRITE_OP is the correct type. Also dropped the assertion. From what I understand it was a nice to have not a necessity. Dropped because it was prohibiting the unit from running. Change-Id: I7127b3a316092b2dd155dda0a6aa3ccfdac8fa0a RTC:208343 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/78918 Tested-by: Jenkins Server Tested-by: FSP CI Jenkins Tested-by: Jenkins OP Build CI Reviewed-by: Daniel M. Crowell