diff options
author | Prem Shanker Jha <premjha2@in.ibm.com> | 2018-12-11 05:37:48 -0600 |
---|---|---|
committer | Christian R. Geddes <crgeddes@us.ibm.com> | 2019-02-13 11:51:24 -0600 |
commit | 3de3e3bbc10faed278ef5dcd247a770c87308884 (patch) | |
tree | 24436ec936b143212bb225a7050eb84227ea55f0 /src/import/chips/p9/procedures/xml | |
parent | d4c084086a47ae01448a9870f04dd43baf4455a8 (diff) | |
download | talos-hostboot-3de3e3bbc10faed278ef5dcd247a770c87308884.tar.gz talos-hostboot-3de3e3bbc10faed278ef5dcd247a770c87308884.zip |
SMF: Populates unsecure HOMER with SC2 instruction.
On an SMF enabled system, if HV engages self save, core must
must transition from HV to UV privilege to accomplish this.
So, before attempting self save, core must execute SC2
instruction with HV privilege inorder to be a UV. Commit is a
partial implementaiton of this solution. It populates a
buffer with SC2 instruction which hostboot will copy to an
unsecure memory.
- Updated attribute tags for MRW Hiding
- Removed writing of ATTR_UNSECURE_HOMER_SIZE in deference
to attribute default checking
Key_Cronus_Test=PM_REGRESS
Change-Id: Idb5e219e2a5f9b2274602aedc420f390f34f2609
RTC: 202400
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/69655
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com>
Tested-by: HWSV CI <hwsv-ci+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Tested-by: Cronus HW CI <cronushw-ci+hostboot@us.ibm.com>
Reviewed-by: RANGANATHPRASAD G. BRAHMASAMUDRA <prasadbgr@in.ibm.com>
Reviewed-by: Gregory S. Still <stillgs@us.ibm.com>
Dev-Ready: Gregory S. Still <stillgs@us.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/69662
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com>
Diffstat (limited to 'src/import/chips/p9/procedures/xml')
-rw-r--r-- | src/import/chips/p9/procedures/xml/attribute_info/pm_plat_attributes.xml | 7 | ||||
-rwxr-xr-x | src/import/chips/p9/procedures/xml/error_info/p9_hcode_image_build_errors.xml | 15 |
2 files changed, 18 insertions, 4 deletions
diff --git a/src/import/chips/p9/procedures/xml/attribute_info/pm_plat_attributes.xml b/src/import/chips/p9/procedures/xml/attribute_info/pm_plat_attributes.xml index da5732eb7..12ce171ba 100644 --- a/src/import/chips/p9/procedures/xml/attribute_info/pm_plat_attributes.xml +++ b/src/import/chips/p9/procedures/xml/attribute_info/pm_plat_attributes.xml @@ -2604,7 +2604,7 @@ <attribute> <id>ATTR_UNSECURE_HOMER_SIZE</id> <targetType>TARGET_TYPE_SYSTEM</targetType> - <description>Size of memory which hostboot must copy from a temp buffer to + <description>Size of memory in bytes which hostboot must copy from a temp buffer to an unsecure region of main store memory. Once HOMER is built, hostboot reads the attribute and copies the amount of memory specified in this attribute from a temp buffer to an unsecure region of main memory specifed in an attribute @@ -2612,8 +2612,9 @@ Producer: Initialized by hostboot firmware based on platform support Consumers: Read by p9_hcode_image_build.C </description> - <writeable/> + <platInit/> <valueType>uint32</valueType> - <initToZero/> + <mrwHide/> + <default>1024</default> </attribute> </attributes> diff --git a/src/import/chips/p9/procedures/xml/error_info/p9_hcode_image_build_errors.xml b/src/import/chips/p9/procedures/xml/error_info/p9_hcode_image_build_errors.xml index e6dfaaf33..37f720cdc 100755 --- a/src/import/chips/p9/procedures/xml/error_info/p9_hcode_image_build_errors.xml +++ b/src/import/chips/p9/procedures/xml/error_info/p9_hcode_image_build_errors.xml @@ -5,7 +5,7 @@ <!-- --> <!-- OpenPOWER HostBoot Project --> <!-- --> -<!-- Contributors Listed Below - COPYRIGHT 2016,2018 --> +<!-- Contributors Listed Below - COPYRIGHT 2016,2019 --> <!-- [+] International Business Machines Corp. --> <!-- --> <!-- --> @@ -766,4 +766,17 @@ </callout> </hwpError> <!-- *********************************************************************** --> + <hwpError> + <rc>RC_FAILED_TO_INIT_UNSECURE_HOMER</rc> + <description>Buffer representing unsecure HOMER could not be initialized. It may be due to + insufficient memory size or bad attribute values</description> + <ffdc>UNSECURE_HOMER_MIN_SIZE</ffdc> + <ffdc>UNSECURE_HOMER_ACTUAL_SIZE</ffdc> + <ffdc>TEMP_BUFFER_SIZE</ffdc> + <callout> + <procedure>CODE</procedure> + <priority>HIGH</priority> + </callout> + </hwpError> + <!-- *********************************************************************** --> </hwpErrors> |