ShellPkg/UefiShellAcpiViewCommandLib: Add support for PPTT

Added the acpiview parser for the PPTT table.

Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Krzysztof Koch <krzysztof.koch@arm.com>
Reviewed-by: Sami Mujawar <sami.mujawar@arm.com>
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com>

5 files changed, 391 insertions, 4 deletions

diff --git a/ShellPkg/Library/UefiShellAcpiViewCommandLib/AcpiParser.h b/ShellPkg/Library/UefiShellAcpiViewCommandLib/AcpiParser.h
index ecf7dae903..a42450c143 100644
--- a/ShellPkg/Library/UefiShellAcpiViewCommandLib/AcpiParser.h
+++ b/ShellPkg/Library/UefiShellAcpiViewCommandLib/AcpiParser.h
@@ -1,7 +1,7 @@
 /** @file

   Header file for ACPI parser

 

-  Copyright (c) 2016 - 2018, ARM Limited. All rights reserved.

+  Copyright (c) 2016 - 2019, ARM Limited. All rights reserved.

   This program and the accompanying materials

   are licensed and made available under the terms and conditions of the BSD License

   which accompanies this distribution.  The full text of the license may be found at

@@ -656,6 +656,27 @@ ParseAcpiMcfg (
   );

 

 /**

+  This function parses the ACPI PPTT table.

+  When trace is enabled this function parses the PPTT 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

+ParseAcpiPptt (

+  IN BOOLEAN Trace,

+  IN UINT8*  Ptr,

+  IN UINT32  AcpiTableLength,

+  IN UINT8   AcpiTableRevision

+  );

+

+/**

   This function parses the ACPI RSDP table.

 

   This function invokes the parser for the XSDT table.

diff --git a/ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Pptt/PpttParser.c b/ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Pptt/PpttParser.c
new file mode 100644
index 0000000000..d97ddf8e92
--- /dev/null
+++ b/ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Pptt/PpttParser.c
@@ -0,0 +1,362 @@
+/** @file

+  PPTT table parser

+

+  Copyright (c) 2019, ARM Limited. All rights reserved.

+  This program and the accompanying materials

+  are licensed and made available under the terms and conditions of the BSD License

+  which accompanies this distribution.  The full text of the license may be found at

+  http://opensource.org/licenses/bsd-license.php

+

+  THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,

+  WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.

+

+  @par Reference(s):

+    - ACPI 6.2 Specification - Errata A, September 2017

+**/

+

+#include <Library/PrintLib.h>

+#include <Library/UefiLib.h>

+#include "AcpiParser.h"

+

+// Local variables

+STATIC CONST UINT8*  ProcessorTopologyStructureType;

+STATIC CONST UINT8*  ProcessorTopologyStructureLength;

+STATIC CONST UINT32* NumberOfPrivateResources;

+STATIC ACPI_DESCRIPTION_HEADER_INFO AcpiHdrInfo;

+

+/**

+  An ACPI_PARSER array describing the ACPI PPTT Table.

+**/

+STATIC CONST ACPI_PARSER PpttParser[] = {

+  PARSE_ACPI_HEADER (&AcpiHdrInfo)

+};

+

