#
ad6c880f
|
| 18-Jun-2025 |
Aushim Nagarkatti <anagarkatti@nvidia.com> |
Support to stringify CPER output
Initial commit to add a "message" property that provides a single line description of some important properties. This makes it easier to parse multiple CPERs in crow
Support to stringify CPER output
Initial commit to add a "message" property that provides a single line description of some important properties. This makes it easier to parse multiple CPERs in crowded logs.
For now, "message" is supported for nvidia, arm processor and memory types. The other types contain generic messages.
Example output:
``` "sections":[ { "message":"A Corrected CCPLEXSCF NVIDIA Error occurred on CPU 0", "Nvidia":{ "signature":"CCPLEXSC",
"sections":[ { "message":"An ARM Processor Error occurred on CPU 0; Error Type(s): {Cache Error at Virtual Addr=0x41D6AA12D528 Physical Addr=0x80003A198DDA10}", "ArmProcessor":{ "errorInfoNum":1,
"sections":[ { "message":"A Multi-bit ECC Memory Error occurred at address 0x0000000080000000 at node 0", ```
Change-Id: I395d0370ec60579b8f7fede825b45a3ced8ff18f Signed-off-by: Aushim Nagarkatti <anagarkatti@nvidia.com>
show more ...
|
#
bd1814de
|
| 31-Mar-2025 |
Khang D Nguyen <khangng@os.amperecomputing.com> |
Ensure FRU text is printable ASCII
Currently, libcper currently fails to compile on my machine (GCC 13):
../cper-utils.c: In function ‘add_untrusted_string’: ../cper-utils.c:467:23: error:
Ensure FRU text is printable ASCII
Currently, libcper currently fails to compile on my machine (GCC 13):
../cper-utils.c: In function ‘add_untrusted_string’: ../cper-utils.c:467:23: error: comparison is always false due to limited range of data type [-Werror=type-limits] 467 | if (c < 0) { | ^
The reason seems to be that char signedness is implementation-defined, we have to explicitly use unsigned char or signed char to get a portable char type. In our case, char is unsigned char, hence the warning.
Apparently we are trying to validate ASCII strings from the records. Those strings seem to be used for display purpose only, so I think replacing it with a more precise printable ASCII test, which also does not care about char signedness, is appropriate here.
This changes the JSON fruText property to appear only with printable ASCII FRU content. As a result, all of the examples have been changed where applicable. Some sections use FRU content with a predefined format (pcie, cxlprotocol) so fruText has been completely removed from those JSON objects like in the case of non-printable ASCII FRU content.
Tested: oompile successfully
Change-Id: I98c7c10a674c8817e0b2cbe82c26f6590d8d716a Signed-off-by: Khang D Nguyen <khangng@os.amperecomputing.com>
show more ...
|
#
a2dce4bc
|
| 05-Mar-2025 |
Ed Tanous <etanous@nvidia.com> |
Convert files to hex
It was pointed out in code review these files would be easier to review diffs on if they were in hex format on disk. This commit converts all the existing files to "cperhex" wh
Convert files to hex
It was pointed out in code review these files would be easier to review diffs on if they were in hex format on disk. This commit converts all the existing files to "cperhex" which is cper in hex hexadecimal format using the command 'xxd -p -l 64'
Change-Id: I5e762ec27a02b3d918b926a966074da8178d73b8 Signed-off-by: Ed Tanous <etanous@nvidia.com>
show more ...
|