diff options
author | Claus Michael Olsen <cmolsen@us.ibm.com> | 2016-10-19 21:20:45 -0500 |
---|---|---|
committer | Christian R. Geddes <crgeddes@us.ibm.com> | 2016-12-19 14:57:26 -0500 |
commit | e592c081ef64501070ea2964f3d953fc8f30eb29 (patch) | |
tree | 9be390cf75dd0d9ba6dfcd07ecdd9d516c97692a /src/import/chips/p9/procedures/hwp/accessors | |
parent | f4dc308e69fcdadeecc4432e93f9eafd0f64dccb (diff) | |
download | talos-hostboot-e592c081ef64501070ea2964f3d953fc8f30eb29.tar.gz talos-hostboot-e592c081ef64501070ea2964f3d953fc8f30eb29.zip |
TOR reduction: Ditching DeltaRingLayout and RingLayout_t
- Complete overhaul of get_ring_from_sbe_image for improved clarity
in effort to remove anything related to 8Byte RingLayout header,
incl built-in and hardcoded assumptions.
- Functional overhaul of get_ring_from_cme/sgpe_image, but only
slight improvement in clarity. Also removed any traces of
RingLayout assumptions. (Beautification of these functions
will be in separate commit.)
- Overhaul of tor_append_ring for clarity and improved clarity
elsewhere where PUT_SINGLE_RING is used.
- Updated p9_tor.C|H in order to expose version 2 of TOR API,
which eliminates the 8B RingLayout structure, in order to
work with Cronus which currently assumes, due to pulling in
the p9_tor.H header file, that RingLayout structure is being
used.
- Updated p9_scan_compression.C to remove most zero padding at end
of compressed ring (i.e., merged commit 32647 into this commit
in order to be able to merge into gerrit).
- The scan compression code used to pad the compressed ring at its
end with zeros up to the next 8 byte boundary. For P9, no
padding is needed. Most of this padding is now removed, just
keeping a padding to the next 2 byte boundary for convenience.
Removal of these padding bytes saves about 400 bytes in SEEPROM.
- Added p9_ipl_build.C because it assumes an 8B alignment of the
SBE ring block which is no longer the case with RS4v3. We've
been lucky so far in our tests.
Change-Id: I08edef8e53302dab765a73dc4d6ca68cd8dce440
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/31524
Reviewed-by: Richard J. Knight <rjknight@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: Prachi Gupta <pragupta@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/32269
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com>
Diffstat (limited to 'src/import/chips/p9/procedures/hwp/accessors')
0 files changed, 0 insertions, 0 deletions