diff options
| author | Dan Crowell <dcrowell@us.ibm.com> | 2017-06-08 21:44:44 -0500 |
|---|---|---|
| committer | Sachin Gupta <sgupta2m@in.ibm.com> | 2017-06-13 00:12:33 -0400 |
| commit | 69184204d4d86b53f795acc25867c03f4885bf3b (patch) | |
| tree | 71f3d3699a8fd237ea79bab4f62dd39dd85dabcc /src/import/chips/p9/procedures/hwp/nest/p9_sbe_hb_structures.H | |
| parent | de7270c25782072676995c26a262cf5ef0b8762a (diff) | |
| download | talos-sbe-69184204d4d86b53f795acc25867c03f4885bf3b.tar.gz talos-sbe-69184204d4d86b53f795acc25867c03f4885bf3b.zip | |
Fix alignment issues in SBE-HB structure
Add padding to keep all fields aligned to 8-bytes
Add explicit attribute to ensure structure is packed
Change-Id: I55ca10034d7adf3e766edb4d0071f649c7c90446
CQ: SW391259
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/41584
Reviewed-by: Martin Gloff <mgloff@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: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Thi N. Tran <thi@us.ibm.com>
Reviewed-by: Sachin Gupta <sgupta2m@in.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/41613
Reviewed-by: Hostboot Team <hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Diffstat (limited to 'src/import/chips/p9/procedures/hwp/nest/p9_sbe_hb_structures.H')
| -rw-r--r-- | src/import/chips/p9/procedures/hwp/nest/p9_sbe_hb_structures.H | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/src/import/chips/p9/procedures/hwp/nest/p9_sbe_hb_structures.H b/src/import/chips/p9/procedures/hwp/nest/p9_sbe_hb_structures.H index 38ce24c1..d82448e9 100644 --- a/src/import/chips/p9/procedures/hwp/nest/p9_sbe_hb_structures.H +++ b/src/import/chips/p9/procedures/hwp/nest/p9_sbe_hb_structures.H @@ -74,6 +74,8 @@ union BootloaderSecureSettings }; // Structure starts at the bootloader zero address +// Note - this structure must remain 64-bit aligned to +// maintain compatibility with Hostboot struct BootloaderConfigData_t { uint32_t version; // bytes 4:7 Version field so we know if there is new data being added @@ -82,8 +84,10 @@ struct BootloaderConfigData_t uint16_t pnorSizeMB; // bytes 10:11 Size of PNOR in MB [ATTR_PNOR_SIZE] uint64_t blLoadSize; // bytes 12:19 Size of Load (Exception vectors and Bootloader) BootloaderSecureSettings secureSettings ; // byte 20 - uint64_t xscomBAR; // bytes 21:28 XSCOM MMIO BAR - uint64_t lpcBAR; // bytes 29:36 LPC MMIO BAR -}; + uint8_t reserved[7]; // bytes 21:27 Reserved space to maintain 64-bit alignment + uint64_t xscomBAR; // bytes 28:35 XSCOM MMIO BAR + uint64_t lpcBAR; // bytes 36:43 LPC MMIO BAR +}; // Note: Want to use '__attribute__((packed))' but compiler won't let us + #endif |
