From: "Kinney, Michael D" <michael.d.kinney@intel.com>
To: Ard Biesheuvel <ard.biesheuvel@linaro.org>,
"Kinney, Michael D" <michael.d.kinney@intel.com>
Cc: "edk2-devel@lists.01.org" <edk2-devel@lists.01.org>,
Leif Lindholm <leif.lindholm@linaro.org>,
"Gao, Liming" <liming.gao@intel.com>,
"Lorenzo Pieralisi" <lorenzo.pieralisi@arm.com>
Subject: Re: [PATCH] MdePkg/IndustryStandard: add definitions for ACPI 6.0 IORT
Date: Tue, 18 Apr 2017 23:51:27 +0000 [thread overview]
Message-ID: <E92EE9817A31E24EB0585FDF735412F57D16A9AC@ORSMSX113.amr.corp.intel.com> (raw)
In-Reply-To: <CAKv+Gu8Xjzc=pk3vm0h6UcCe9QxEuPfdYTKp1M79-FAaMfw9rg@mail.gmail.com>
Ard,
I found IORT signature defined in Acpi60.h and Acpi61.h, so no
changes required there.
///
/// "IORT" I/O Remapping Table
///
#define EFI_ACPI_6_1_IO_REMAPPING_TABLE_SIGNATURE SIGNATURE_32('I', 'O', 'R', 'T')
I do see some other structures in Acpixx.h that have commented out fields
for the same reason you provided, so the commented out fields look ok.
Acpixx.h uses // style comments instead of /* */ style.
Are any of these structures defined in the ACPI 6.0 Spec? Or are they
only defined in the ARM spec. If any of them are in the ACPI 6.0 Spec,
then those should go into the Acpi60.h and Acpi61.h files.
Mike
> -----Original Message-----
> From: Ard Biesheuvel [mailto:ard.biesheuvel@linaro.org]
> Sent: Tuesday, April 18, 2017 3:21 PM
> To: Kinney, Michael D <michael.d.kinney@intel.com>
> Cc: edk2-devel@lists.01.org; Leif Lindholm <leif.lindholm@linaro.org>; Gao,
> Liming <liming.gao@intel.com>; Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
> Subject: Re: [PATCH] MdePkg/IndustryStandard: add definitions for ACPI 6.0 IORT
>
> On 18 April 2017 at 21:54, Kinney, Michael D <michael.d.kinney@intel.com> wrote:
> > Ard,
> >
> > 1) Please add the link to the document in the file header
>
> OK
>
> > 2) Why are there commented out /* */ fields in 2 of the structures?
>
> Those are subtables/subfields that may legally occur 0 or more times,
> so they should not be accounted for in the size of the respective
> structures. So they are essentially for documentation purposes only,
> and when used in .aslc code, the table should be composed using an
> ad-hoc struct typedef that concatenates them together.
>
> > 3) Why isn’t there a #define for the table signature?
> >
>
> OK, I will add that.
>
> --
> Ard.
>
>
> >> -----Original Message-----
> >> From: Ard Biesheuvel [mailto:ard.biesheuvel@linaro.org]
> >> Sent: Tuesday, April 18, 2017 10:57 AM
> >> To: edk2-devel@lists.01.org; Leif Lindholm <leif.lindholm@linaro.org>; Kinney,
> >> Michael D <michael.d.kinney@intel.com>; Gao, Liming <liming.gao@intel.com>
> >> Cc: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>; Ard Biesheuvel
> >> <ard.biesheuvel@linaro.org>
> >> Subject: Re: [PATCH] MdePkg/IndustryStandard: add definitions for ACPI 6.0
> IORT
> >>
> >> On 13 April 2017 at 17:37, Ard Biesheuvel <ard.biesheuvel@linaro.org> wrote:
> >> > This adds #defines and struct typedefs for the various node types in
> >> > the ACPI 6.0 IO Remapping Table (IORT).
> >> >
> >> > Contributed-under: TianoCore Contribution Agreement 1.0
> >> > Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> >> > ---
> >> > MdePkg/Include/IndustryStandard/IoRemappingTable.h | 187
> ++++++++++++++++++++
> >> > 1 file changed, 187 insertions(+)
> >> >
> >>
> >> Liming, Michael,
> >>
> >> Are there any concerns with this patch?
> >>
> >> Thanks,
> >> Ard.
> >>
> >>
> >> > diff --git a/MdePkg/Include/IndustryStandard/IoRemappingTable.h
> >> b/MdePkg/Include/IndustryStandard/IoRemappingTable.h
> >> > new file mode 100644
> >> > index 000000000000..674cb611961d
> >> > --- /dev/null
> >> > +++ b/MdePkg/Include/IndustryStandard/IoRemappingTable.h
> >> > @@ -0,0 +1,187 @@
> >> > +/** @file
> >> > + ACPI IO Remapping Table (IORT) as specified in ARM spec DEN0049B
> >> > +
> >> > + Copyright (c) 2017, Linaro 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
> >> > + 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.
> >> > +**/
> >> > +
> >> > +#ifndef __IO_REMAPPING_TABLE_H__
> >> > +#define __IO_REMAPPING_TABLE_H__
> >> > +
> >> > +#include <IndustryStandard/Acpi.h>
> >> > +
> >> > +#define EFI_ACPI_IO_REMAPPING_TABLE_REVISION 0x0
> >> > +
> >> > +#define EFI_ACPI_IORT_TYPE_ITS_GROUP 0x0
> >> > +#define EFI_ACPI_IORT_TYPE_NAMED_COMP 0x1
> >> > +#define EFI_ACPI_IORT_TYPE_ROOT_COMPLEX 0x2
> >> > +#define EFI_ACPI_IORT_TYPE_SMMUv1v2 0x3
> >> > +#define EFI_ACPI_IORT_TYPE_SMMUv3 0x4
> >> > +
> >> > +#define EFI_ACPI_IORT_MEM_ACCESS_PROP_CCA BIT0
> >> > +
> >> > +#define EFI_ACPI_IORT_MEM_ACCESS_PROP_AH_TR BIT0
> >> > +#define EFI_ACPI_IORT_MEM_ACCESS_PROP_AH_WA BIT1
> >> > +#define EFI_ACPI_IORT_MEM_ACCESS_PROP_AH_RA BIT2
> >> > +#define EFI_ACPI_IORT_MEM_ACCESS_PROP_AH_AHO BIT3
> >> > +
> >> > +#define EFI_ACPI_IORT_MEM_ACCESS_FLAGS_CPM BIT0
> >> > +#define EFI_ACPI_IORT_MEM_ACCESS_FLAGS_DACS BIT1
> >> > +
> >> > +#define EFI_ACPI_IORT_SMMUv1v2_MODEL_v1 0x0
> >> > +#define EFI_ACPI_IORT_SMMUv1v2_MODEL_v2 0x1
> >> > +#define EFI_ACPI_IORT_SMMUv1v2_MODEL_MMU400 0x2
> >> > +#define EFI_ACPI_IORT_SMMUv1v2_MODEL_MMU500 0x3
> >> > +
> >> > +#define EFI_ACPI_IORT_SMMUv1v2_FLAG_DVM BIT0
> >> > +#define EFI_ACPI_IORT_SMMUv1v2_FLAG_COH_WALK BIT1
> >> > +
> >> > +#define EFI_ACPI_IORT_SMMUv1v2_INT_FLAG_LEVEL 0x0
> >> > +#define EFI_ACPI_IORT_SMMUv1v2_INT_FLAG_EDGE 0x1
> >> > +
> >> > +#define EFI_ACPI_IORT_SMMUv3_FLAG_COHAC_OVERRIDE BIT0
> >> > +#define EFI_ACPI_IORT_SMMUv3_FLAG_HTTU_OVERRIDE BIT1
> >> > +
> >> > +#define EFI_ACPI_IORT_ROOT_COMPLEX_ATS_UNSUPPORTED 0x0
> >> > +#define EFI_ACPI_IORT_ROOT_COMPLEX_ATS_SUPPORTED 0x1
> >> > +
> >> > +#define EFI_ACPI_IORT_ID_MAPPING_FLAGS_SINGLE BIT0
> >> > +
> >> > +#pragma pack(1)
> >> > +
> >> > +//
> >> > +// Table header
> >> > +//
> >> > +typedef struct {
> >> > + EFI_ACPI_DESCRIPTION_HEADER Header;
> >> > + INT32 NumNodes;
> >> > + INT32 NodeOffset;
> >> > + INT32 Reserved;
> >> > +} EFI_ACPI_6_0_IO_REMAPPING_TABLE;
> >> > +
> >> > +//
> >> > +// Definition for ID mapping table shared by all node types
> >> > +//
> >> > +typedef struct {
> >> > + UINT32 InputBase;
> >> > + UINT32 NumIds;
> >> > + UINT32 OutputBase;
> >> > + UINT32 OutputReference;
> >> > + UINT32 Flags;
> >> > +} EFI_ACPI_6_0_IO_REMAPPING_ID_TABLE;
> >> > +
> >> > +//
> >> > +// Header definition shared by all node types
> >> > +//
> >> > +typedef struct {
> >> > + UINT8 Type;
> >> > + UINT16 Length;
> >> > + UINT8 Revision;
> >> > + UINT32 Reserved;
> >> > + UINT32 NumIdMappings;
> >> > + UINT32 IdReference;
> >> > +} EFI_ACPI_6_0_IO_REMAPPING_NODE;
> >> > +
> >> > +//
> >> > +// Node type 0: ITS node
> >> > +//
> >> > +typedef struct {
> >> > + EFI_ACPI_6_0_IO_REMAPPING_NODE Node;
> >> > +
> >> > + UINT32 NumIts;
> >> > +/*
> >> > + UINT32 ItsIdentifiers[0];
> >> > +*/
> >> > +} EFI_ACPI_6_0_IO_REMAPPING_ITS_NODE;
> >> > +
> >> > +//
> >> > +// Node type 1: root complex node
> >> > +//
> >> > +typedef struct {
> >> > + EFI_ACPI_6_0_IO_REMAPPING_NODE Node;
> >> > +
> >> > + UINT32 CacheCoherent;
> >> > + UINT8 AllocationHints;
> >> > + UINT16 Reserved;
> >> > + UINT8 MemoryAccessFlags;
> >> > +
> >> > + UINT32 AtsAttribute;
> >> > + UINT32 PciSegmentNumber;
> >> > +} EFI_ACPI_6_0_IO_REMAPPING_RC_NODE;
> >> > +
> >> > +//
> >> > +// Node type 2: named component node
> >> > +//
> >> > +typedef struct {
> >> > + EFI_ACPI_6_0_IO_REMAPPING_NODE Node;
> >> > +
> >> > + UINT32 Flags;
> >> > + UINT32 CacheCoherent;
> >> > + UINT8 AllocationHints;
> >> > + UINT16 Reserved;
> >> > + UINT8 MemoryAccessFlags;
> >> > + UINT8 AddressSizeLimit;
> >> > +/*
> >> > + CHAR8 ObjectName[0];
> >> > +*/
> >> > +} EFI_ACPI_6_0_IO_REMAPPING_NAMED_COMP_NODE;
> >> > +
> >> > +//
> >> > +// Node type 3: SMMUv1 or SMMUv2 node
> >> > +//
> >> > +typedef struct {
> >> > + UINT32 Interrupt;
> >> > + UINT32 InterruptFlags;
> >> > +} EFI_ACPI_6_0_IO_REMAPPING_SMMU_INT;
> >> > +
> >> > +typedef struct {
> >> > + EFI_ACPI_6_0_IO_REMAPPING_NODE Node;
> >> > +
> >> > + UINT64 Base;
> >> > + UINT64 Span;
> >> > + UINT32 Model;
> >> > + UINT32 Flags;
> >> > + UINT32 GlobalInterruptArrayRef;
> >> > + UINT32 NumContextInterrupts;
> >> > + UINT32 ContextInterruptArrayRef;
> >> > + UINT32 NumPmuInterrupts;
> >> > + UINT32 PmuInterruptArrayRef;
> >> > +
> >> > + UINT32 SMMU_NSgIrpt;
> >> > + UINT32 SMMU_NSgIrptFlags;
> >> > + UINT32 SMMU_NSgCfgIrpt;
> >> > + UINT32 SMMU_NSgCfgIrptFlags;
> >> > +
> >> > +/*
> >> > + EFI_ACPI_6_0_IO_REMAPPING_SMMU_CTX_INT ContextInterrupt[0];
> >> > + EFI_ACPI_6_0_IO_REMAPPING_SMMU_CTX_INT PmuInterrupt[0];
> >> > +*/
> >> > +} EFI_ACPI_6_0_IO_REMAPPING_SMMU_NODE;
> >> > +
> >> > +//
> >> > +// Node type 4: SMMUv4 node
> >> > +//
> >> > +typedef struct {
> >> > + EFI_ACPI_6_0_IO_REMAPPING_NODE Node;
> >> > +
> >> > + UINT64 Base;
> >> > + UINT32 Flags;
> >> > + UINT32 Reserved;
> >> > + UINT64 VatosAddress;
> >> > + UINT32 Model;
> >> > + UINT32 Event;
> >> > + UINT32 Pri;
> >> > + UINT32 Gerr;
> >> > + UINT32 Sync;
> >> > +} EFI_ACPI_6_0_IO_REMAPPING_SMMU3_NODE;
> >> > +
> >> > +#pragma pack()
> >> > +
> >> > +#endif
> >> > --
> >> > 2.9.3
> >> >
next prev parent reply other threads:[~2017-04-18 23:51 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-04-13 16:37 [PATCH] MdePkg/IndustryStandard: add definitions for ACPI 6.0 IORT Ard Biesheuvel
2017-04-18 10:07 ` Alexei Fedorov
2017-04-18 10:13 ` Ard Biesheuvel
2017-04-18 11:01 ` Alexei Fedorov
2017-04-18 11:11 ` Ard Biesheuvel
2017-04-18 17:56 ` Ard Biesheuvel
2017-04-18 20:54 ` Kinney, Michael D
2017-04-18 22:21 ` Ard Biesheuvel
2017-04-18 23:51 ` Kinney, Michael D [this message]
2017-04-19 7:03 ` Ard Biesheuvel
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=E92EE9817A31E24EB0585FDF735412F57D16A9AC@ORSMSX113.amr.corp.intel.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