+/**

+  This function validates the Cache Type Structure (Type 1) Line size field.

+

+  @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

+ValidateCacheLineSize (

+  IN UINT8* Ptr,

+  IN VOID*  Context

+  )

+{

+#if defined(MDE_CPU_ARM) || defined (MDE_CPU_AARCH64)

+  // Reference: ARM Architecture Reference Manual ARMv8 (D.a)

+  // Section D12.2.25: CCSIDR_EL1, Current Cache Size ID Register

+  //   LineSize, bits [2:0]

+  //     (Log2(Number of bytes in cache line)) - 4.

+

+  UINT16 LineSize;

+  LineSize = *(UINT16*)Ptr;

+

+  if ((LineSize < 16) || (LineSize > 2048)) {

+    IncrementErrorCount ();

+    Print (

+      L"\nERROR: The cache line size must be between 16 and 2048 bytes"

+        L" on ARM Platforms."

+      );

+    return;

+  }

+

+  if ((LineSize & (LineSize - 1)) != 0) {

+    IncrementErrorCount ();

+    Print (L"\nERROR: The cache line size is not a power of 2.");

+  }

+#endif

+}

+

+/**

+  This function validates the Cache Type Structure (Type 1) Attributes field.

+

+  @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

+ValidateCacheAttributes (

+  IN UINT8* Ptr,

+  IN VOID*  Context

+  )

+{

+#if defined(MDE_CPU_ARM) || defined (MDE_CPU_AARCH64)

+  // Reference: Advanced Configuration and Power Interface (ACPI) Specification

+  //            Version 6.2 Errata A, September 2017

+  // Table 5-153: Cache Type Structure

+

+  UINT8 Attributes;

+  Attributes = *(UINT8*)Ptr;

+

+  if ((Attributes & 0xE0) != 0) {

+    IncrementErrorCount ();

+    Print (

+      L"\nERROR: Attributes bits [7:5] are reserved and must be zero.",

+      Attributes

+      );

+    return;

+  }

+#endif

+}

+

+/**

+  An ACPI_PARSER array describing the processor topology structure header.

+**/

+STATIC CONST ACPI_PARSER ProcessorTopologyStructureHeaderParser[] = {

+  {L"Type", 1, 0, NULL, NULL, (VOID**)&ProcessorTopologyStructureType,

+   NULL, NULL},

+  {L"Length", 1, 1, NULL, NULL, (VOID**)&ProcessorTopologyStructureLength,

+   NULL, NULL},

+  {L"Reserved", 2, 2, NULL, NULL, NULL, NULL, NULL}

+};

+

+/**

+  An ACPI_PARSER array describing the Processor Hierarchy Node Structure - Type 0.

+**/

+STATIC CONST ACPI_PARSER ProcessorHierarchyNodeStructureParser[] = {

+  {L"Type", 1, 0, L"0x%x", NULL, NULL, NULL, NULL},

+  {L"Length", 1, 1, L"%d", NULL, NULL, NULL, NULL},

+  {L"Reserved", 2, 2, L"0x%x", NULL, NULL, NULL, NULL},

+

+  {L"Flags", 4, 4, L"0x%x", NULL, NULL, NULL, NULL},

+  {L"Parent", 4, 8, L"0x%x", NULL, NULL, NULL, NULL},

+  {L"ACPI Processor ID", 4, 12, L"0x%x", NULL, NULL, NULL, NULL},

+  {L"Number of private resources", 4, 16, L"%d", NULL,

+   (VOID**)&NumberOfPrivateResources, NULL, NULL}

+};

+

+/**

+  An ACPI_PARSER array describing the Cache Type Structure - Type 1.

+**/

+STATIC CONST ACPI_PARSER CacheTypeStructureParser[] = {

+  {L"Type", 1, 0, L"0x%x", NULL, NULL, NULL, NULL},

+  {L"Length", 1, 1, L"%d", NULL, NULL, NULL, NULL},

+  {L"Reserved", 2, 2, L"0x%x", NULL, NULL, NULL, NULL},

+

+  {L"Flags", 4, 4, L"0x%x", NULL, NULL, NULL, NULL},

+  {L"Next Level of Cache", 4, 8, L"0x%x", NULL, NULL, NULL, NULL},

+  {L"Size", 4, 12, L"0x%x", NULL, NULL, NULL, NULL},

+  {L"Number of sets", 4, 16, L"%d", NULL, NULL, NULL, NULL},

+  {L"Associativity", 1, 20, L"%d", NULL, NULL, NULL, NULL},

+  {L"Attributes", 1, 21, L"0x%x", NULL, NULL, ValidateCacheAttributes, NULL},

+  {L"Line size", 2, 22, L"%d", NULL, NULL, ValidateCacheLineSize, NULL}

+};

