| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change fixes an issue where the HBRT reserved trace buffer can
be placed in a new location than it's origination.
When HB crashes, the reserved trace buffer is persisted until next IPL.
With OPAL, the reserved trace buffer may be in a different memory
location than where the buffer was when it crashed, therefore all
internal pointers will be invalid. This change will detect the change
in buffer location and realign all internal pointers to be valid
once again. This is not an issue with PHYP, only with OPAL.
Change-Id: I476845550062433fba190294b0bd2bbcf8dad658
RTC: 206137
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82094
Tested-by: Jenkins Server <pfd-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>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Christian R Geddes <crgeddes@us.ibm.com>
Reviewed-by: Glenn Miles <milesg@ibm.com>
Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This fix is to correct an issue with the flattening of error logs.
In particular 3 issues have been addressed in the method
RsvdTraceBuffer::getTraceEntries(void* o_data, uint32_t i_dataSize)
1) The offset to insert the data entries into a returning buffer(o_data)
was incorrect. It was using the size of the data for the first entry
as the offset when it needed to be the size of the
struct trace_buf_head_t because that struct is put first in the
outgoing buffer(o_data) before the data items.
2) The method was returning the actual size of the data when it needed
to be the size of the data in an alignment of 8 using ALIGN_8 method.
3) The method was returning the data in a skewed manner. Basically
it was off by one. Which explains why the last entry was the foul one.
Also added more comments and fixed spelling errors.
Change-Id: Idabf519a36990cb1857d63d43304b6c0b9c04373
CQ: SW460919
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/76075
Tested-by: Jenkins Server <pfd-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: Glenn Miles <milesg@ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
|
|
|
This buffer is designed to be used in a runtime environment.
I am laying the ground work for an over arching story that will
save traces in a buffer for future consumption.
For this commit, I have created the methods necessary to add, remove and find
available space (for an Entry) in a buffer bound region of memory using a
circular, doubly linked list.
Change-Id: I259bb0f6051611a17b7b919bf026919ffdb12eb1
RTC: 191303
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/59575
Tested-by: Jenkins Server <pfd-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: Prachi Gupta <pragupta@us.ibm.com>
Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
|
