From: "Jeshua Smith" <jeshuas@nvidia.com>
To: "Gao, Zhichao" <zhichao.gao@intel.com>,
"devel@edk2.groups.io" <devel@edk2.groups.io>
Cc: "Ni, Ray" <ray.ni@intel.com>
Subject: Re: [edk2-devel] [PATCH] ShellPkg/AcpiView: ERST Parser
Date: Tue, 31 Jan 2023 18:35:12 +0000 [thread overview]
Message-ID: <DM6PR12MB3371AC8BDE01458983F01C45DBD09@DM6PR12MB3371.namprd12.prod.outlook.com> (raw)
In-Reply-To: <PH7PR11MB637728D770A3808CCDC55645F6CD9@PH7PR11MB6377.namprd11.prod.outlook.com>
Thanks. Does this work for you?
https://github.com/NVIDIA/edk2/pull/19
-----Original Message-----
From: Gao, Zhichao <zhichao.gao@intel.com>
Sent: Friday, January 27, 2023 8:59 PM
To: devel@edk2.groups.io; Jeshua Smith <jeshuas@nvidia.com>
Cc: Ni, Ray <ray.ni@intel.com>
Subject: RE: [edk2-devel] [PATCH] ShellPkg/AcpiView: ERST Parser
External email: Use caution opening links or attachments
Can you share then code change link of your personal repo? The patch I download from this email cannot apply to the open source branch.
Thanks,
Zhichao
> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Jeshua
> Smith via groups.io
> Sent: Wednesday, January 25, 2023 4:55 AM
> To: Gao, Zhichao <zhichao.gao@intel.com>; devel@edk2.groups.io
> Cc: Ni, Ray <ray.ni@intel.com>
> Subject: Re: [edk2-devel] [PATCH] ShellPkg/AcpiView: ERST Parser
>
> Thanks for the review. What is the next step to get this merged?
>
> -----Original Message-----
> From: Gao, Zhichao <zhichao.gao@intel.com>
> Sent: Sunday, January 15, 2023 5:59 PM
> To: Jeshua Smith <jeshuas@nvidia.com>; devel@edk2.groups.io
> Cc: Ni, Ray <ray.ni@intel.com>
> Subject: RE: [PATCH] ShellPkg/AcpiView: ERST Parser
>
> External email: Use caution opening links or attachments
>
>
> Reviewed-by: Zhichao Gao <zhichao.gao@intel.com>
>
> Thanks,
> Zhichao
>
> > -----Original Message-----
> > From: Jeshua Smith <jeshuas@nvidia.com>
> > Sent: Thursday, December 1, 2022 2:31 AM
> > To: devel@edk2.groups.io
> > Cc: Ni, Ray <ray.ni@intel.com>; Gao, Zhichao
> > <zhichao.gao@intel.com>; Jeshua Smith <jeshuas@nvidia.com>
> > Subject: [PATCH] ShellPkg/AcpiView: ERST Parser
> >
> > Add a new parser for the Error Record Serialization Table.
> >
> > The ERST table describes how an OS can save and retrieve
> >
> > hardware error information to and from a persistent store.
> >
> >
> >
> > Signed-off-by: Jeshua Smith <jeshuas@nvidia.com>
> >
> > ---
> >
> > .../UefiShellAcpiViewCommandLib/AcpiParser.h | 22 ++
> >
> > .../Parsers/Erst/ErstParser.c | 278 ++++++++++++++++++
> >
> > .../UefiShellAcpiViewCommandLib.c | 2 +
> >
> > .../UefiShellAcpiViewCommandLib.inf | 2 +
> >
> > 4 files changed, 304 insertions(+)
> >
> > create mode 100644
> > ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Erst/ErstParser
> > .c
> >
> >
> >
> > diff --git
> > a/ShellPkg/Library/UefiShellAcpiViewCommandLib/AcpiParser.h
> > b/ShellPkg/Library/UefiShellAcpiViewCommandLib/AcpiParser.h
> >
> > index db8c88f6df..66c992c55c 100644
> >
> > --- a/ShellPkg/Library/UefiShellAcpiViewCommandLib/AcpiParser.h
> >
> > +++ b/ShellPkg/Library/UefiShellAcpiViewCommandLib/AcpiParser.h
> >
> > @@ -1,6 +1,7 @@
> >
> > /** @file
> >
> > Header file for ACPI parser
> >
> >
> >
> > + Copyright (c) 2022, NVIDIA CORPORATION. All rights reserved.
> >
> > Copyright (c) 2016 - 2020, Arm Limited. All rights reserved.
> >
> > Copyright (c) 2022, AMD Incorporated. All rights reserved.
> >
> > SPDX-License-Identifier: BSD-2-Clause-Patent
> >
> > @@ -594,6 +595,27 @@ ParseAcpiDsdt (
> >
> > IN UINT8 AcpiTableRevision
> >
> > );
> >
> >
> >
> > +/**
> >
> > + This function parses the ACPI ERST table.
> >
> > + When trace is enabled this function parses the ERST table and
> >
> > + traces the ACPI table fields.
> >
> > +
> >
> > + This function also performs validation of the ACPI table fields.
> >
> > +
> >
> > + @param [in] Trace If TRUE, trace the ACPI fields.
> >
> > + @param [in] Ptr Pointer to the start of the buffer.
> >
> > + @param [in] AcpiTableLength Length of the ACPI table.
> >
> > + @param [in] AcpiTableRevision Revision of the ACPI table.
> >
> > +**/
> >
> > +VOID
> >
> > +EFIAPI
> >
> > +ParseAcpiErst (
> >
> > + IN BOOLEAN Trace,
> >
> > + IN UINT8 *Ptr,
> >
> > + IN UINT32 AcpiTableLength,
> >
> > + IN UINT8 AcpiTableRevision
> >
> > + );
> >
> > +
> >
> > /**
> >
> > This function parses the ACPI FACS table.
> >
> > When trace is enabled this function parses the FACS table and
> >
> > diff --git
> > a/ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Erst/ErstPars
> > er
> > .c
> > b/ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Erst/ErstPars
> > er
> > .c
> >
> > new file mode 100644
> >
> > index 0000000000..e2af0c44b4
> >
> > --- /dev/null
> >
> > +++
> > b/ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Erst/ErstPars
> > er
> > .c
> >
> > @@ -0,0 +1,278 @@
> >
> > +/** @file
> >
> > + ERST table parser
> >
> > +
> >
> > + Copyright (c) 2022, NVIDIA CORPORATION. All rights reserved.
> >
> > + Copyright (c) 2016 - 2018, ARM Limited. All rights reserved.
> >
> > + SPDX-License-Identifier: BSD-2-Clause-Patent
> >
> > +
> >
> > + @par Reference(s):
> >
> > + - ACPI 6.5 Specification - August 2022
> >
> > +**/
> >
> > +
> >
> > +#include <IndustryStandard/Acpi.h>
> >
> > +#include <Library/UefiLib.h>
> >
> > +#include "AcpiParser.h"
> >
> > +#include "AcpiTableParser.h"
> >
> > +
> >
> > +// Local variables
> >
> > +STATIC ACPI_DESCRIPTION_HEADER_INFO AcpiHdrInfo;
> >
> > +STATIC UINT32 *InstructionEntryCount;
> >
> > +
> >
> > +/**
> >
> > + An array of strings describing the Erst actions
> >
> > +**/
> >
> > +STATIC CONST CHAR16 *ErstActionTable[] = {
> >
> > + L"BEGIN_WRITE_OPERATION",
> >
> > + L"BEGIN_READ_OPERATION",
> >
> > + L"BEGIN_CLEAR_OPERATION",
> >
> > + L"END_OPERATION",
> >
> > + L"SET_RECORD_OFFSET",
> >
> > + L"EXECUTE_OPERATION",
> >
> > + L"CHECK_BUSY_STATUS",
> >
> > + L"GET_COMMAND_STATUS",
> >
> > + L"GET_RECORD_IDENTIFIER",
> >
> > + L"SET_RECORD_IDENTIFIER",
> >
> > + L"GET_RECORD_COUNT",
> >
> > + L"BEGIN_DUMMY_WRITE_OPERATION",
> >
> > + L"RESERVED",
> >
> > + L"GET_ERROR_LOG_ADDRESS_RANGE",
> >
> > + L"GET_ERROR_LOG_ADDRESS_RANGE_LENGTH",
> >
> > + L"GET_ERROR_LOG_ADDRESS_RANGE_ATTRIBUTES",
> >
> > + L"GET_EXECUTE_OPERATION_TIMINGS"
> >
> > +};
> >
> > +
> >
> > +/**
> >
> > + An array of strings describing the Erst instructions
> >
> > +**/
> >
> > +STATIC CONST CHAR16 *ErstInstructionTable[] = {
> >
> > + L"READ_REGISTER",
> >
> > + L"READ_REGISTER_VALUE",
> >
> > + L"WRITE_REGISTER",
> >
> > + L"WRITE_REGISTER_VALUE",
> >
> > + L"NOOP",
> >
> > + L"LOAD_VAR1",
> >
> > + L"LOAD_VAR2",
> >
> > + L"STORE_VAR1",
> >
> > + L"ADD",
> >
> > + L"SUBTRACT",
> >
> > + L"ADD_VALUE",
> >
> > + L"SUBTRACT_VALUE",
> >
> > + L"STALL",
> >
> > + L"STALL_WHILE_TRUE",
> >
> > + L"SKIP_NEXT_INSTRUCTION_IF_TRUE",
> >
> > + L"GOTO",
> >
> > + L"SET_SRC_ADDRESS_BASE",
> >
> > + L"SET_DST_ADDRESS_BASE",
> >
> > + L"MOVE_DATA"
> >
> > +};
> >
> > +
> >
> > +/**
> >
> > + Validate Erst action.
> >
> > +
> >
> > + @param [in] Ptr Pointer to the start of the field data.
> >
> > + @param [in] Context Pointer to context specific information e.g. this
> >
> > + could be a pointer to the ACPI table header.
> >
> > +**/
> >
> > +STATIC
> >
> > +VOID
> >
> > +EFIAPI
> >
> > +ValidateErstAction (
> >
> > + IN UINT8 *Ptr,
> >
> > + IN VOID *Context
> >
> > + )
> >
> > +{
> >
> > + if (*Ptr > EFI_ACPI_6_4_ERST_GET_EXECUTE_OPERATION_TIMINGS) {
> >
> > + IncrementErrorCount ();
> >
> > + Print (L"\nError: 0x%02x is not a valid action encoding",
> > + *Ptr);
> >
> > + }
> >
> > +}
> >
> > +
> >
> > +/**
> >
> > + Validate Erst instruction.
> >
> > +
> >
> > + @param [in] Ptr Pointer to the start of the field data.
> >
> > + @param [in] Context Pointer to context specific information e.g. this
> >
> > + could be a pointer to the ACPI table header.
> >
> > +**/
> >
> > +STATIC
> >
> > +VOID
> >
> > +EFIAPI
> >
> > +ValidateErstInstruction (
> >
> > + IN UINT8 *Ptr,
> >
> > + IN VOID *Context
> >
> > + )
> >
> > +{
> >
> > + if (*Ptr > EFI_ACPI_6_4_ERST_MOVE_DATA) {
> >
> > + IncrementErrorCount ();
> >
> > + Print (L"\nError: 0x%02x is not a valid instruction encoding",
> > + *Ptr);
> >
> > + }
> >
> > +}
> >
> > +
> >
> > +/**
> >
> > + Validate Erst flags.
> >
> > +
> >
> > + @param [in] Ptr Pointer to the start of the field data.
> >
> > + @param [in] Context Pointer to context specific information e.g. this
> >
> > + could be a pointer to the ACPI table header.
> >
> > +**/
> >
> > +STATIC
> >
> > +VOID
> >
> > +EFIAPI
> >
> > +ValidateErstFlags (
> >
> > + IN UINT8 *Ptr,
> >
> > + IN VOID *Context
> >
> > + )
> >
> > +{
> >
> > + if ((*Ptr & 0xfe) != 0) {
> >
> > + IncrementErrorCount ();
> >
> > + Print (L"\nError: Reserved Flag bits not set to 0");
> >
> > + }
> >
> > +}
> >
> > +
> >
> > +/**
> >
> > + Looks up and prints the string corresponding to the index
> >
> > +
> >
> > + @param [in] Table Lookup table
> >
> > + @param [in] Index Entry to print
> >
> > + @param [in] NumEntries Number of valid entries in the table
> >
> > +**/
> >
> > +STATIC
> >
> > +VOID
> >
> > +EFIAPI
> >
> > +FormatByte (
> >
> > + IN CONST CHAR16 *Table[],
> >
> > + IN UINT8 Index,
> >
> > + IN UINT8 NumEntries
> >
> > + )
> >
> > +{
> >
> > + CONST CHAR16 *String;
> >
> > +
> >
> > + if (Index < NumEntries) {
> >
> > + String = Table[Index];
> >
> > + } else {
> >
> > + String = L"**INVALID**";
> >
> > + }
> >
> > +
> >
> > + Print (
> >
> > + L"0x%02x (%s)",
> >
> > + Index,
> >
> > + String
> >
> > + );
> >
> > +}
> >
> > +
> >
> > +/**
> >
> > + Prints the Erst Action
> >
> > +
> >
> > + @param [in] Format Optional format string for tracing the data.
> >
> > + @param [in] Ptr Pointer to the Action byte
> >
> > +**/
> >
> > +STATIC
> >
> > +VOID
> >
> > +EFIAPI
> >
> > +DumpErstAction (
> >
> > + IN CONST CHAR16 *Format OPTIONAL,
> >
> > + IN UINT8 *Ptr
> >
> > + )
> >
> > +{
> >
> > + FormatByte (ErstActionTable, *Ptr, ARRAY_SIZE (ErstActionTable));
> >
> > +}
> >
> > +
> >
> > +/**
> >
> > + Prints the Erst Instruction
> >
> > +
> >
> > + @param [in] Format Optional format string for tracing the data.
> >
> > + @param [in] Ptr Pointer to the Instruction byte
> >
> > +**/
> >
> > +STATIC
> >
> > +VOID
> >
> > +EFIAPI
> >
> > +DumpErstInstruction (
> >
> > + IN CONST CHAR16 *Format OPTIONAL,
> >
> > + IN UINT8 *Ptr
> >
> > + )
> >
> > +{
> >
> > + FormatByte (ErstInstructionTable, *Ptr, ARRAY_SIZE
> > + (ErstInstructionTable));
> >
> > +}
> >
> > +
> >
> > +/**
> >
> > + An ACPI_PARSER array describing the ACPI ERST Table.
> >
> > +**/
> >
> > +STATIC CONST ACPI_PARSER ErstParser[] = {
> >
> > + PARSE_ACPI_HEADER (&AcpiHdrInfo),
> >
> > + { L"Serialization Header Size", 4, 36, L"0x%x", NULL, NULL,
> > NULL, NULL },
> >
> > + { L"Reserved", 4, 40, L"0x%x", NULL, NULL, NULL,
> > NULL },
> >
> > + { L"Instruction Entry Count", 4, 44, L"0x%x", NULL, (VOID
> > **)&InstructionEntryCount, NULL, NULL }
> >
> > +};
> >
> > +
> >
> > +/**
> >
> > + An ACPI_PARSER array describing the Serialization Instruction
> > + Entry
> > structure.
> >
> > +**/
> >
> > +STATIC CONST ACPI_PARSER SerializationInstructionEntryParser[] = {
> >
> > + { L"Serialization Action", 1, 0, L"0x%x", DumpErstAction, NULL,
> > ValidateErstAction, NULL },
> >
> > + { L"Instruction", 1, 1, L"0x%x", DumpErstInstruction, NULL,
> > ValidateErstInstruction, NULL },
> >
> > + { L"Flags", 1, 2, L"0x%x", NULL, NULL, ValidateErstFlags,
> > NULL },
> >
> > + { L"Reserved", 1, 3, L"0x%x", NULL, NULL, NULL,
> > NULL },
> >
> > + { L"Register Region", 12, 4, NULL, DumpGas, NULL, NULL,
> > NULL },
> >
> > + { L"Value", 8, 16, L"0x%llx", NULL, NULL, NULL,
> > NULL },
> >
> > + { L"Mask", 8, 24, L"0x%llx", NULL, NULL, NULL,
> > NULL }
> >
> > +};
> >
> > +
> >
> > +/**
> >
> > + This function parses the ACPI ERST table.
> >
> > + When trace is enabled this function parses the ERST table and
> >
> > + traces the ACPI table fields.
> >
> > +
> >
> > + This function also performs validation of the ACPI table fields.
> >
> > +
> >
> > + @param [in] Trace If TRUE, trace the ACPI fields.
> >
> > + @param [in] Ptr Pointer to the start of the buffer.
> >
> > + @param [in] AcpiTableLength Length of the ACPI table.
> >
> > + @param [in] AcpiTableRevision Revision of the ACPI table.
> >
> > +**/
> >
> > +VOID
> >
> > +EFIAPI
> >
> > +ParseAcpiErst (
> >
> > + IN BOOLEAN Trace,
> >
> > + IN UINT8 *Ptr,
> >
> > + IN UINT32 AcpiTableLength,
> >
> > + IN UINT8 AcpiTableRevision
> >
> > + )
> >
> > +{
> >
> > + UINT32 Offset;
> >
> > +
> >
> > + if (!Trace) {
> >
> > + return;
> >
> > + }
> >
> > +
> >
> > + Offset = ParseAcpi (
> >
> > + Trace,
> >
> > + 0,
> >
> > + "ERST",
> >
> > + Ptr,
> >
> > + AcpiTableLength,
> >
> > + PARSER_PARAMS (ErstParser)
> >
> > + );
> >
> > +
> >
> > + if (sizeof
> >
> (EFI_ACPI_6_4_ERST_SERIALIZATION_INSTRUCTION_ENTRY)*(*InstructionE
> nt
> > ryCount) != (AcpiTableLength - Offset)) {
> >
> > + IncrementErrorCount ();
> >
> > + Print (
> >
> > + L"ERROR: Invalid InstructionEntryCount. " \
> >
> > + L"Count = %d. Offset = %d. AcpiTableLength = %d.\n",
> >
> > + *InstructionEntryCount,
> >
> > + Offset,
> >
> > + AcpiTableLength
> >
> > + );
> >
> > + return;
> >
> > + }
> >
> > +
> >
> > + while (Offset < AcpiTableLength) {
> >
> > + Offset += ParseAcpi (
> >
> > + Trace,
> >
> > + 2,
> >
> > + "Serialization Action",
> >
> > + Ptr + Offset,
> >
> > + (AcpiTableLength - Offset),
> >
> > + PARSER_PARAMS (SerializationInstructionEntryParser)
> >
> > + );
> >
> > + }
> >
> > +}
> >
> > diff --git
> >
> a/ShellPkg/Library/UefiShellAcpiViewCommandLib/UefiShellAcpiViewComm
> a
> > ndLib.c
> >
> b/ShellPkg/Library/UefiShellAcpiViewCommandLib/UefiShellAcpiViewComm
> a
> > ndLib.c
> >
> > index 09bdddb56e..d37ad7cacc 100644
> >
> > ---
> >
> a/ShellPkg/Library/UefiShellAcpiViewCommandLib/UefiShellAcpiViewComm
> a
> > ndLib.c
> >
> > +++
> >
> b/ShellPkg/Library/UefiShellAcpiViewCommandLib/UefiShellAcpiViewComm
> a
> > ndLib.c
> >
> > @@ -1,6 +1,7 @@
> >
> > /** @file
> >
> > Main file for 'acpiview' Shell command function.
> >
> >
> >
> > + Copyright (c) 2022, NVIDIA CORPORATION. All rights reserved.
> >
> > Copyright (c) 2016 - 2021, Arm Limited. All rights reserved.<BR>
> >
> > SPDX-License-Identifier: BSD-2-Clause-Patent
> >
> > **/
> >
> > @@ -52,6 +53,7 @@ ACPI_TABLE_PARSER ParserList[] = {
> >
> > { EFI_ACPI_6_2_DEBUG_PORT_2_TABLE_SIGNATURE,
> > ParseAcpiDbg2 },
> >
> >
> >
> { EFI_ACPI_6_2_DIFFERENTIATED_SYSTEM_DESCRIPTION_TABLE_SIGNATUR
> E,
> >
> > ParseAcpiDsdt },
> >
> > + { EFI_ACPI_6_4_ERROR_RECORD_SERIALIZATION_TABLE_SIGNATURE,
> > ParseAcpiErst },
> >
> > { EFI_ACPI_6_3_FIRMWARE_ACPI_CONTROL_STRUCTURE_SIGNATURE,
> > ParseAcpiFacs },
> >
> > { EFI_ACPI_6_2_FIXED_ACPI_DESCRIPTION_TABLE_SIGNATURE,
> > ParseAcpiFadt },
> >
> > { EFI_ACPI_6_4_GENERIC_TIMER_DESCRIPTION_TABLE_SIGNATURE,
> > ParseAcpiGtdt },
> >
> > diff --git
> >
> a/ShellPkg/Library/UefiShellAcpiViewCommandLib/UefiShellAcpiViewComm
> a
> > ndLib.inf
> >
> b/ShellPkg/Library/UefiShellAcpiViewCommandLib/UefiShellAcpiViewComm
> a
> > ndLib.inf
> >
> > index 63fc5a1281..904fea83de 100644
> >
> > ---
> >
> a/ShellPkg/Library/UefiShellAcpiViewCommandLib/UefiShellAcpiViewComm
> a
> > ndLib.inf
> >
> > +++
> >
> b/ShellPkg/Library/UefiShellAcpiViewCommandLib/UefiShellAcpiViewComm
> a
> > ndLib.inf
> >
> > @@ -1,6 +1,7 @@
> >
> > ## @file
> >
> > # Provides Shell 'acpiview' command functions
> >
> > #
> >
> > +# Copyright (c) 2022, NVIDIA CORPORATION. All rights reserved.
> >
> > # Copyright (c) 2016 - 2020, Arm Limited. All rights reserved.<BR>
> >
> > #
> >
> > # SPDX-License-Identifier: BSD-2-Clause-Patent
> >
> > @@ -31,6 +32,7 @@
> >
> > Parsers/Bgrt/BgrtParser.c
> >
> > Parsers/Dbg2/Dbg2Parser.c
> >
> > Parsers/Dsdt/DsdtParser.c
> >
> > + Parsers/Erst/ErstParser.c
> >
> > Parsers/Facs/FacsParser.c
> >
> > Parsers/Fadt/FadtParser.c
> >
> > Parsers/Gtdt/GtdtParser.c
> >
> > --
> >
> > 2.25.1
> >
> >
>
>
>
>
>
next prev parent reply other threads:[~2023-01-31 18:35 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-11-30 18:31 [PATCH] ShellPkg/AcpiView: ERST Parser jeshuas
2023-01-16 0:58 ` Gao, Zhichao
2023-01-24 20:55 ` Jeshua Smith
2023-01-28 3:58 ` [edk2-devel] " Gao, Zhichao
2023-01-31 18:35 ` Jeshua Smith [this message]
2023-02-01 2:22 ` Gao, Zhichao
[not found] <172C707E2AD4FEF1.1705@groups.io>
2022-12-14 15:30 ` Jeshua Smith
2023-01-05 16:47 ` Jeshua Smith
2023-01-06 5:32 ` Gao, Zhichao
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=DM6PR12MB3371AC8BDE01458983F01C45DBD09@DM6PR12MB3371.namprd12.prod.outlook.com \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox