cc367011 | 05-Dec-2024 |
Aushim Nagarkatti <anagarkatti@nvidia.com> |
Include hex decode for human readable fields
Hexadecimal decode for some fields like deviceAddress make sense to be represented in hex over decimal to make scripting and human-usability easier.
Cha
Include hex decode for human readable fields
Hexadecimal decode for some fields like deviceAddress make sense to be represented in hex over decimal to make scripting and human-usability easier.
Change-Id: I7d0d100162bc681c3c6885ca01ed23020c3b5063 Signed-off-by: Aushim Nagarkatti <anagarkatti@nvidia.com>
show more ...
|
3cebfc28 | 20-Nov-2024 |
Andrew Adriance <aadriance@nvidia.com> |
Add AER registers to PCIe decoding
Break out AER registers so aerinfo doesn't require manual interpretation
Change-Id: I5e626155270636420a1f6e7c473a2b15bfa7ecf0 Signed-off-by: Andrew Adriance <aadr
Add AER registers to PCIe decoding
Break out AER registers so aerinfo doesn't require manual interpretation
Change-Id: I5e626155270636420a1f6e7c473a2b15bfa7ecf0 Signed-off-by: Andrew Adriance <aadriance@nvidia.com>
show more ...
|
17bc66a0 | 25-Nov-2024 |
Aushim Nagarkatti <anagarkatti@nvidia.com> |
Add affinity3 field to json output
Affinity3 in arm processor errors is used to determine the socket number generating the error. This is critical information that needs to be presented to the end-u
Add affinity3 field to json output
Affinity3 in arm processor errors is used to determine the socket number generating the error. This is critical information that needs to be presented to the end-user
Change-Id: I66c7d29407bb610625087b0f46e18ff868f7da64 Signed-off-by: Aushim Nagarkatti <anagarkatti@nvidia.com>
show more ...
|
e42fb487 | 15-Oct-2024 |
Thu Nguyen <thu@os.amperecomputing.com> |
Change include style to use system includes
The libcper header files in `libcper` are installed to `usr/include/libcper`. Use that system includes in `libcper` source instead of using the project in
Change include style to use system includes
The libcper header files in `libcper` are installed to `usr/include/libcper`. Use that system includes in `libcper` source instead of using the project includes.
Change-Id: I596edc2c754dae4829844f535a1e34caa246fb43 Signed-off-by: Thu Nguyen <thu@os.amperecomputing.com>
show more ...
|
a3b7f8a2 | 04-Nov-2024 |
Ed Tanous <etanous@nvidia.com> |
Unflatten edk includes
Installing the edk headers into a folder requires us to do odd things with imports, and either include headers with "..", or otherwise get the include directories lined up.
M
Unflatten edk includes
Installing the edk headers into a folder requires us to do odd things with imports, and either include headers with "..", or otherwise get the include directories lined up.
Move the contents of edk/*.c/h up a level, and just simplify the include structure. This is done to fix the immediate change of the prior patch and make this build again. Happy to discuss other options.
Change-Id: I328f20bca6d23100993493445bee0e5e11d2866a Signed-off-by: Ed Tanous <etanous@nvidia.com>
show more ...
|
5b793009 | 26-Sep-2024 |
Aushim Nagarkatti <anagarkatti@nvidia.com> |
Fix ArmProcessor ErrorInformation types
ErrorInformation currently has 2 supported formats according to the libcper schema, cacheError and tlbError. Remove "unknown" microarch types as we need corre
Fix ArmProcessor ErrorInformation types
ErrorInformation currently has 2 supported formats according to the libcper schema, cacheError and tlbError. Remove "unknown" microarch types as we need correctly formatted output only. Add a property name to identify which ErrorInformation type has been detected in the cper record
Tested: Used cper-convert to-json to convert a known good cper to json Used cper-convert to-cper to convert same json to cper json was generated again and validated against schema
Change-Id: I510f2ae7fef195721b618065c6ef643ab1191b76 Signed-off-by: Aushim Nagarkatti <anagarkatti@nvidia.com>
show more ...
|
b07061ab | 22-Sep-2024 |
Ed Tanous <etanous@nvidia.com> |
Make key types based on id
This commit separates out the libcper json output by section name. Previously, each section was
i.e., for e.g., add "Nvidia" for the NVIDIA section ``` "sections": [
Make key types based on id
This commit separates out the libcper json output by section name. Previously, each section was
i.e., for e.g., add "Nvidia" for the NVIDIA section ``` "sections": [ { "Nvidia":{ "socket": 0 } } ] ``` instead of ``` "sections": [ { "socket": 0 } ] ```
This allows disambiguating between multiple fields with different types and removes collisions between the field names.
Change-Id: I4e257f1e04fc5fbf2798955d3a5d93214c81f0fc Signed-off-by: Karthik Rajagopalan <krajagopalan@nvidia.com> Signed-off-by: Ed Tanous <etanous@nvidia.com>
show more ...
|
04f57716 | 29-Aug-2024 |
Dung Cao <dung@os.amperecomputing.com> |
Add support for AMPERE CPERs
Support Ampere CPER entries
Change-Id: I607a89209138fa53914c55c07aba8b7d6f382e5e Signed-off-by: Dung Cao <dung@os.amperecomputing.com> |
2d17acec | 27-Aug-2024 |
Ed Tanous <etanous@nvidia.com> |
Improve Nvidia CPER decode
Add decoding of registers to the structure. Note, this requires COUNTED_BY support which is borrowed from LIBPLDM.
Also add unit-tests for NVIDIA section, and update sch
Improve Nvidia CPER decode
Add decoding of registers to the structure. Note, this requires COUNTED_BY support which is borrowed from LIBPLDM.
Also add unit-tests for NVIDIA section, and update schema to match existing register decoding.
Change-Id: If1c9cae97de35ba6a5dad1f462d3989ec6ac6a90 Signed-off-by: Karthik Rajagopalan <krajagopalan@nvidia.com> Signed-off-by: Ed Tanous <etanous@nvidia.com>
show more ...
|
255bd81a | 06-Sep-2024 |
Karthik Rajagopalan <krajagopalan@nvidia.com> |
Use extern C guards in all headers
The project uses a mix of C and C++ requiring extern "C" guards from C++ code both within the project & from C++ apps that use libcper. That won't be required with
Use extern C guards in all headers
The project uses a mix of C and C++ requiring extern "C" guards from C++ code both within the project & from C++ apps that use libcper. That won't be required with this change.
Change-Id: I835dd05166732ca213c72eae2904815a8769599b Signed-off-by: Karthik Rajagopalan <krajagopalan@nvidia.com>
show more ...
|
379e492a | 28-Aug-2024 |
Patrick Williams <patrick@stwcx.xyz> |
Adjust strncpy sizes
When building under bitbake with the latest openbmc, we get compile warnings such as these:
``` | ../git/sections/cper-section-nvidia.c: In function 'ir_section_nvidia_to_cper'
Adjust strncpy sizes
When building under bitbake with the latest openbmc, we get compile warnings such as these:
``` | ../git/sections/cper-section-nvidia.c: In function 'ir_section_nvidia_to_cper': | ../git/sections/cper-section-nvidia.c:67:9: error: '__builtin_strncpy' specified bound 16 equals destination size [-Werror=stringop-truncation] | 67 | strncpy(section_cper->Signature, ```
Using `strncpy` on its own is unsafe because a string too long will end up in the destination buffer without NUL termination. Adjust the strncpy to be one shorter than the buffer and force the trailing byte to be a NUL.
Repeat this pattern for all `strncpy` calls.
Change-Id: I45c630733f0138d2b089a60f698d75e1c09de9e2 Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
show more ...
|
683e0550 | 07-Mar-2024 |
Karthik Rajagopalan <krajagopalan@nvidia.com> |
Add support for NVIDIA CPERs
Support Nvidia CPER entries.
Change-Id: Iea9bde181ead55ad99cdb2a341501bf48e1d82a8 Signed-off-by: Ed Tanous <etanous@nvidia.com> |
a7d2cddd | 15-Jul-2024 |
Ed Tanous <etanous@nvidia.com> |
Move to embedded base64
Base64 encode/decode is a relatively simple algorithm, and currently libcper takes a dependency on libb64 for this. libb64 does not have methods for determining the encoded
Move to embedded base64
Base64 encode/decode is a relatively simple algorithm, and currently libcper takes a dependency on libb64 for this. libb64 does not have methods for determining the encoded size or decoded size, and rely on the user to provide the right buffer sizes, which libcper currently approximates as 2X the input size (which is incorrect).
This commit removes the libb64 dependency entirely, and inlines a libcper specific base64 encoder and decoder, using EDK2-allowed types.
The implementation itself is unique to libcper and makes the following design decisions. 1. Malloc is performed within the base64_<> functions. This reduces the number of malloc calls total, and removes the need for separately determining the output size. 2. Arguments are passed in by EDK2-types under the assumption that this will keep compatibility with EDK2 implementations. 3. Incremental parsing is not supported. CPER records are expected to be algorithmically small, and buffered such that the entire value fits in memory. This was already an assumption, but dropping the support for incremental encoding significantly reduces the amount of code to support it. It could be added back in the future if needed.
Change-Id: Idb010db105067ea317dbee05c2663511ab3c6611 Signed-off-by: Ed Tanous <ed@tanous.net>
show more ...
|
fedd457d | 12-Jul-2024 |
Ed Tanous <ed@tanous.net> |
Remove trailing whitespace
clang-format won't remove trailing whitespace if that's the only change. Fix them all.
Change-Id: Ic6e14af43cdd11905d3b58430d49b9ec1484f812 Signed-off-by: Ed Tanous <ed@t
Remove trailing whitespace
clang-format won't remove trailing whitespace if that's the only change. Fix them all.
Change-Id: Ic6e14af43cdd11905d3b58430d49b9ec1484f812 Signed-off-by: Ed Tanous <ed@tanous.net>
show more ...
|
6981acbf | 18-Jun-2024 |
Ed Tanous <etanous@nvidia.com> |
Fix warning
Clang warns that this line is missing a comma, which it is. Add the comma.
Change-Id: I448427ce97d4f2577690578ee37b1198681be7a6 Signed-off-by: Ed Tanous <etanous@nvidia.com> |
0b9c9402 | 22-May-2024 |
John Chung <john.chung@arm.com> |
Fix compiler error : strict-aliasing
Signed-off-by: John Chung <john.chung@arm.com> Change-Id: Iae4d7962a7ffca20633b3a49c8c2cbc763a5f22d |
f8fc7052 | 03-May-2024 |
John Chung <john.chung@arm.com> |
Formatting .c/.h files and fix memory leakage issues
Signed-off-by: John Chung <john.chung@arm.com> Change-Id: Id8328f412c2724992d80c0b3f895c8f85bc4ae68 |
580423fe | 24-Aug-2022 |
Lawrence Tang <lawrence.tang@arm.com> |
Shift section definitions into separate file, add testing.
Change-Id: Idb0b41d7fa2999485580fca770958a27c1086f65 |
5202bbb4 | 12-Aug-2022 |
Lawrence Tang <lawrence.tang@arm.com> |
Source json.h non-locally.
Change-Id: Ia42c41fde74596b394a1f7ae0021f1a5510991a8 |
efe17e2c | 08-Aug-2022 |
Lawrence Tang <lawrence.tang@arm.com> |
Fix several review comments. |
3592da71 | 21-Jul-2022 |
Lawrence Tang <lawrence.tang@arm.com> |
Add human readable name to IA32x64 error info GUIDs. |
e407b4c8 | 21-Jul-2022 |
Lawrence Tang <lawrence.tang@arm.com> |
Partial reformat to kernel code style. |
0a4b3f2d | 21-Jul-2022 |
Lawrence Tang <lawrence.tang@arm.com> |
Fix bugs appearing from fuzzing. |
04750a9e | 20-Jul-2022 |
Lawrence Tang <lawrence.tang@arm.com> |
Fix errata in IA32x64 processing. |
3ab351fe | 20-Jul-2022 |
Lawrence Tang <lawrence.tang@arm.com> |
Fix errors in CPER write, remove PCIe AER support. |