diff options
author | Stephen Cprek <smcprek@us.ibm.com> | 2017-08-21 16:54:52 -0500 |
---|---|---|
committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2017-08-29 10:18:58 -0400 |
commit | cd1d9e3aa3aca78d6b9d3bfce3f944a7be3918d0 (patch) | |
tree | aa5adf9ba9dd75fed7373e1a95858251233ea98a /src/usr/runtime/hdatstructs.H | |
parent | 5ee8711939091cb92bc7637383eb31605b8195d5 (diff) | |
download | talos-hostboot-cd1d9e3aa3aca78d6b9d3bfce3f944a7be3918d0.tar.gz talos-hostboot-cd1d9e3aa3aca78d6b9d3bfce3f944a7be3918d0.zip |
Add HDAT Permissions type to Hostboot Reserved Memory Sections
Additionally pre-verified images are now marked read only
Change-Id: Ieba7a4d3e787d155bb3217d52fd9f7cd6ea3dae2
RTC:178470
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/44936
Reviewed-by: Nicholas E. Bofferding <bofferdn@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: Michael Baiocchi <mbaiocch@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>
Diffstat (limited to 'src/usr/runtime/hdatstructs.H')
-rw-r--r-- | src/usr/runtime/hdatstructs.H | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/src/usr/runtime/hdatstructs.H b/src/usr/runtime/hdatstructs.H index 6692275e2..1ea8319bd 100644 --- a/src/usr/runtime/hdatstructs.H +++ b/src/usr/runtime/hdatstructs.H @@ -344,7 +344,9 @@ struct hdatMsVpdRhbAddrRange_t hdatMsVpdRhbAddrRange_t(): hdatRhbRngType(HDAT::RHB_TYPE_INVALID), hdatRhbRngId(0), hdatRhbAddrRngStrAddr(0), hdatRhbAddrRngEndAddr(0), hdatRhbLabelSize(1), - hdatRhbLabelString{}, reserved{} {} + hdatRhbLabelString{}, + hdatRhbPermission(HDAT::RHB_READ_WRITE), + reserved{} {} /** * Set all member variables of class @@ -353,11 +355,13 @@ struct hdatMsVpdRhbAddrRange_t * @param[in] i_startAddr, Range Starting Address * @param[in] i_size, Size of memory region * @param[in] i_label, Label String Ptr + * @param[in] i_permission, R/W permissions for RHB. * @note size of i_label is calculated in function versus a input parameter. */ void set(const HDAT::hdatMsVpdRhbAddrRangeType i_type, const uint16_t i_rangeId, const uint64_t i_startAddr, - const uint64_t i_endAddr, const char* i_label); + const uint64_t i_endAddr, const char* i_label, + const HDAT::hdatRhbPermType i_permission = HDAT::RHB_READ_WRITE); HDAT::hdatMsVpdRhbAddrRangeType hdatRhbRngType; // 0x0000 Range type uint8_t hdatRhbRngRes; // 0x0001 Reserved @@ -366,7 +370,8 @@ struct hdatMsVpdRhbAddrRange_t uint64_t hdatRhbAddrRngEndAddr; // 0x000C Range ending uint32_t hdatRhbLabelSize; // 0x0014 Label size uint8_t hdatRhbLabelString[64]; // 0x0018 Label string Ptr - uint8_t reserved[8]; // 0x0058 Reserved + HDAT::hdatRhbPermType hdatRhbPermission; // 0x0058 R/W Permissions + uint8_t reserved[7]; // 0x0059 Reserved } __attribute__ ((packed)); #endif |