public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Gao, Zhichao" <zhichao.gao@intel.com>
To: "devel@edk2.groups.io" <devel@edk2.groups.io>,
	"jeshuas@nvidia.com" <jeshuas@nvidia.com>
Cc: "Ni, Ray" <ray.ni@intel.com>
Subject: Re: [edk2-devel] [PATCH] ShellPkg/AcpiView: ERST Parser
Date: Wed, 1 Feb 2023 02:22:05 +0000	[thread overview]
Message-ID: <PH7PR11MB6377D27F1CA575C82BB495B0F6D19@PH7PR11MB6377.namprd11.prod.outlook.com> (raw)
In-Reply-To: <DM6PR12MB3371AC8BDE01458983F01C45DBD09@DM6PR12MB3371.namprd12.prod.outlook.com>

PR: https://github.com/tianocore/edk2/pull/3977

Thanks,
Zhichao

> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Jeshua
> Smith via groups.io
> Sent: Wednesday, February 1, 2023 2:35 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. 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
> > >
> > >
> >
> >
> >
> >
> >
> 
> 
> 
> 
> 


  reply	other threads:[~2023-02-01  2:22 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
2023-02-01  2:22         ` Gao, Zhichao [this message]
     [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=PH7PR11MB6377D27F1CA575C82BB495B0F6D19@PH7PR11MB6377.namprd11.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