+

+/**

+  An ACPI_PARSER array describing the ID Type Structure - Type 2.

+**/

+STATIC CONST ACPI_PARSER IdStructureParser[] = {

+  {L"Type", 1, 0, L"0x%x", NULL, NULL, NULL, NULL},

+  {L"Length", 1, 1, L"%d", NULL, NULL, NULL, NULL},

+  {L"Reserved", 2, 2, L"0x%x", NULL, NULL, NULL, NULL},

+

+  {L"VENDOR_ID", 4, 4, L"0x%x", NULL, NULL, NULL, NULL},

+  {L"LEVEL_1_ID", 8, 8, L"0x%x", NULL, NULL, NULL, NULL},

+  {L"LEVEL_2_ID", 8, 16, L"0x%x", NULL, NULL, NULL, NULL},

+  {L"MAJOR_REV", 2, 24, L"0x%x", NULL, NULL, NULL, NULL},

+  {L"MINOR_REV", 2, 26, L"0x%x", NULL, NULL, NULL, NULL},

+  {L"SPIN_REV", 2, 28, L"0x%x", NULL, NULL, NULL, NULL},

+};

+

+/**

+  This function parses the Processor Hierarchy Node Structure (Type 0).

+

+  @param [in] Ptr     Pointer to the start of the Processor Hierarchy Node

+                      Structure data.

+  @param [in] Length  Length of the Processor Hierarchy Node Structure.

+**/

+STATIC

+VOID

+DumpProcessorHierarchyNodeStructure (

+  IN UINT8* Ptr,

+  IN UINT8  Length

+  )

+{

+  UINT8  Offset;

+  UINT8* PrivateResourcePtr;

+  UINT32 Index;

+  CHAR16 Buffer[OUTPUT_FIELD_COLUMN_WIDTH];

+

+  Offset = ParseAcpi (

+             TRUE,

+             2,

+             "Processor Hierarchy Node Structure",

+             Ptr,

+             Length,

+             PARSER_PARAMS (ProcessorHierarchyNodeStructureParser)

+             );

+

+  PrivateResourcePtr = Ptr + Offset;

+  Index = 0;

+  while (Index < *NumberOfPrivateResources) {

+    UnicodeSPrint (

+      Buffer,

+      sizeof (Buffer),

+      L"Private resources [%d]",

+      Index

+      );

+

+    PrintFieldName (4, Buffer);

+    Print (

+      L"0x%x\n",

+      *((UINT32*) PrivateResourcePtr)

+      );

+

+    PrivateResourcePtr += sizeof(UINT32);

+    Index++;

+  }

+}

+

+/**

+  This function parses the Cache Type Structure (Type 1).

+

+  @param [in] Ptr     Pointer to the start of the Cache Type Structure data.

+  @param [in] Length  Length of the Cache Type Structure.

+**/

+STATIC

+VOID

+DumpCacheTypeStructure (

+  IN UINT8* Ptr,

+  IN UINT8  Length

+  )

+{

+  ParseAcpi (

+    TRUE,

+    2,

+    "Cache Type Structure",

+    Ptr,

+    Length,

+    PARSER_PARAMS (CacheTypeStructureParser)

+    );

+}

+

+/**

+  This function parses the ID Structure (Type 2).

+

+  @param [in] Ptr     Pointer to the start of the ID Structure data.

+  @param [in] Length  Length of the ID Structure.

+**/

+STATIC

+VOID

+DumpIDStructure (

+  IN UINT8* Ptr,

+  IN UINT8 Length

+  )

+{

+  ParseAcpi (

+    TRUE,

+    2,

+    "ID Structure",

+    Ptr,

+    Length,

+    PARSER_PARAMS (IdStructureParser)

+    );

+}

+

