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 08:56:15 +0000 [thread overview]
Message-ID: <CY4PR11MB19289FB3B088BFDED2EEB11AE1C60@CY4PR11MB1928.namprd11.prod.outlook.com> (raw)
In-Reply-To: <00fb01d6d2b2$306f4ae0$914de0a0$@byosoft.com.cn>
Hi Liming,
I just check the PR, there is a "tianocore.PatchCheck" failing.
The fail belongs to the commit "Merge branch 'master' into MdeDmar" c5741f1.
The commit of " MdePkg/include: Add DMAR SATC Table Definition " is af411fd.
Is it any thing needs to change on this PR ?
BR
Sheng Wei
> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of
> gaoliming
> Sent: 2020年12月15日 15:16
> To: devel@edk2.groups.io; Sheng, W <w.sheng@intel.com>; 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: 回复: [edk2-devel] 回复: [PATCH] MdePkg/include: Add DMAR
> SATC Table Definition
>
> PR https://github.com/tianocore/edk2/pull/1230 is created.
>
> > -----邮件原件-----
> > 发件人: bounce+27952+68842+4905953+8761045@groups.io
> > <bounce+27952+68842+4905953+8761045@groups.io> 代表 Sheng Wei
> > 发送时间: 2020年12月15日 13:59
> > 收件人: devel@edk2.groups.io; gaoliming@byosoft.com.cn; Liu, Zhiguang
> > <zhiguang.liu@intel.com>
> > 抄送: 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>
> > 主题: Re: [edk2-devel] 回复: [PATCH] MdePkg/include: Add DMAR SATC
> Table
> > Definition
> >
> > 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/docume
> > > n
> > > 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
> > >
> > >
> > >
> > >
> > >
> > >
> > >
> >
> >
> >
> >
> >
>
>
>
>
>
>
>
next prev parent reply other threads:[~2020-12-15 8:56 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 ` [edk2-devel] " Sheng Wei
2020-12-15 7:16 ` 回复: " gaoliming
2020-12-15 8:56 ` Sheng Wei [this message]
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=CY4PR11MB19289FB3B088BFDED2EEB11AE1C60@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