|
|
edb6c78 |
From dfe414b053fb9feb905271785b319afa7ba72932 Mon Sep 17 00:00:00 2001
|
|
|
edb6c78 |
From: Al Stone <ahs3@redhat.com>
|
|
|
edb6c78 |
Date: Wed, 23 Sep 2020 16:08:24 -0600
|
|
|
edb6c78 |
Subject: [PATCH 22/40] Support MPST in a big-endian world
|
|
|
edb6c78 |
|
|
|
edb6c78 |
Signed-off-by: Al Stone <ahs3@redhat.com>
|
|
|
edb6c78 |
---
|
|
|
edb6c78 |
source/common/dmtbdump2.c | 33 +++++++++++++++++++--------------
|
|
|
edb6c78 |
source/compiler/dttable2.c | 8 ++++----
|
|
|
edb6c78 |
2 files changed, 23 insertions(+), 18 deletions(-)
|
|
|
edb6c78 |
|
|
|
edb6c78 |
Index: acpica-unix2-20200925/source/common/dmtbdump2.c
|
|
|
edb6c78 |
===================================================================
|
|
|
edb6c78 |
--- acpica-unix2-20200925.orig/source/common/dmtbdump2.c
|
|
|
edb6c78 |
+++ acpica-unix2-20200925/source/common/dmtbdump2.c
|
|
|
edb6c78 |
@@ -834,11 +834,13 @@ AcpiDmDumpMpst (
|
|
|
edb6c78 |
UINT16 SubtableCount;
|
|
|
edb6c78 |
UINT32 PowerStateCount;
|
|
|
edb6c78 |
UINT32 ComponentCount;
|
|
|
edb6c78 |
+ UINT32 TableLength = AcpiUtReadUint32(&Table->Length);
|
|
|
edb6c78 |
+ UINT16 Tmp16;
|
|
|
edb6c78 |
|
|
|
edb6c78 |
|
|
|
edb6c78 |
/* Main table */
|
|
|
edb6c78 |
|
|
|
edb6c78 |
- Status = AcpiDmDumpTable (Table->Length, 0, Table, 0, AcpiDmTableInfoMpst);
|
|
|
edb6c78 |
+ Status = AcpiDmDumpTable (TableLength, 0, Table, 0, AcpiDmTableInfoMpst);
|
|
|
edb6c78 |
if (ACPI_FAILURE (Status))
|
|
|
edb6c78 |
{
|
|
|
edb6c78 |
return;
|
|
|
edb6c78 |
@@ -846,13 +848,14 @@ AcpiDmDumpMpst (
|
|
|
edb6c78 |
|
|
|
edb6c78 |
/* Subtable: Memory Power Node(s) */
|
|
|
edb6c78 |
|
|
|
edb6c78 |
- SubtableCount = (ACPI_CAST_PTR (ACPI_TABLE_MPST, Table))->PowerNodeCount;
|
|
|
edb6c78 |
+ Tmp16 = (ACPI_CAST_PTR (ACPI_TABLE_MPST, Table))->PowerNodeCount;
|
|
|
edb6c78 |
+ SubtableCount = AcpiUtReadUint16(&Tmp16);
|
|
|
edb6c78 |
Subtable0 = ACPI_ADD_PTR (ACPI_MPST_POWER_NODE, Table, Offset);
|
|
|
edb6c78 |
|
|
|
edb6c78 |
- while ((Offset < Table->Length) && SubtableCount)
|
|
|
edb6c78 |
+ while ((Offset < TableLength) && SubtableCount)
|
|
|
edb6c78 |
{
|
|
|
edb6c78 |
AcpiOsPrintf ("\n");
|
|
|
edb6c78 |
- Status = AcpiDmDumpTable (Table->Length, Offset, Subtable0,
|
|
|
edb6c78 |
+ Status = AcpiDmDumpTable (TableLength, Offset, Subtable0,
|
|
|
edb6c78 |
sizeof (ACPI_MPST_POWER_NODE), AcpiDmTableInfoMpst0);
|
|
|
edb6c78 |
if (ACPI_FAILURE (Status))
|
|
|
edb6c78 |
{
|
|
|
edb6c78 |
@@ -861,8 +864,8 @@ AcpiDmDumpMpst (
|
|
|
edb6c78 |
|
|
|
edb6c78 |
/* Extract the sub-subtable counts */
|
|
|
edb6c78 |
|
|
|
edb6c78 |
- PowerStateCount = Subtable0->NumPowerStates;
|
|
|
edb6c78 |
- ComponentCount = Subtable0->NumPhysicalComponents;
|
|
|
edb6c78 |
+ PowerStateCount = AcpiUtReadUint16(&Subtable0->NumPowerStates);
|
|
|
edb6c78 |
+ ComponentCount = AcpiUtReadUint16(&Subtable0->NumPhysicalComponents);
|
|
|
edb6c78 |
Offset += sizeof (ACPI_MPST_POWER_NODE);
|
|
|
edb6c78 |
|
|
|
edb6c78 |
/* Sub-subtables - Memory Power State Structure(s) */
|
|
|
edb6c78 |
@@ -873,7 +876,7 @@ AcpiDmDumpMpst (
|
|
|
edb6c78 |
while (PowerStateCount)
|
|
|
edb6c78 |
{
|
|
|
edb6c78 |
AcpiOsPrintf ("\n");
|
|
|
edb6c78 |
- Status = AcpiDmDumpTable (Table->Length, Offset, Subtable0A,
|
|
|
edb6c78 |
+ Status = AcpiDmDumpTable (TableLength, Offset, Subtable0A,
|
|
|
edb6c78 |
sizeof (ACPI_MPST_POWER_STATE), AcpiDmTableInfoMpst0A);
|
|
|
edb6c78 |
if (ACPI_FAILURE (Status))
|
|
|
edb6c78 |
{
|
|
|
edb6c78 |
@@ -896,7 +899,7 @@ AcpiDmDumpMpst (
|
|
|
edb6c78 |
|
|
|
edb6c78 |
while (ComponentCount)
|
|
|
edb6c78 |
{
|
|
|
edb6c78 |
- Status = AcpiDmDumpTable (Table->Length, Offset, Subtable0B,
|
|
|
edb6c78 |
+ Status = AcpiDmDumpTable (TableLength, Offset, Subtable0B,
|
|
|
edb6c78 |
sizeof (ACPI_MPST_COMPONENT), AcpiDmTableInfoMpst0B);
|
|
|
edb6c78 |
if (ACPI_FAILURE (Status))
|
|
|
edb6c78 |
{
|
|
|
edb6c78 |
@@ -913,22 +916,24 @@ AcpiDmDumpMpst (
|
|
|
edb6c78 |
SubtableCount--;
|
|
|
edb6c78 |
Subtable0 = ACPI_ADD_PTR (ACPI_MPST_POWER_NODE, Subtable0,
|
|
|
edb6c78 |
sizeof (ACPI_MPST_POWER_NODE) +
|
|
|
edb6c78 |
- (sizeof (ACPI_MPST_POWER_STATE) * Subtable0->NumPowerStates) +
|
|
|
edb6c78 |
- (sizeof (ACPI_MPST_COMPONENT) * Subtable0->NumPhysicalComponents));
|
|
|
edb6c78 |
+ (sizeof (ACPI_MPST_POWER_STATE) *
|
|
|
edb6c78 |
+ AcpiUtReadUint16(&Subtable0->NumPowerStates)) +
|
|
|
edb6c78 |
+ (sizeof (ACPI_MPST_COMPONENT) *
|
|
|
edb6c78 |
+ AcpiUtReadUint16(&Subtable0->NumPhysicalComponents)));
|
|
|
edb6c78 |
}
|
|
|
edb6c78 |
|
|
|
edb6c78 |
/* Subtable: Count of Memory Power State Characteristic structures */
|
|
|
edb6c78 |
|
|
|
edb6c78 |
AcpiOsPrintf ("\n");
|
|
|
edb6c78 |
Subtable1 = ACPI_CAST_PTR (ACPI_MPST_DATA_HDR, Subtable0);
|
|
|
edb6c78 |
- Status = AcpiDmDumpTable (Table->Length, Offset, Subtable1,
|
|
|
edb6c78 |
+ Status = AcpiDmDumpTable (TableLength, Offset, Subtable1,
|
|
|
edb6c78 |
sizeof (ACPI_MPST_DATA_HDR), AcpiDmTableInfoMpst1);
|
|
|
edb6c78 |
if (ACPI_FAILURE (Status))
|
|
|
edb6c78 |
{
|
|
|
edb6c78 |
return;
|
|
|
edb6c78 |
}
|
|
|
edb6c78 |
|
|
|
edb6c78 |
- SubtableCount = Subtable1->CharacteristicsCount;
|
|
|
edb6c78 |
+ SubtableCount = AcpiUtReadUint16(&Subtable1->CharacteristicsCount);
|
|
|
edb6c78 |
Offset += sizeof (ACPI_MPST_DATA_HDR);
|
|
|
edb6c78 |
|
|
|
edb6c78 |
/* Subtable: Memory Power State Characteristics structure(s) */
|
|
|
edb6c78 |
@@ -936,10 +941,10 @@ AcpiDmDumpMpst (
|
|
|
edb6c78 |
Subtable2 = ACPI_ADD_PTR (ACPI_MPST_POWER_DATA, Subtable1,
|
|
|
edb6c78 |
sizeof (ACPI_MPST_DATA_HDR));
|
|
|
edb6c78 |
|
|
|
edb6c78 |
- while ((Offset < Table->Length) && SubtableCount)
|
|
|
edb6c78 |
+ while ((Offset < TableLength) && SubtableCount)
|
|
|
edb6c78 |
{
|
|
|
edb6c78 |
AcpiOsPrintf ("\n");
|
|
|
edb6c78 |
- Status = AcpiDmDumpTable (Table->Length, Offset, Subtable2,
|
|
|
edb6c78 |
+ Status = AcpiDmDumpTable (TableLength, Offset, Subtable2,
|
|
|
edb6c78 |
sizeof (ACPI_MPST_POWER_DATA), AcpiDmTableInfoMpst2);
|
|
|
edb6c78 |
if (ACPI_FAILURE (Status))
|
|
|
edb6c78 |
{
|
|
|
edb6c78 |
Index: acpica-unix2-20200925/source/compiler/dttable2.c
|
|
|
edb6c78 |
===================================================================
|
|
|
edb6c78 |
--- acpica-unix2-20200925.orig/source/compiler/dttable2.c
|
|
|
edb6c78 |
+++ acpica-unix2-20200925/source/compiler/dttable2.c
|
|
|
edb6c78 |
@@ -345,7 +345,7 @@ DtCompileMpst (
|
|
|
edb6c78 |
DtPushSubtable (Subtable);
|
|
|
edb6c78 |
|
|
|
edb6c78 |
MpstChannelInfo = ACPI_CAST_PTR (ACPI_MPST_CHANNEL, Subtable->Buffer);
|
|
|
edb6c78 |
- SubtableCount = MpstChannelInfo->PowerNodeCount;
|
|
|
edb6c78 |
+ SubtableCount = AcpiUtReadUint16(&MpstChannelInfo->PowerNodeCount);
|
|
|
edb6c78 |
|
|
|
edb6c78 |
while (*PFieldList && SubtableCount)
|
|
|
edb6c78 |
{
|
|
|
edb6c78 |
@@ -363,8 +363,8 @@ DtCompileMpst (
|
|
|
edb6c78 |
DtPushSubtable (Subtable);
|
|
|
edb6c78 |
|
|
|
edb6c78 |
MpstPowerNode = ACPI_CAST_PTR (ACPI_MPST_POWER_NODE, Subtable->Buffer);
|
|
|
edb6c78 |
- PowerStateCount = MpstPowerNode->NumPowerStates;
|
|
|
edb6c78 |
- ComponentCount = MpstPowerNode->NumPhysicalComponents;
|
|
|
edb6c78 |
+ PowerStateCount = AcpiUtReadUint16(&MpstPowerNode->NumPowerStates);
|
|
|
edb6c78 |
+ ComponentCount = AcpiUtReadUint16(&MpstPowerNode->NumPhysicalComponents);
|
|
|
edb6c78 |
|
|
|
edb6c78 |
ParentTable = DtPeekSubtable ();
|
|
|
edb6c78 |
|
|
|
edb6c78 |
@@ -417,7 +417,7 @@ DtCompileMpst (
|
|
|
edb6c78 |
DtPushSubtable (Subtable);
|
|
|
edb6c78 |
|
|
|
edb6c78 |
MpstDataHeader = ACPI_CAST_PTR (ACPI_MPST_DATA_HDR, Subtable->Buffer);
|
|
|
edb6c78 |
- SubtableCount = MpstDataHeader->CharacteristicsCount;
|
|
|
edb6c78 |
+ SubtableCount = AcpiUtReadUint16(&MpstDataHeader->CharacteristicsCount);
|
|
|
edb6c78 |
|
|
|
edb6c78 |
ParentTable = DtPeekSubtable ();
|
|
|
edb6c78 |
|