<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40"> <head> <meta http-equiv="Content-Type" content="text/html; charset=utf-8"> <meta name="Generator" content="Microsoft Word 15 (filtered medium)"> <style></style> </head> <body lang="EN-GB" link="blue" vlink="purple" style="word-wrap:break-word"> <div class="WordSection1"> Hi Rohit,<o:p></o:p> <o:p> </o:p> Please send an updated series with the issues fixed.<o:p></o:p> <o:p> </o:p> Regards,<o:p></o:p> <o:p> </o:p> Sami Mujawar<o:p></o:p> <o:p> </o:p> <div style="border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0cm 0cm 0cm"> From: Rohit Mathew <Rohit.Mathew@arm.com> Date: Monday, 22 May 2023 at 14:08 To: Sami Mujawar <Sami.Mujawar@arm.com>, "devel@edk2.groups.io" <devel@edk2.groups.io> Cc: Michael D Kinney <michael.d.kinney@intel.com>, Liming Gao <gaoliming@byosoft.com.cn>, Thomas Abraham <thomas.abraham@arm.com>, James Morse <James.Morse@arm.com>, nd <nd@arm.com> Subject: RE: [PATCH V2 1/3] MdePkg/IndustryStandard: add definitions for MPAM ACPI specification<o:p></o:p> </div> <div> <o:p> </o:p> </div> Hi Sami,<o:p></o:p> <o:p> </o:p> Thank you for the review. Please find my comments inline.<o:p></o:p> <o:p> </o:p> Regards,<o:p></o:p> Rohit<o:p></o:p> <div style="border:none;border-left:solid blue 1.5pt;padding:0cm 0cm 0cm 4.0pt"> <div> <div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0cm 0cm 0cm"> From: Sami Mujawar <Sami.Mujawar@arm.com> Sent: Monday, May 22, 2023 1:17 PM To: Rohit Mathew <Rohit.Mathew@arm.com>; devel@edk2.groups.io Cc: Michael D Kinney <michael.d.kinney@intel.com>; Liming Gao <gaoliming@byosoft.com.cn>; Thomas Abraham <thomas.abraham@arm.com>; James Morse <James.Morse@arm.com>; nd <nd@arm.com> Subject: Re: [PATCH V2 1/3] MdePkg/IndustryStandard: add definitions for MPAM ACPI specification<o:p></o:p> </div> </div> <o:p> </o:p> Hi Rohit,<o:p></o:p> Please find my response inline marked [SAMI].<o:p></o:p> Regards,<o:p></o:p> Sami Mujawar<o:p></o:p> <div> On 18/05/2023 04:06 pm, Rohit Mathew wrote:<o:p></o:p> </div> <blockquote style="margin-top:5.0pt;margin-bottom:5.0pt"> <pre>From: Rohit Mathew <a href="mailto:rohit.mathew@arm.com"><rohit.mathew@arm.com></a><o:p></o:p></pre> <pre><o:p> </o:p></pre> <pre>add definitions, macros and types for elements associated with MPAM<o:p></o:p></pre> <pre>ACPI 2.0 specification.<o:p></o:p></pre> <pre><o:p> </o:p></pre> <pre>Signed-off-by: Rohit Mathew <a href="mailto:rohit.mathew@arm.com"><rohit.mathew@arm.com></a><o:p></o:p></pre> <pre>---<o:p></o:p></pre> <pre> MdePkg/Include/IndustryStandard/Acpi65.h | 7 +-<o:p></o:p></pre> <pre> MdePkg/Include/IndustryStandard/Mpam.h | 258 ++++++++++++++++++++<o:p></o:p></pre> <pre> 2 files changed, 264 insertions(+), 1 deletion(-)<o:p></o:p></pre> <pre><o:p> </o:p></pre> <pre>diff --git a/MdePkg/Include/IndustryStandard/Acpi65.h b/MdePkg/Include/IndustryStandard/Acpi65.h<o:p></o:p></pre> <pre>index 1e41ae9a27..8a1d3d125a 100644<o:p></o:p></pre> <pre>--- a/MdePkg/Include/IndustryStandard/Acpi65.h<o:p></o:p></pre> <pre>+++ b/MdePkg/Include/IndustryStandard/Acpi65.h<o:p></o:p></pre> <pre>@@ -2,7 +2,7 @@<o:p></o:p></pre> <pre> ACPI 6.5 definitions from the ACPI Specification Revision 6.5 Aug, 2022.<o:p></o:p></pre> <pre> <o:p></o:p></pre> <pre> Copyright (c) 2017 - 2022, Intel Corporation. All rights reserved. <o:p></o:p></pre> <pre>- Copyright (c) 2019 - 2021, ARM Ltd. All rights reserved. <o:p></o:p></pre> <pre>+ Copyright (c) 2019 - 2023, ARM Ltd. All rights reserved. <o:p></o:p></pre> <pre> Copyright (c) 2023, Loongson Technology Corporation Limited. All rights reserved. <o:p></o:p></pre> <pre> <o:p></o:p></pre> <pre> SPDX-License-Identifier: BSD-2-Clause-Patent<o:p></o:p></pre> <pre>@@ -3251,6 +3251,11 @@ typedef struct {<o:p></o:p></pre> <pre> ///<o:p></o:p></pre> <pre> #define EFI_ACPI_6_5_XEN_PROJECT_TABLE_SIGNATURE SIGNATURE_32('X', 'E', 'N', 'V')<o:p></o:p></pre> <pre> <o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+/// "MPAM" Memory System Resource Partitioning and Monitoring Table<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+#define EFI_ACPI_MEMORY_SYSTEM_RESOURCE_PARTITIONING_AND_MONITORING_TABLE_SIGNATURE SIGNATURE_32('M', 'P', 'A', 'M')<o:p></o:p></pre> <pre>+<o:p></o:p></pre> <pre> #pragma pack()<o:p></o:p></pre> <pre> <o:p></o:p></pre> <pre> #endif<o:p></o:p></pre> <pre>diff --git a/MdePkg/Include/IndustryStandard/Mpam.h b/MdePkg/Include/IndustryStandard/Mpam.h<o:p></o:p></pre> <pre>new file mode 100644<o:p></o:p></pre> <pre>index 0000000000..5d4c466abb<o:p></o:p></pre> <pre>--- /dev/null<o:p></o:p></pre> <pre>+++ b/MdePkg/Include/IndustryStandard/Mpam.h<o:p></o:p></pre> <pre>@@ -0,0 +1,258 @@<o:p></o:p></pre> <pre>+/** @file<o:p></o:p></pre> <pre>+ ACPI for Memory System Resource Partitioning and Monitoring 2.0 (MPAM) as<o:p></o:p></pre> <pre>+ specified in ARM spec DEN0065<o:p></o:p></pre> <pre>+<o:p></o:p></pre> <pre>+ Copyright (c) 2023, Arm Limited. All rights reserved.<o:p></o:p></pre> <pre>+<o:p></o:p></pre> <pre>+ SPDX-License-Identifier: BSD-2-Clause-Patent<o:p></o:p></pre> <pre>+<o:p></o:p></pre> <pre>+ @par Specification Reference:<o:p></o:p></pre> <pre>+ - [1] ACPI for Memory System Resource Partitioning and Monitoring 2.0<o:p></o:p></pre> <pre>+ (<a href="https://developer.arm.com/documentation/den0065/latest">https://developer.arm.com/documentation/den0065/latest</a>)<o:p></o:p></pre> <pre>+<o:p></o:p></pre> <pre>+ @par Glossary:<o:p></o:p></pre> <pre>+ - MPAM - Memory System Resource Partitioning And Monitoring<o:p></o:p></pre> <pre>+ - MSC - Memory System Component<o:p></o:p></pre> <pre>+ - PCC - Platform Communication Channel<o:p></o:p></pre> <pre>+ - RIS - Resource Instance Selection<o:p></o:p></pre> <pre>+ - SMMU - Arm System Memory Management Unit<o:p></o:p></pre> <pre>+ **/<o:p></o:p></pre> <pre>+<o:p></o:p></pre> <pre>+#ifndef MPAM_H_<o:p></o:p></pre> <pre>+#define MPAM_H_<o:p></o:p></pre> <pre>+<o:p></o:p></pre> <pre>+#include <IndustryStandard/Acpi.h><o:p></o:p></pre> <pre>+<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+/// MPAM Revision<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+#define EFI_ACPI_MEMORY_SYSTEM_RESOURCE_PARTITIONING_AND_MONITORING_TABLE_REVISION (0x01)<o:p></o:p></pre> <pre>+<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+/// MPAM Interrupt mode<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+#define EFI_ACPI_MPAM_INTERRUPT_LEVEL_TRIGGERED (0x0)<o:p></o:p></pre> <pre>+#define EFI_ACPI_MPAM_INTERRUPT_EDGE_TRIGGERED (0x1)<o:p></o:p></pre> <pre>+<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+/// MPAM Interrupt type<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+#define EFI_ACPI_MPAM_INTERRUPT_WIRED (0x0)<o:p></o:p></pre> <pre>+<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+/// MPAM Interrupt affinity type<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+#define EFI_ACPI_MPAM_INTERRUPT_PROCESSOR_AFFINITY (0x0)<o:p></o:p></pre> <pre>+#define EFI_ACPI_MPAM_INTERRUPT_PROCESSOR_CONTAINER_AFFINITY (0x1)<o:p></o:p></pre> <pre>+<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+/// MPAM MSC affinity valid<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+#define EFI_ACPI_MPAM_INTERRUPT_AFFINITY_NOT_VALID (0x0)<o:p></o:p></pre> <pre>+#define EFI_ACPI_MPAM_INTERRUPT_AFFINITY_VALID (0x1)<o:p></o:p></pre> <pre>+<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+/// MPAM Interrupt flag - bit positions<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+#define EFI_ACPI_MPAM_INTERRUPT_MODE_SHIFT (0)<o:p></o:p></pre> <pre>+#define EFI_ACPI_MPAM_INTERRUPT_TYPE_SHIFT (1)<o:p></o:p></pre> <pre>+#define EFI_ACPI_MPAM_INTERRUPT_AFFINITY_TYPE_SHIFT (3)<o:p></o:p></pre> <pre>+#define EFI_ACPI_MPAM_INTERRUPT_AFFINITY_VALID_SHIFT (4)<o:p></o:p></pre> <pre>+#define EFI_ACPI_MPAM_INTERRUPT_RESERVED_SHIFT (5)<o:p></o:p></pre> <pre>+<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+/// MPAM Interrupt flag - bit masks<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+#define EFI_ACPI_MPAM_INTERRUPT_MODE_MASK (0x1)<o:p></o:p></pre> <pre>+#define EFI_ACPI_MPAM_INTERRUPT_TYPE_MASK (0x3)<o:p></o:p></pre> <pre>+#define EFI_ACPI_MPAM_INTERRUPT_AFFINITY_TYPE_MASK (0x8)<o:p></o:p></pre> <pre>+#define EFI_ACPI_MPAM_INTERRUPT_AFFINITY_VALID_MASK (0x10)<o:p></o:p></pre> <pre>+#define EFI_ACPI_MPAM_INTERRUPT_RESERVED_MASK (0xFFFFFFE0)<o:p></o:p></pre> <pre>+<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+/// MPAM_MEMORY_LOCATOR_RESERVED_MASK should be used along with an 8 byte object<o:p></o:p></pre> <pre>+/// starting at the base of the locator field.<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+#define EFI_ACPI_MPAM_MEM_LOCATOR_RESERVED_FIELD_MASK (0x00FFFFFFFFFFFFFFULL)<o:p></o:p></pre> <pre>+#define EFI_ACPI_MPAM_MEM_LOCATOR_RESERVED_FIELD_SHIFT (0)<o:p></o:p></pre> <pre>+<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+/// MPAM_MEMORY_LOCATOR_LEVEL_MASK should be used along with an 8 byte object<o:p></o:p></pre> <pre>+/// starting at the base of the locator field.<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+#define EFI_ACPI_MPAM_MEM_LOCATOR_LEVEL_FIELD_MASK (0xFF00000000000000ULL)<o:p></o:p></pre> <pre>+#define EFI_ACPI_MPAM_MEM_LOCATOR_LEVEL_FIELD_SHIFT (56)<o:p></o:p></pre> <pre>+<o:p></o:p></pre> </blockquote> [SAMI] Can you point me to the section in the spec where I can find this information, please? Are these definitions for Memory-side cache locator descriptor by any chance? [Rohit] These are from mem cache locator field, like you pointed (Section 2.3.3.4 – table 16). I’ll rename these to EFI_ACPI_MPAM_MEM_CACHE_* if the masks and shifts look fine.<o:p></o:p> <blockquote style="margin-top:5.0pt;margin-bottom:5.0pt"> <pre><o:p> </o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+/// MPAM Location types<o:p></o:p></pre> <pre>+/// as described in document [1], table 11<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+#define EFI_ACPI_MPAM_LOCATION_PROCESSOR_CACHE (0x0)<o:p></o:p></pre> <pre>+#define EFI_ACPI_MPAM_LOCATION_MEMORY (0x1)<o:p></o:p></pre> <pre>+#define EFI_ACPI_MPAM_LOCATION_SMMU (0x2)<o:p></o:p></pre> <pre>+#define EFI_ACPI_MPAM_LOCATION_MEMORY_CACHE (0x3)<o:p></o:p></pre> <pre>+#define EFI_ACPI_MPAM_LOCATION_ACPI_DEVICE (0x4)<o:p></o:p></pre> <pre>+#define EFI_ACPI_MPAM_LOCATION_INTERCONNECT (0x5)<o:p></o:p></pre> <pre>+#define EFI_ACPI_MPAM_LOCATION_UNKNOWN (0xFF)<o:p></o:p></pre> <pre>+<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+/// MPAM Interface types<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+#define EFI_ACPI_MPAM_INTERFACE_MMIO (0x0)<o:p></o:p></pre> <pre>+#define EFI_ACPI_MPAM_INTERFACE_PCC (0x1)<o:p></o:p></pre> <pre>+<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+/// MPAM Link types<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+#define EFI_ACPI_MPAM_LINK_TYPE_NUMA (0x0)<o:p></o:p></pre> <pre>+#define EFI_ACPI_MPAM_LINK_TYPE_PROC (0x0A)<o:p></o:p></pre> </blockquote> [SAMI] Is the EFI_ACPI_MPAM_LINK_TYPE_PROC values and EFI_ACPI_MPAM_INTERFACE_PCC value swapped by any chance? Or I am looking at a completely wrong spec. <o:p></o:p> Can you point me to the correct spec, please?<o:p></o:p> [/SAMI]<o:p></o:p> [Rohit] Ah! That indeed got swapped around. Let me fix this in the next revision. Do you want me to send out a new revision for this patch alone (or I can hold until the series is reviewed). PS: We are both looking at the spec spec revision.<o:p></o:p> [/Rohit]<o:p></o:p> <blockquote style="margin-top:5.0pt;margin-bottom:5.0pt"> <pre><o:p> </o:p></pre> <pre>+<o:p></o:p></pre> <pre>+#pragma pack(1)<o:p></o:p></pre> <pre>+<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+/// MPAM MSC generic locator descriptor<o:p></o:p></pre> <pre>+/// as described in document [1], table 12<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+typedef struct {<o:p></o:p></pre> <pre>+ UINT64 Descriptor1;<o:p></o:p></pre> <pre>+ UINT32 Descriptor2;<o:p></o:p></pre> <pre>+} EFI_ACPI_MPAM_GENERIC_LOCATOR;<o:p></o:p></pre> <pre>+<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+/// MPAM processor cache locator descriptor<o:p></o:p></pre> <pre>+/// as described in document [1], table 13<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+typedef struct {<o:p></o:p></pre> <pre>+ UINT64 CacheReference;<o:p></o:p></pre> <pre>+ UINT32 Reserved;<o:p></o:p></pre> <pre>+} EFI_ACPI_MPAM_CACHE_LOCATOR;<o:p></o:p></pre> <pre>+<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+/// MPAM memory locator descriptor<o:p></o:p></pre> <pre>+/// as described in document [1], table 14<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+typedef struct {<o:p></o:p></pre> <pre>+ UINT64 ProximityDomain;<o:p></o:p></pre> <pre>+ UINT32 Reserved;<o:p></o:p></pre> <pre>+} EFI_ACPI_MPAM_MEMORY_LOCATOR;<o:p></o:p></pre> <pre>+<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+/// MPAM SMMU locator descriptor<o:p></o:p></pre> <pre>+/// as described in document [1], table 15<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+typedef struct {<o:p></o:p></pre> <pre>+ UINT64 SmmuInterface;<o:p></o:p></pre> <pre>+ UINT32 Reserved;<o:p></o:p></pre> <pre>+} EFI_ACPI_MPAM_SMMU_LOCATOR;<o:p></o:p></pre> <pre>+<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+/// MPAM memory-side cache locator descriptor<o:p></o:p></pre> <pre>+/// as described in Document [1], table 16<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+typedef struct {<o:p></o:p></pre> <pre>+ UINT8 Reserved[7];<o:p></o:p></pre> <pre>+ UINT8 Level;<o:p></o:p></pre> <pre>+ UINT32 Reference;<o:p></o:p></pre> <pre>+} EFI_ACPI_MPAM_MEMORY_CACHE_LOCATOR;<o:p></o:p></pre> <pre>+<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+/// MPAM ACPI device locator descriptor<o:p></o:p></pre> <pre>+/// as described in document [1], table 17<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+typedef struct {<o:p></o:p></pre> <pre>+ UINT64 AcpiHardwareId;<o:p></o:p></pre> <pre>+ UINT32 AcpiUniqueId;<o:p></o:p></pre> <pre>+} EFI_ACPI_MPAM_ACPI_LOCATOR;<o:p></o:p></pre> <pre>+<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+/// MPAM interconnect locator descriptor<o:p></o:p></pre> <pre>+/// as described in document [1], table 18<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+typedef struct {<o:p></o:p></pre> <pre>+ UINT64 InterconnectDescTblOff;<o:p></o:p></pre> <pre>+ UINT32 Reserved;<o:p></o:p></pre> <pre>+} EFI_ACPI_MPAM_INTERCONNECT_LOCATOR;<o:p></o:p></pre> <pre>+<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+/// MPAM interconnect descriptor<o:p></o:p></pre> <pre>+/// as described in document [1], table 19<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+typedef struct {<o:p></o:p></pre> <pre>+ UINT32 SourceId;<o:p></o:p></pre> <pre>+ UINT32 DestinationId;<o:p></o:p></pre> <pre>+ UINT8 LinkType;<o:p></o:p></pre> <pre>+ UINT8 Reserved[3];<o:p></o:p></pre> <pre>+} EFI_ACPI_MPAM_INTERCONNECT_DESCRIPTOR;<o:p></o:p></pre> <pre>+<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+/// MPAM interconnect descriptor table<o:p></o:p></pre> <pre>+/// as described in document [1], table 20<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+typedef struct {<o:p></o:p></pre> <pre>+ UINT8 Signature[16];<o:p></o:p></pre> <pre>+ UINT32 NumDescriptors;<o:p></o:p></pre> <pre>+} EFI_ACPI_MPAM_INTERCONNECT_DESCRIPTOR_TABLE;<o:p></o:p></pre> <pre>+<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+/// MPAM resource locator<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+typedef union {<o:p></o:p></pre> <pre>+ EFI_ACPI_MPAM_CACHE_LOCATOR CacheLocator;<o:p></o:p></pre> <pre>+ EFI_ACPI_MPAM_MEMORY_LOCATOR MemoryLocator;<o:p></o:p></pre> <pre>+ EFI_ACPI_MPAM_SMMU_LOCATOR SmmuLocator;<o:p></o:p></pre> <pre>+ EFI_ACPI_MPAM_MEMORY_CACHE_LOCATOR MemCacheLocator;<o:p></o:p></pre> <pre>+ EFI_ACPI_MPAM_ACPI_LOCATOR AcpiLocator;<o:p></o:p></pre> <pre>+ EFI_ACPI_MPAM_INTERCONNECT_LOCATOR InterconnectIfcLocator;<o:p></o:p></pre> <pre>+ EFI_ACPI_MPAM_GENERIC_LOCATOR GenericLocator;<o:p></o:p></pre> <pre>+} EFI_ACPI_MPAM_LOCATOR;<o:p></o:p></pre> <pre>+<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+/// MPAM MSC node body<o:p></o:p></pre> <pre>+/// as described document [1], table 4<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+typedef struct {<o:p></o:p></pre> <pre>+ UINT16 Length;<o:p></o:p></pre> <pre>+ UINT8 InterfaceType;<o:p></o:p></pre> <pre>+ UINT8 Reserved;<o:p></o:p></pre> <pre>+ UINT32 Identifier;<o:p></o:p></pre> <pre>+ UINT64 BaseAddress;<o:p></o:p></pre> <pre>+ UINT32 MmioSize;<o:p></o:p></pre> <pre>+ UINT32 OverflowInterrupt;<o:p></o:p></pre> <pre>+ UINT32 OverflowInterruptFlags;<o:p></o:p></pre> <pre>+ UINT32 Reserved1;<o:p></o:p></pre> <pre>+ UINT32 OverflowInterruptAffinity;<o:p></o:p></pre> <pre>+ UINT32 ErrorInterrupt;<o:p></o:p></pre> <pre>+ UINT32 ErrorInterruptFlags;<o:p></o:p></pre> <pre>+ UINT32 Reserved2;<o:p></o:p></pre> <pre>+ UINT32 ErrorInterruptAffinity;<o:p></o:p></pre> <pre>+ UINT32 MaxNrdyUsec;<o:p></o:p></pre> <pre>+ UINT64 HardwareIdLinkedDevice;<o:p></o:p></pre> <pre>+ UINT32 InstanceIdLinkedDevice;<o:p></o:p></pre> <pre>+ UINT32 NumResources;<o:p></o:p></pre> <pre>+} EFI_ACPI_MPAM_MSC_NODE;<o:p></o:p></pre> <pre>+<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+/// MPAM MSC resource<o:p></o:p></pre> <pre>+/// as described in document [1], table 9<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+typedef struct {<o:p></o:p></pre> <pre>+ UINT32 Identifier;<o:p></o:p></pre> <pre>+ UINT8 RisIndex;<o:p></o:p></pre> <pre>+ UINT16 Reserved1;<o:p></o:p></pre> <pre>+ UINT8 LocatorType;<o:p></o:p></pre> <pre>+ EFI_ACPI_MPAM_LOCATOR Locator;<o:p></o:p></pre> <pre>+ UINT32 NumFunctionalDependencies;<o:p></o:p></pre> <pre>+} EFI_ACPI_MPAM_MSC_RESOURCE;<o:p></o:p></pre> <pre>+<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+/// MPAM Function dependency descriptor<o:p></o:p></pre> <pre>+/// as described in document [1], table 10<o:p></o:p></pre> <pre>+///<o:p></o:p></pre> <pre>+typedef struct {<o:p></o:p></pre> <pre>+ UINT32 Producer;<o:p></o:p></pre> <pre>+ UINT32 Reserved;<o:p></o:p></pre> <pre>+} EFI_ACPI_MPAM_FUNCTIONAL_DEPENDENCY_DESCRIPTOR;<o:p></o:p></pre> <pre>+<o:p></o:p></pre> <pre>+#pragma pack()<o:p></o:p></pre> <pre>+<o:p></o:p></pre> <pre>+#endif<o:p></o:p></pre> </blockquote> </div> </div> </body> </html> <div width="1" style="color:white;clear:both">_._,_._,_</div> <hr> Groups.io Links: You receive all messages sent to this group. <a target="_blank" href="https://edk2.groups.io/g/devel/message/105124">View/Reply Online (#105124)</a> | | <a target="_blank" href="https://groups.io/mt/98993032/1813853">Mute This Topic</a> | <a href="https://edk2.groups.io/g/devel/post">New Topic</a> <a href="https://edk2.groups.io/g/devel/editsub/1813853">Your Subscription</a> | <a href="mailto:devel+owner@edk2.groups.io">Contact Group Owner</a> | <a href="https://edk2.groups.io/g/devel/unsub">Unsubscribe</a> [edk2-devel-archive@redhat.com] <div width="1" style="color:white;clear:both">_._,_._,_</div>