| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
| |
Change-Id: I913dce08862c05aedd21ea1e00bc02a147786536
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/39634
Reviewed-by: Joseph J. McGill <jmcgill@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: Claus M. Olsen <cmolsen@us.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
RTC172494 - ring_apply currently allocates for 13 CME chiplets
when the CME(s) only handle one chipletType, namely the EC core.
** This will NOT break the lab **
Change-Id: I3ead5727d0306438cd69f0919d7f9ad0cc39a245
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/39130
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com>
Reviewed-by: Sumit Kumar <sumit_kumar@in.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
HW-ImageBuild-Prereq: e7236c22506612c52f0f7812dd8317ff4108ba33
Change-Id: I713702bfea372c085c9562f2c0b230ada6cf43ed
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/38729
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: Joseph J. McGill <jmcgill@us.ibm.com>
Reviewed-by: Claus M. Olsen <cmolsen@us.ibm.com>
Reviewed-by: AMIT KUMAR <akumar3@us.ibm.com>
Reviewed-by: Gregory S. Still <stillgs@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Adds support for eq_inex ring buckets in hardware image.
commit intends to avoid co-req between hardware image and
hcode image build.
Change-Id: I732032d02ae1ffdc6614233020e1ca3286897bba
RTC: 165533
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/38138
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: RANGANATHPRASAD G. BRAHMASAMUDRA <prasadbgr@in.ibm.com>
Reviewed-by: AMIT KUMAR <akumar3@us.ibm.com>
Reviewed-by: Claus M. Olsen <cmolsen@us.ibm.com>
Reviewed-by: Joseph J. McGill <jmcgill@us.ibm.com>
Reviewed-by: Gregory S. Still <stillgs@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
RTC172496 - ring_apply currently allocates for two SGPE chiplets
when there only is one. So reducing allocation to one Quad chiplet.
** This will not break the lab. **
Change-Id: I5ef1a7922da3e3ee2633f8930c9ce7d472140d1f
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/39065
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: Sumit Kumar <sumit_kumar@in.ibm.com>
Reviewed-by: Martin Peschke <mpeschke@de.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- ex_l2_fure_1 and ex_l3_fure_1 rings have been removed
Change-Id: Ia9fd9bce71b24a8512338c26020d60ef38bbf96c
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/37720
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: Prem Shanker Jha <premjha2@in.ibm.com>
Reviewed-by: RANGANATHPRASAD G. BRAHMASAMUDRA <prasadbgr@in.ibm.com>
Reviewed-by: Claus M. Olsen <cmolsen@us.ibm.com>
Reviewed-by: Martin Peschke <mpeschke@de.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
| |
and related structures SbeTorId and CmeTorId.
Change-Id: I4bde34a633a8db2468b32fde8ea7432086b27b0c
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/36781
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: Martin Peschke <mpeschke@de.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
| |
Change-Id: Ic6fc899956e0690f75224471917ff904aa03713e
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/37768
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: RANGANATHPRASAD G. BRAHMASAMUDRA <prasadbgr@in.ibm.com>
Reviewed-by: Benjamin Gass <bgass@us.ibm.com>
Reviewed-by: Joseph J. McGill <jmcgill@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Replace some redundant code by calling a common helper function.
Change-Id: I702cc67e06f71162c164fb82ab3335c6cb23fd7f
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/32650
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: Claus M. Olsen <cmolsen@us.ibm.com>
Reviewed-by: Sumit Kumar <sumit_kumar@in.ibm.com>
Reviewed-by: Martin Peschke <mpeschke@de.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Now CHIPLET_TYPE reflects the index of chiplet slots in TOR.
This changes allows to stream-line some code, mostly TOR related.
This change is transparent for users of CHIPLET_TYPE, since its only
intended use is to distingiush chiplet types.
Change-Id: I599945741fe3ee63c8a9d2e7ea611438f01f1675
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/36556
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: Claus M. Olsen <cmolsen@us.ibm.com>
Reviewed-by: Sumit Kumar <sumit_kumar@in.ibm.com>
Reviewed-by: Martin Peschke <mpeschke@de.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
There are 4 padding bytes needed for an 8 byte alignment at the end
of the TOR slot array for PERV common rings, which sometimes
exhibited non-zero content. These could have been mistaken as
valid offsets pointing to rings that didn't exist in the image,
if a users assumed a 16th PERV common ring, which might be added
in the future.
The original code used an array and a loop - which didn't account
for padding bytes - to zero all slots. A following memcpy() - which
accounted for padding bytes - copied bytes beyound the array boundary.
This code was a total mess anyway, and hence it is replaced by single
memset(), which uses a corrected size value.
This problem was only found in the function used for ring slots.
Nevertheless, similar code has been cleaned up for two other
use cases.
Change-Id: I1c81518ff81a6af735240090cdbc7735f60f3edb
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/37287
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: Claus M. Olsen <cmolsen@us.ibm.com>
Reviewed-by: Martin Peschke <mpeschke@de.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
| |
FSM control (i.e., set and reset latches) during scanning.
Change-Id: Ic0f323ae08d0d9c31d5cee37a61fb476eaf3f2d9
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/35178
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: RANGANATHPRASAD G. BRAHMASAMUDRA <prasadbgr@in.ibm.com>
Reviewed-by: Martin Peschke <mpeschke@de.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Commit 33317 removed a couple of the lines that need to be kept.
Impacts CME and SGPE.
Change-Id: I5fccae529051c9ccfcff9042b67d238f8f5a94d3
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/36801
Dev-Ready: Brent Wieman <bwieman@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com>
Reviewed-by: Claus M. Olsen <cmolsen@us.ibm.com>
Reviewed-by: Brent Wieman <bwieman@us.ibm.com>
Reviewed-by: Martin Peschke <mpeschke@de.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The p9_tor code used htobe* in many places where it should have used
be*toh, because the code reads values from the big-endian TOR image
to host memory. This misuse should not show during execution, because
these functions are equivalent. But the code was misleading and raising
the wrong questions (How can we ever use values converted from host
endian to big-endian in calculations and as sizes or offsets
in memcpy operation?)
Change-Id: I2117a22e3e0765dff44279e2bb291e13811b2f15
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/33317
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com>
Reviewed-by: Martin Peschke <mpeschke@de.ibm.com>
|
|
|
|
|
|
|
|
|
|
| |
Change-Id: Ib3b7aeb8045a28ca05d33370c1ef1dcb9ba1fb64
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/29448
Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Martin Gloff <mgloff@us.ibm.com>
Reviewed-by: Michael C. Sgro <mcs793@us.ibm.com>
Reviewed-by: Martin Peschke <mpeschke@de.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
These messages used to clutter the p9_xip_tool ring dissect output.
I am reclassifying them so that they require a higher debug level.
Change-Id: I19ef206068f0750d625c8cbbe6e5bb019c52d37d
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/36421
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: Claus M. Olsen <cmolsen@us.ibm.com>
Reviewed-by: Sumit Kumar <sumit_kumar@in.ibm.com>
Reviewed-by: Martin Peschke <mpeschke@de.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
-Add option to p9_xip_tool to verify the size of the
DD specific .rings section intended for the SBE image
-Update makefiles to correctly run p9_ring_apply to insert
and extract the DD level specific rings.
-Update tor_access_ring to correctly advance the pointer
to the next DD level in the HW image so the size can
be extracted correctlty.
Change-Id: I21a53b5d11341a14a4e9cc88388c9c2050394095
RTC:165558
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/33873
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: Joseph J. McGill <jmcgill@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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
that doesn't exist in TOR. (This dbg msg was recently added to the
RS4v3 commit 33993 and is causing confusion, so we're putting it
under local debug control.)
Change-Id: I1b00f5b19951134b3d3517db59639d39e5c257a6
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/35419
Reviewed-by: Kahn C. Evans <kahnevan@us.ibm.com>
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
These changes avoid some waste of valuable memory used to store
TOR and rings contained in TOR. This is mostly needed as a SEEPROM relief.
Contains:
- 12 byte ring header (former 31551 commit)
- less zero padding at the end of compressed ring (former 31524 commit)
- removed an additional ring meta data structure (former 31524 commit)
In addition, it contains these fixes:
- fixed a bunch of scan addresses (former 33969 commit)
- increased size of ring decompression buffer (former 32796 commit)
- zero byte ring padding up to 4 byte boundaries for SBE (former 33969
commit)
- added dbgl control of error trace after call to get_ring_from_sbe_image
- added enumerated ringId to the RS4v3 header
- fixed incorrect scan address masks for even/odd EX RS4 v3 ring handling
in MVPD accessor functions
- fixed incorrect scan address region bits for odd EX rings translated from
RS4 v2 to RS4 v3 in MVPD accessor functions
Change-Id: I8fd00760e6ac2b3760994d1ca819fffbf35188ca
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/33993
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com>
Reviewed-by: Prachi Gupta <pragupta@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Revert "TOR reduction: Ditching DeltaRingLayout and RingLayout_t"
This reverts commit 0ea8cddae37eb5186571e6c66469dad694de9c5a.
Revert "Shrinking RS4 header"
This reverts commit c1a7c86e800e8bb38f6549b443bb4801feb4f1e3.
Change-Id: I7832e23491ea34f86db3cbdc5d25c98e3a9f1821
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/33985
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com>
Reviewed-by: Prachi Gupta <pragupta@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- 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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Implements a new 12 byte CompressedScanData structure with less
and refined members in order to save space for rings.
Used test tool to verify correctness of p9_scan_compression changes.
Used p9_xip_tool dissect to verify .rings of HW Image with new RS4
header.
Change-Id: I68b3044b62908d013778f87343798ebaa6829923
RTC:158078
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/31551
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: Claus M. Olsen <cmolsen@us.ibm.com>
Reviewed-by: Prem Shanker Jha <premjha2@in.ibm.com>
Reviewed-by: Prachi Gupta <pragupta@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
MikeO:
- Updated the fix in anticipation of 31524+31551 merge conflicts
and changed variable name accordingly from ringTorofset to
acc_offset.
- Also updated the corresponding CME and SGPE functions.
Change-Id: I8c9f3bc3ff07a9283771d7ea71ce30156b914acb
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/33662
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: Martin Gloff <mgloff@us.ibm.com>
Reviewed-by: Richard J. Knight <rjknight@us.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
1) Now calculating correct value of MAX_SEEPROM_IMAGE_SIZE.
2) Now returning error RC!=0 whenever image size exceeds Seeprom space.
3) Now allowing for temporarily exceeding max image size during
customization of Vpd rings since we know with certain that the
image will loose three sections (toc, fixed_toc and strings) as
the very last step in xip_customize.
4) Now extracting platform dependent value of MAX_SBE_SEEPROM_SIZE
to allow Cronus to use larger image size than can fit into SBE
Seeprom.
5) Updated wrapper as well to illustrate the new rule how to call
xip_customize, namely by supplying buffer sizes greater than
MAX_SEEPROM_IMAGE_SIZE, e.g. using MAX_SBE_SEEPROM_SIZE instead.
Please see xip_customize.H for new API rules.
Change-Id: Ia87fc285378ec15ab53d171757fde33b5a7f3399
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/33359
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com>
Reviewed-by: Sachin Gupta <sgupta2m@in.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Kahn C. Evans <kahnevan@us.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
| |
Change-Id: Iaa6c1d042baa5449c4c7d572965f327a3866589a
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/32531
Reviewed-by: Sachin Gupta <sgupta2m@in.ibm.com>
Reviewed-by: Martin Peschke <mpeschke@de.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com>
Reviewed-by: Claus M. Olsen <cmolsen@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Prachi Gupta <pragupta@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This commit simply adds the rings at the very end of the RingID
list in p9_ring_id.h in an attempt to not interfere with the
precompiled built-in version of this header file in Cronus.
Change-Id: I3f7b5db5835688adddf060910a2af88bb9b477ba
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/32272
Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Richard J. Knight <rjknight@us.ibm.com>
Reviewed-by: Martin Peschke <mpeschke@de.ibm.com>
|
|
|
|
|
|
|
|
|
|
| |
Change-Id: Id13f51cde14a4f0f7d59c1bb09d26d8d65342b05
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/32074
Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: Kahn C. Evans <kahnevan@us.ibm.com>
Reviewed-by: Martin Peschke <mpeschke@de.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Naming didn't follow the rules and thereby got in my way
when I was searching for something else.
Change-Id: I185b1531a554daf6cc8dbffb46e42cb17f75677d
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/31773
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com>
Reviewed-by: Richard J. Knight <rjknight@us.ibm.com>
Reviewed-by: Claus M. Olsen <cmolsen@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Martin Peschke <mpeschke@de.ibm.com>
|
|
|
|
|
|
|
|
|
|
| |
Change-Id: I95b33b5472ac6d18b266b5c3868d0da92e9f0f39
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/32130
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: Martin Peschke <mpeschke@de.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com>
Reviewed-by: Prachi Gupta <pragupta@us.ibm.com>
|
|
|
|
|
|
|
|
|
| |
Change-Id: I44fdbb857ca1fe578e097ce3871b3d65f608a6d9
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/31739
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: Joseph J. McGill <jmcgill@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- Catching all abrupt ending if-elseif endings.
- Updated all API comments in *.C file.
- Making local functions static.
- Accommodated additional six EX rings in tor_get_block_of_rings(CPLT).
- Now limiting max DD levels to 20 to prevent infinite trace out loop.
- Fixing bunch of inconsistent error return code situations.
Change-Id: I516c9554836b32a7717ef626afcba0cf4c03d6bf
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/30798
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Joachim Fenkes <fenkes@de.ibm.com>
Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com>
Reviewed-by: Richard J. Knight <rjknight@us.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
| |
Change-Id: I5701bf15e064144d0106eeb921819424777116dc
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/31355
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: Martin Peschke <mpeschke@de.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Key cleanup items:
- Eliminated ringNameImg in GenRingIdList
- Getting rid of FIXED_RING_BUF_SIZE, but not until accommodation
on HB side to switch to MAX_RING_BUF_SIZE.
Change-Id: If23bd5f190b9f83f43805fc618ffa1f54ea40d44
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/31118
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: Richard J. Knight <rjknight@us.ibm.com>
Reviewed-by: Martin Peschke <mpeschke@de.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
provide number of instance rings with different scan addresses per
chiplet basis. This will only affect EQ rings since each EX ring has
two different rings.
Change-Id: I5d7149e35e420c28e01de7e967d6bb0ec7eee00f
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/31172
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: Prem Shanker Jha <premjha2@in.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
| |
Change-Id: Ia0eddaeb69c25115fe16089ee0734e0d93ecd7dd
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/31109
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com>
Reviewed-by: Richard J. Knight <rjknight@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
requesting a chiplet block of rings.
Change-Id: I36427b9b74e2d34416ab0e50c277c08beec2080f
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/30704
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: Martin Peschke <mpeschke@de.ibm.com>
Reviewed-by: GIRISANKAR PAULRAJ <gpaulraj@in.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
|
|
|
|
|
|
|
|
|
| |
Change-Id: Ib0f01181bfea432a33efd9f3128c07edda24fc87
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/30972
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com>
Reviewed-by: Dean Sanner <dsanner@us.ibm.com>
Reviewed-by: Joseph J. McGill <jmcgill@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
hardcoded numbers.
Also, made some cosmetic changes to p9_ringId.C.
Change-Id: If5ec8074aeaf835137aaf8761de45808713dd252
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/30368
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: Richard J. Knight <rjknight@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: Martin Peschke <mpeschke@de.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Change-Id: I43090f95b6ee2729549d962c1af6827d59b4277c
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/30458
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>
Dev-Ready: Matt K. Light <mklight@us.ibm.com>
Reviewed-by: RANGANATHPRASAD G. BRAHMASAMUDRA <prasadbgr@in.ibm.com>
Reviewed-by: Martin Gloff <mgloff@us.ibm.com>
Reviewed-by: Benjamin Gass <bgass@us.ibm.com>
Reviewed-by: Joseph J. McGill <jmcgill@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Includes TOR API fix for improper BE->LE conversion of ddLevelOffset from
HW image.
Includes cleanup and renaming of local variables around above section
to improve clarity of code.
Change-Id: If37518ed24b3431241a2aa9456e709d91ecea696
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/28935
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Martin Gloff <mgloff@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Reviewed-by: Martin Peschke <mpeschke@de.ibm.com>
|
|
|
|
|
|
|
|
| |
Change-Id: I67153e2e00c2f09b407020fb14039b1fab2cf794
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/29286
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Prachi Gupta <pragupta@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The p9_scan_compression code uses htobe32() and htobe64() calls. It
needs to include the endian.h header file to compile in the Hostboot
environment.
Change-Id: I2a50641e08ac3e04276fa348853169a69784fec5
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/29093
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: Claus M. Olsen <cmolsen@us.ibm.com>
Reviewed-by: Prachi Gupta <pragupta@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Moving MY_INF under cotrol of i_dbgl params
Improved io_ringBlockSize size checking condition
Change-Id: Ieecbd27fe7ea5c4d7422a16764981e612025e042
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/28891
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Kahn C. Evans <kahnevan@us.ibm.com>
Reviewed-by: Claus M. Olsen <cmolsen@us.ibm.com>
Reviewed-by: GIRISANKAR PAULRAJ <gpaulraj@in.ibm.com>
Reviewed-by: Prachi Gupta <pragupta@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
eq_ana_bndy_bucket_26 ring name is not correct. there is typo. new
name is eq_ana_bndy_l3dcc_bucket_26.
Change-Id: I8a223b7241c155900bdeb478a1e8578eb2f0373f
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/28780
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Joseph J. McGill <jmcgill@us.ibm.com>
Reviewed-by: GIRISANKAR PAULRAJ <gpaulraj@in.ibm.com>
Reviewed-by: PARVATHI RACHAKONDA <prachako@in.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
get_ring_from_sgpe_image and get_ring_from_cme_image function
support only HW_image get ring. add condition for support CME and
SGPE ppe type get ring functionality. Now, Extracted CME and SGPE
ppe ring passed through tor_access_ring. So,User can extract
single ring container from CME and SGPE ppe type rings block.
Change-Id: I198976deac00e4d0237be14933dded1c35131931
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/28299
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: Prachi Gupta <pragupta@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
xip_customize supports the following:
- Updates mailbox attributes in Seeprom image
- Appends Mvpd #G and #R rings (but excl #G GPTR) into .rings
ring section.
- Shuffles around .hbbl, .overrides and .rings so that .rings is the
last section in the Seeprom image as otherwise we can't reappend
.rings section after updating it (since it wasn't last to begin
with).
- Honors of the bootCoreMask to append Vpd rings only for EC/EX/EQs
associated with the ECs as indicated in the bootCoreMask.
- Removes .toc, .fixedtoc and .strings in the Seeprom image.
- Captures FFDC (100% complete).
- Detects potential image overflow condition.
Change-Id: I68bd2f39e3c407ea196f2228bdd31472a00a4c9e
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/26150
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
declared tor_append_ring API.
Added new image magic parameter for cme and
spge tor_get_ring function for supporting tor_append_ring API.
Added new return code for tor_append_ring.
Added code for tor_append_ring function.
TOR starting address. Earlier it was referred from starting ring
section starting address
Change-Id: Ia07dc52dc3fe9e8d2c9acc67c7822357e559f45c
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/25454
Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: GIRISANKAR PAULRAJ <gpaulraj@in.ibm.com>
Reviewed-by: Claus M. Olsen <cmolsen@us.ibm.com>
Reviewed-by: Prachi Gupta <pragupta@us.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
p9_tor builds on p9_ringId, not the other way around.
Hence p9_ringId should not include p9_tor.
Resolving by moving a definition and fixing some includes.
Change-Id: Ibfe4d7c0c28838a836bf6ee5fc704ff515827c72
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/27918
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: RANGANATHPRASAD G. BRAHMASAMUDRA <prasadbgr@in.ibm.com>
Reviewed-by: Claus M. Olsen <cmolsen@us.ibm.com>
Reviewed-by: GIRISANKAR PAULRAJ <gpaulraj@in.ibm.com>
Reviewed-by: Martin Peschke <mpeschke@de.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
There is whole bunch of extern declarations in p9_ring_identification.H
which refer to stuff that is actually declared in p9_ringId.C.
So the right place for these extern declaration is p9_ringId.H.
Moving these allows to move a couple of structure definitions,
which belong to p9_ringId as well. Besides an unused structure
definition is removed entirely.
Having these things fall into the right places has enabled
cleaning up a few include statements. This makes sure that other
code doesn't pick up stuff that it doesn't need.
Change-Id: Ic6e8748b8b56653859d1e146ea44e19b7243fd23
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/27909
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: RANGANATHPRASAD G. BRAHMASAMUDRA <prasadbgr@in.ibm.com>
Reviewed-by: Prachi Gupta <pragupta@us.ibm.com>
Reviewed-by: Martin Peschke <mpeschke@de.ibm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The enum RingID is moved from p9_ringId.H to p9_ring_id.h.
Now the latter has minimum content to support manufacturing. Its plain C
content is compatible with wafer test Sun SC4.0 18 Oct 1995 C++ 4.1 .
Besides HWP usually content themselves with p9_ring_id.h.
Change-Id: I2d6cffb33a8c16a1083a02c6c04d4be7078480d2
RTC: 158310
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/27682
Reviewed-by: Claus M. Olsen <cmolsen@us.ibm.com>
Reviewed-by: Joseph E. Dery <dery@us.ibm.com>
Dev-Ready: Joseph E. Dery <dery@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: RANGANATHPRASAD G. BRAHMASAMUDRA <prasadbgr@in.ibm.com>
Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Martin Peschke <mpeschke@de.ibm.com>
|