public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Sheng Wei" <w.sheng@intel.com>
To: "devel@edk2.groups.io" <devel@edk2.groups.io>,
	"gaoliming@byosoft.com.cn" <gaoliming@byosoft.com.cn>,
	"Liu, Zhiguang" <zhiguang.liu@intel.com>
Cc: "Kinney, Michael D" <michael.d.kinney@intel.com>,
	"Yao, Jiewen" <jiewen.yao@intel.com>,
	"Huang, Jenny" <jenny.huang@intel.com>,
	"Kowalewski, Robert" <robert.kowalewski@intel.com>,
	"Feng, Roger" <roger.feng@intel.com>
Subject: Re: [edk2-devel] 回复: [PATCH] MdePkg/include: Add DMAR SATC Table Definition
Date: Tue, 15 Dec 2020 05:58:49 +0000	[thread overview]
Message-ID: <CY4PR11MB1928003FA115C6A51EBA7883E1C60@CY4PR11MB1928.namprd11.prod.outlook.com> (raw)
In-Reply-To: <006001d6d1c3$bcdc2420$36946c60$@byosoft.com.cn>

[-- Attachment #1: Type: text/plain, Size: 6667 bytes --]

Hi Liming,
Thank you for the "Reviewed-by".
Could you help to merge the patch to ekd2 master branch ?
Thank you.
BR
Sheng Wei

> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of
> gaoliming
> Sent: 2020年12月14日 10:49
> To: Liu, Zhiguang <zhiguang.liu@intel.com>; Sheng, W <w.sheng@intel.com>;
> devel@edk2.groups.io
> Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Yao, Jiewen
> <jiewen.yao@intel.com>; Huang, Jenny <jenny.huang@intel.com>;
> Kowalewski, Robert <robert.kowalewski@intel.com>; Feng, Roger
> <roger.feng@intel.com>
> Subject: [edk2-devel] 回复: [PATCH] MdePkg/include: Add DMAR SATC
> Table Definition
> 
> Reviewed-by: Liming Gao <gaoliming@byosoft.com.cn>
> 
> > -----邮件原件-----
> > 发件人: Liu, Zhiguang <zhiguang.liu@intel.com>
> > 发送时间: 2020年12月11日 10:51
> > 收件人: Sheng, W <w.sheng@intel.com>; devel@edk2.groups.io
> > 抄送: Kinney, Michael D <michael.d.kinney@intel.com>; Liming Gao
> > <gaoliming@byosoft.com.cn>; Yao, Jiewen <jiewen.yao@intel.com>;
> Huang,
> > Jenny <jenny.huang@intel.com>; Kowalewski, Robert
> > <robert.kowalewski@intel.com>; Feng, Roger <roger.feng@intel.com>
> > 主题: RE: [PATCH] MdePkg/include: Add DMAR SATC Table Definition
> >
> > Reviewed-by: Zhiguang Liu <zhiguang.liu@intel.com>
> >
> > > -----Original Message-----
> > > From: Sheng, W <w.sheng@intel.com>
> > > Sent: Friday, December 11, 2020 9:37 AM
> > > To: devel@edk2.groups.io
> > > Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Liming Gao
> > > <gaoliming@byosoft.com.cn>; Liu, Zhiguang <zhiguang.liu@intel.com>;
> > > Yao, Jiewen <jiewen.yao@intel.com>; Huang, Jenny
> > > <jenny.huang@intel.com>; Kowalewski, Robert
> > > <robert.kowalewski@intel.com>; Feng, Roger <roger.feng@intel.com>
> > > Subject: [PATCH] MdePkg/include: Add DMAR SATC Table Definition
> > >
> > > SoC Integrated Address Translation Cache (SATC) reporting structure
> > > is
> one
> > > of the Remapping Structure, which is imported since Intel(R)
> Virtualization
> > > Technology for Directed I/O (VT-D) Architecture Specification v3.2.
> > >
> > > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3109
> > >
> > > Signed-off-by: Sheng Wei <w.sheng@intel.com>
> > > Cc: Michael D Kinney <michael.d.kinney@intel.com>
> > > Cc: Liming Gao <gaoliming@byosoft.com.cn>
> > > Cc: Zhiguang Liu <zhiguang.liu@intel.com>
> > > Cc: Jiewen Yao <jiewen.yao@intel.com>
> > > Cc: Jenny Huang <jenny.huang@intel.com>
> > > Cc: Kowalewski Robert <robert.kowalewski@intel.com>
> > > Cc: Feng Roger <roger.feng@intel.com>
> > > ---
> > >  .../IndustryStandard/DmaRemappingReportingTable.h  | 34
> > > ++++++++++++++++++++--
> > >  1 file changed, 31 insertions(+), 3 deletions(-)
> > >
> > > diff --git
> > a/MdePkg/Include/IndustryStandard/DmaRemappingReportingTable.h
> > > b/MdePkg/Include/IndustryStandard/DmaRemappingReportingTable.h
> > > index 7c50dc972e..48f6959fec 100644
> > > ---
> a/MdePkg/Include/IndustryStandard/DmaRemappingReportingTable.h
> > > +++
> b/MdePkg/Include/IndustryStandard/DmaRemappingReportingTable.h
> > > @@ -2,13 +2,13 @@
> > >    DMA Remapping Reporting (DMAR) ACPI table definition from Intel(R)
> > >    Virtualization Technology for Directed I/O (VT-D) Architecture
> > Specification.
> > >
> > > -  Copyright (c) 2016 - 2018, Intel Corporation. All rights
> reserved.<BR>
> > > +  Copyright (c) 2016 - 2020, Intel Corporation. All rights
> reserved.<BR>
> > >    SPDX-License-Identifier: BSD-2-Clause-Patent
> > >
> > >    @par Revision Reference:
> > >      - Intel(R) Virtualization Technology for Directed I/O (VT-D)
> > Architecture
> > > -      Specification v2.5, Dated November 2017.
> > > -
> > >
> >
> http://www.intel.com/content/dam/www/public/us/en/documents/produc
> t-
> > > specifications/vt-directed-io-spec.pdf
> > > +      Specification v3.2, Dated October 2020.
> > > +
> > >
> >
> https://software.intel.com/content/dam/develop/external/us/en/documen
> t
> > s
> > /vt-
> > > directed-io-spec.pdf
> > >
> > >    @par Glossary:
> > >      - HPET - High Precision Event Timer @@ -39,6 +39,7 @@
> > >  #define EFI_ACPI_DMAR_TYPE_ATSR                   0x02
> > >  #define EFI_ACPI_DMAR_TYPE_RHSA                   0x03
> > >  #define EFI_ACPI_DMAR_TYPE_ANDD                   0x04
> > > +#define EFI_ACPI_DMAR_TYPE_SATC                   0x05
> > >  ///@}
> > >
> > >  ///
> > > @@ -216,6 +217,32 @@ typedef struct {
> > >    UINT8                           AcpiDeviceNumber;
> > >  } EFI_ACPI_DMAR_ANDD_HEADER;
> > >
> > > +/**
> > > +  An SoC Integrated Address Translation Cache (SATC) reporting
> structure
> > is
> > > +  defined in section 8.8.
> > > +**/
> > > +typedef struct {
> > > +  EFI_ACPI_DMAR_STRUCTURE_HEADER  Header;
> > > +  /**
> > > +    - Bit[0]: ATC_REQUIRED:
> > > +              - If Set, indicates that every SoC integrated device
> > enumerated
> > > +                in this table has a functional requirement to
> > > + enable
> its
> > ATC
> > > +                (via the ATS capability) for device operation.
> > > +              - If Clear, any device enumerated in this table can
> > operate when
> > > +                its respective ATC is not enabled (albeit with reduced
> > > +                performance or functionality).
> > > +    - Bits[7:1] Reserved.
> > > +  **/
> > > +  UINT8                           Flags;
> > > +  UINT8                           Reserved;
> > > +  ///
> > > +  /// The PCI Segment associated with this SATC structure. All SoC
> > integrated
> > > +  /// devices within a PCI segment with same value for Flags field
> > > + must
> be
> > > +  /// enumerated in the same SATC structure.
> > > +  ///
> > > +  UINT16                          SegmentNumber;
> > > +} EFI_ACPI_DMAR_SATC_HEADER;
> > > +
> > >  /**
> > >    DMA Remapping Reporting Structure Header as defined in section 8.1
> > >    This header will be followed by list of Remapping Structures
> > > listed
> > below
> > > @@ -224,6 +251,7 @@ typedef struct {
> > >      - Root Port ATS Capability Reporting (ATSR)
> > >      - Remapping Hardware Static Affinity (RHSA)
> > >      - ACPI Name-space Device Declaration (ANDD)
> > > +    - SoC Integrated Address Translation Cache reporting (SATC)
> > >    These structure types must by reported in numerical order.
> > >    i.e., All remapping structures of type 0 (DRHD) enumerated before
> > > remapping
> > >    structures of type 1 (RMRR), and so forth.
> > > --
> > > 2.16.2.windows.1
> 
> 
> 
> 
> 
> 
> 


[-- Attachment #2: 0001-MdePkg-include-Add-DMAR-SATC-Table-Definition.patch --]
[-- Type: application/octet-stream, Size: 4330 bytes --]

From 20c39ba1bff1fc9c93d443252c659044b44cf0c6 Mon Sep 17 00:00:00 2001
From: Sheng Wei <w.sheng@intel.com>
Date: Mon, 7 Dec 2020 16:34:44 +0800
Subject: [PATCH] MdePkg/include: Add DMAR SATC Table Definition

SoC Integrated Address Translation Cache (SATC) reporting structure is one
of the Remapping Structure, which is imported since Intel(R) Virtualization
Technology for Directed I/O (VT-D) Architecture Specification v3.2.

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3109

Signed-off-by: Sheng Wei <w.sheng@intel.com>
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Cc: Zhiguang Liu <zhiguang.liu@intel.com>
Cc: Jiewen Yao <jiewen.yao@intel.com>
Cc: Jenny Huang <jenny.huang@intel.com>
Cc: Kowalewski Robert <robert.kowalewski@intel.com>
Cc: Feng Roger <roger.feng@intel.com>
Reviewed-by: Zhiguang Liu <zhiguang.liu@intel.com>
Reviewed-by: Liming Gao <gaoliming@byosoft.com.cn>
---
 .../IndustryStandard/DmaRemappingReportingTable.h  | 34 ++++++++++++++++++++--
 1 file changed, 31 insertions(+), 3 deletions(-)

diff --git a/MdePkg/Include/IndustryStandard/DmaRemappingReportingTable.h b/MdePkg/Include/IndustryStandard/DmaRemappingReportingTable.h
index 7c50dc972e..48f6959fec 100644
--- a/MdePkg/Include/IndustryStandard/DmaRemappingReportingTable.h
+++ b/MdePkg/Include/IndustryStandard/DmaRemappingReportingTable.h
@@ -2,13 +2,13 @@
   DMA Remapping Reporting (DMAR) ACPI table definition from Intel(R)
   Virtualization Technology for Directed I/O (VT-D) Architecture Specification.
 
-  Copyright (c) 2016 - 2018, Intel Corporation. All rights reserved.<BR>
+  Copyright (c) 2016 - 2020, Intel Corporation. All rights reserved.<BR>
   SPDX-License-Identifier: BSD-2-Clause-Patent
 
   @par Revision Reference:
     - Intel(R) Virtualization Technology for Directed I/O (VT-D) Architecture
-      Specification v2.5, Dated November 2017.
-      http://www.intel.com/content/dam/www/public/us/en/documents/product-specifications/vt-directed-io-spec.pdf
+      Specification v3.2, Dated October 2020.
+      https://software.intel.com/content/dam/develop/external/us/en/documents/vt-directed-io-spec.pdf
 
   @par Glossary:
     - HPET - High Precision Event Timer
@@ -39,6 +39,7 @@
 #define EFI_ACPI_DMAR_TYPE_ATSR                   0x02
 #define EFI_ACPI_DMAR_TYPE_RHSA                   0x03
 #define EFI_ACPI_DMAR_TYPE_ANDD                   0x04
+#define EFI_ACPI_DMAR_TYPE_SATC                   0x05
 ///@}
 
 ///
@@ -216,6 +217,32 @@ typedef struct {
   UINT8                           AcpiDeviceNumber;
 } EFI_ACPI_DMAR_ANDD_HEADER;
 
+/**
+  An SoC Integrated Address Translation Cache (SATC) reporting structure is
+  defined in section 8.8.
+**/
+typedef struct {
+  EFI_ACPI_DMAR_STRUCTURE_HEADER  Header;
+  /**
+    - Bit[0]: ATC_REQUIRED:
+              - If Set, indicates that every SoC integrated device enumerated
+                in this table has a functional requirement to enable its ATC
+                (via the ATS capability) for device operation.
+              - If Clear, any device enumerated in this table can operate when
+                its respective ATC is not enabled (albeit with reduced
+                performance or functionality).
+    - Bits[7:1] Reserved.
+  **/
+  UINT8                           Flags;
+  UINT8                           Reserved;
+  ///
+  /// The PCI Segment associated with this SATC structure. All SoC integrated
+  /// devices within a PCI segment with same value for Flags field must be
+  /// enumerated in the same SATC structure.
+  ///
+  UINT16                          SegmentNumber;
+} EFI_ACPI_DMAR_SATC_HEADER;
+
 /**
   DMA Remapping Reporting Structure Header as defined in section 8.1
   This header will be followed by list of Remapping Structures listed below
@@ -224,6 +251,7 @@ typedef struct {
     - Root Port ATS Capability Reporting (ATSR)
     - Remapping Hardware Static Affinity (RHSA)
     - ACPI Name-space Device Declaration (ANDD)
+    - SoC Integrated Address Translation Cache reporting (SATC)
   These structure types must by reported in numerical order.
   i.e., All remapping structures of type 0 (DRHD) enumerated before remapping
   structures of type 1 (RMRR), and so forth.
-- 
2.16.2.windows.1


  reply	other threads:[~2020-12-15  5:58 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-12-11  1:36 [PATCH] MdePkg/include: Add DMAR SATC Table Definition Sheng Wei
2020-12-11  2:51 ` Zhiguang Liu
2020-12-14  2:49   ` 回复: " gaoliming
2020-12-15  5:58     ` Sheng Wei [this message]
2020-12-15  7:16       ` 回复: [edk2-devel] " gaoliming
2020-12-15  8:56         ` Sheng Wei
2020-12-15 10:53           ` 回复: " gaoliming

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=CY4PR11MB1928003FA115C6A51EBA7883E1C60@CY4PR11MB1928.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