+/**

+  This function parses the ACPI PPTT table.

+  When trace is enabled this function parses the PPTT table and

+  traces the ACPI table fields.

+

+  This function parses the following processor topology structures:

+    - Processor hierarchy node structure (Type 0)

+    - Cache Type Structure (Type 1)

+    - ID structure (Type 2)

+

+  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

+ParseAcpiPptt (

+  IN BOOLEAN Trace,

+  IN UINT8*  Ptr,

+  IN UINT32  AcpiTableLength,

+  IN UINT8   AcpiTableRevision

+  )

+{

+  UINT32 Offset;

+  UINT8* ProcessorTopologyStructurePtr;

+

+  if (!Trace) {

+    return;

+  }

+

+  Offset = ParseAcpi (

+             TRUE,

+             0,

+             "PPTT",

+             Ptr,

+             AcpiTableLength,

+             PARSER_PARAMS (PpttParser)

+             );

+  ProcessorTopologyStructurePtr = Ptr + Offset;

+

+  while (Offset < AcpiTableLength) {

+    // Parse Processor Hierarchy Node Structure to obtain Type and Length.

+    ParseAcpi (

+      FALSE,

+      0,

+      NULL,

+      ProcessorTopologyStructurePtr,

+      4,  // Length of the processor topology structure header is 4 bytes

+      PARSER_PARAMS (ProcessorTopologyStructureHeaderParser)

+      );

+

+    if ((Offset + (*ProcessorTopologyStructureLength)) > AcpiTableLength) {

+      IncrementErrorCount ();

+      Print (

+        L"ERROR: Invalid processor topology structure length:"

+          L" Type = %d, Length = %d\n",

+        *ProcessorTopologyStructureType,

+        *ProcessorTopologyStructureLength

+        );

+      break;

+    }

+

+    PrintFieldName (2, L"* Structure Offset *");

+    Print (L"0x%x\n", Offset);

+

+    switch (*ProcessorTopologyStructureType) {

+      case EFI_ACPI_6_2_PPTT_TYPE_PROCESSOR:

+        DumpProcessorHierarchyNodeStructure (

+          ProcessorTopologyStructurePtr,

+          *ProcessorTopologyStructureLength

+          );

+        break;

+      case EFI_ACPI_6_2_PPTT_TYPE_CACHE:

+        DumpCacheTypeStructure (

+          ProcessorTopologyStructurePtr,

+          *ProcessorTopologyStructureLength

+          );

+        break;

+      case EFI_ACPI_6_2_PPTT_TYPE_ID:

+        DumpIDStructure (

+          ProcessorTopologyStructurePtr,

+          *ProcessorTopologyStructureLength

+          );

+        break;

+      default:

+        IncrementErrorCount ();

+        Print (

+          L"ERROR: Unknown processor topology structure:"

+            L" Type = %d, Length = %d\n",

+          *ProcessorTopologyStructureType,

+          *ProcessorTopologyStructureLength

+          );

+    }

+

+    ProcessorTopologyStructurePtr += *ProcessorTopologyStructureLength;

+    Offset += *ProcessorTopologyStructureLength;

+  } // while

+}

diff --git a/ShellPkg/Library/UefiShellAcpiViewCommandLib/UefiShellAcpiViewCommandLib.c b/ShellPkg/Library/UefiShellAcpiViewCommandLib/UefiShellAcpiViewCommandLib.c
index c6eb7087cf..2f908bfbd3 100644
--- a/ShellPkg/Library/UefiShellAcpiViewCommandLib/UefiShellAcpiViewCommandLib.c
+++ b/ShellPkg/Library/UefiShellAcpiViewCommandLib/UefiShellAcpiViewCommandLib.c
@@ -1,7 +1,7 @@
 /** @file

   Main file for 'acpiview' Shell command function.

 

-  Copyright (c) 2016 - 2018, ARM Limited. All rights reserved.<BR>

+  Copyright (c) 2016 - 2019, ARM Limited. All rights reserved.<BR>

   This program and the accompanying materials

   are licensed and made available under the terms and conditions of the BSD License

   which accompanies this distribution.  The full text of the license may be found at

@@ -41,6 +41,8 @@ ACPI_TABLE_PARSER ParserList[] = {
   {EFI_ACPI_6_2_MULTIPLE_APIC_DESCRIPTION_TABLE_SIGNATURE, ParseAcpiMadt},

   {EFI_ACPI_6_2_PCI_EXPRESS_MEMORY_MAPPED_CONFIGURATION_SPACE_BASE_ADDRESS_DESCRIPTION_TABLE_SIGNATURE,

    ParseAcpiMcfg},

+  {EFI_ACPI_6_2_PROCESSOR_PROPERTIES_TOPOLOGY_TABLE_STRUCTURE_SIGNATURE,

+   ParseAcpiPptt},

   {RSDP_TABLE_INFO, ParseAcpiRsdp},

   {EFI_ACPI_6_2_SYSTEM_LOCALITY_INFORMATION_TABLE_SIGNATURE, ParseAcpiSlit},

   {EFI_ACPI_6_2_SERIAL_PORT_CONSOLE_REDIRECTION_TABLE_SIGNATURE, ParseAcpiSpcr},

diff --git a/ShellPkg/Library/UefiShellAcpiViewCommandLib/UefiShellAcpiViewCommandLib.inf b/ShellPkg/Library/UefiShellAcpiViewCommandLib/UefiShellAcpiViewCommandLib.inf
index 6df8c08b85..db06e7a066 100644
--- a/ShellPkg/Library/UefiShellAcpiViewCommandLib/UefiShellAcpiViewCommandLib.inf
+++ b/ShellPkg/Library/UefiShellAcpiViewCommandLib/UefiShellAcpiViewCommandLib.inf
@@ -1,7 +1,7 @@
 ##  @file

 # Provides Shell 'acpiview' command functions

 #

-# Copyright (c) 2016 - 2018, ARM Limited. All rights reserved.<BR>

+# Copyright (c) 2016 - 2019, ARM Limited. All rights reserved.<BR>

 #

 #  This program and the accompanying materials

 #  are licensed and made available under the terms and conditions of the BSD License

@@ -41,6 +41,7 @@
   Parsers/Iort/IortParser.c

   Parsers/Madt/MadtParser.c

   Parsers/Mcfg/McfgParser.c

+  Parsers/Pptt/PpttParser.c

   Parsers/Rsdp/RsdpParser.c

   Parsers/Slit/SlitParser.c

   Parsers/Spcr/SpcrParser.c

diff --git a/ShellPkg/Library/UefiShellAcpiViewCommandLib/UefiShellAcpiViewCommandLib.uni b/ShellPkg/Library/UefiShellAcpiViewCommandLib/UefiShellAcpiViewCommandLib.uni
index 1c4d66e1ff..0762eeba53 100644
--- a/ShellPkg/Library/UefiShellAcpiViewCommandLib/UefiShellAcpiViewCommandLib.uni
+++ b/ShellPkg/Library/UefiShellAcpiViewCommandLib/UefiShellAcpiViewCommandLib.uni
@@ -1,6 +1,6 @@
 // /**

 //

-// Copyright (c) 2016 - 2017, ARM Limited. All rights reserved.<BR>

+// Copyright (c) 2016 - 2019, ARM Limited. All rights reserved.<BR>

 // This program and the accompanying materials

 // are licensed and made available under the terms and conditions of the BSD License

 // which accompanies this distribution. The full text of the license may be found at

@@ -82,6 +82,7 @@
 "       GTDT  - Generic Timer Description Table\r\n"

 "       IORT  - IO Remapping Table\r\n"

 "       MCFG  - Memory Mapped Config Space Base Address Description Table\r\n"

+"       PPTT  - Processor Properties Topology Table\r\n"

 "       RSDP  - Root System Description Pointer\r\n"

 "       SLIT  - System Locality Information Table\r\n"

 "       SPCR  - Serial Port Console Redirection Table\r\n"