summaryrefslogtreecommitdiffstats
path: root/src/usr/targeting
diff options
context:
space:
mode:
authorChen Du <duchen@us.ibm.com>2019-10-15 17:42:01 -0500
committerDaniel M Crowell <dcrowell@us.ibm.com>2019-10-16 08:28:42 -0500
commite783d1e865556868147bdec5c36274d72003d3fc (patch)
treec5f8e39f6038114a03be022c6c562b81b78ad3e4 /src/usr/targeting
parented40af7bc74e8790871691591660d65787a816ec (diff)
downloadtalos-hostboot-e783d1e865556868147bdec5c36274d72003d3fc.tar.gz
talos-hostboot-e783d1e865556868147bdec5c36274d72003d3fc.zip
Dynamically generate ocmb cmd/rsp seq id
The current procedure code is using the sequence id portion of the FW message to indicate the type of the command rather than the command number. This makes error handling very difficult because we can't detect which ocmb we are accessing when we detect a failure. Create a function backed ekb attribute to hold the sequence id so that every time we retrieve this attribute, we increment the valuwe on the next get. Change-Id: I93e0e7062d01ddf7ba6a69f89411a5b6bcf01471 RTC: 210371 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/78767 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>
Diffstat (limited to 'src/usr/targeting')
-rwxr-xr-xsrc/usr/targeting/common/xmltohb/attribute_types_hb.xml14
-rw-r--r--src/usr/targeting/common/xmltohb/target_types_hb.xml3
2 files changed, 17 insertions, 0 deletions
diff --git a/src/usr/targeting/common/xmltohb/attribute_types_hb.xml b/src/usr/targeting/common/xmltohb/attribute_types_hb.xml
index 730d251f6..4910ca681 100755
--- a/src/usr/targeting/common/xmltohb/attribute_types_hb.xml
+++ b/src/usr/targeting/common/xmltohb/attribute_types_hb.xml
@@ -1048,6 +1048,20 @@
</attribute>
<attribute>
+ <id>OCMB_COUNTER_HB</id>
+ <description>
+ Tracks the sequence id for OCMB command transactions.
+ The platform is expected to guarantee a unique value on each read.
+ </description>
+ <simpleType>
+ <uint32_t/>
+ </simpleType>
+ <persistency>volatile-zeroed</persistency>
+ <readable/>
+ <writeable/>
+ </attribute>
+
+ <attribute>
<description>
While in Secureboot, this value is set to 1 the first time attribute
override is attempted and error logged.
diff --git a/src/usr/targeting/common/xmltohb/target_types_hb.xml b/src/usr/targeting/common/xmltohb/target_types_hb.xml
index abc2bef2b..03ae7a01d 100644
--- a/src/usr/targeting/common/xmltohb/target_types_hb.xml
+++ b/src/usr/targeting/common/xmltohb/target_types_hb.xml
@@ -97,6 +97,9 @@
<id>MMIO_VM_ADDR</id>
</attribute>
<attribute>
+ <id>OCMB_COUNTER_HB</id>
+ </attribute>
+ <attribute>
<id>VPD_SWITCHES</id>
</attribute>
</targetTypeExtension>
OpenPOWER on IntegriCloud