From: Rob Herring <robh@kernel.org>
To: Jean-Philippe Brucker <jean-philippe@linaro.org>
Cc: bhelgaas@google.com, will@kernel.org, robh+dt@kernel.org,
lorenzo.pieralisi@arm.com, joro@8bytes.org,
baolu.lu@linux.intel.com, linux-doc@vger.kernel.org,
linux-pci@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
devicetree@vger.kernel.org, linux-acpi@vger.kernel.org,
iommu@lists.linux-foundation.org, corbet@lwn.net,
mark.rutland@arm.com, liviu.dudau@arm.com, sudeep.holla@arm.com,
guohanjun@huawei.com, rjw@rjwysocki.net, lenb@kernel.org,
robin.murphy@arm.com, dwmw2@infradead.org,
amurray@thegoodpenguin.co.uk, frowand.list@gmail.com
Subject: Re: [PATCH 01/11] dt-bindings: PCI: generic: Add ats-supported property
Date: Wed, 19 Feb 2020 16:24:54 -0600 [thread overview]
Message-ID: <20200219222454.GA16221@bogus> (raw)
In-Reply-To: <20200213165049.508908-2-jean-philippe@linaro.org>
On Thu, 13 Feb 2020 17:50:39 +0100, Jean-Philippe Brucker wrote:
> Add a way for firmware to tell the OS that ATS is supported by the PCI
> root complex. An endpoint with ATS enabled may send Translation Requests
> and Translated Memory Requests, which look just like Normal Memory
> Requests with a non-zero AT field. So a root controller that ignores the
> AT field may simply forward the request to the IOMMU as a Normal Memory
> Request, which could end badly. In any case, the endpoint will be
> unusable.
>
> The ats-supported property allows the OS to only enable ATS in endpoints
> if the root controller can handle ATS requests. Only add the property to
> pcie-host-ecam-generic for the moment. For non-generic root controllers,
> availability of ATS can be inferred from the compatible string.
>
> Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org>
> ---
> Documentation/devicetree/bindings/pci/host-generic-pci.yaml | 6 ++++++
> 1 file changed, 6 insertions(+)
>
Reviewed-by: Rob Herring <robh@kernel.org>
WARNING: multiple messages have this Message-ID (diff)
From: Rob Herring <robh@kernel.org>
To: Jean-Philippe Brucker <jean-philippe@linaro.org>
Cc: mark.rutland@arm.com, linux-doc@vger.kernel.org,
linux-pci@vger.kernel.org, liviu.dudau@arm.com,
guohanjun@huawei.com, will@kernel.org, corbet@lwn.net,
frowand.list@gmail.com, linux-acpi@vger.kernel.org,
lenb@kernel.org, devicetree@vger.kernel.org, robh+dt@kernel.org,
bhelgaas@google.com, linux-arm-kernel@lists.infradead.org,
dwmw2@infradead.org, rjw@rjwysocki.net,
iommu@lists.linux-foundation.org, sudeep.holla@arm.com,
robin.murphy@arm.com, amurray@thegoodpenguin.co.uk
Subject: Re: [PATCH 01/11] dt-bindings: PCI: generic: Add ats-supported property
Date: Wed, 19 Feb 2020 16:24:54 -0600 [thread overview]
Message-ID: <20200219222454.GA16221@bogus> (raw)
In-Reply-To: <20200213165049.508908-2-jean-philippe@linaro.org>
On Thu, 13 Feb 2020 17:50:39 +0100, Jean-Philippe Brucker wrote:
> Add a way for firmware to tell the OS that ATS is supported by the PCI
> root complex. An endpoint with ATS enabled may send Translation Requests
> and Translated Memory Requests, which look just like Normal Memory
> Requests with a non-zero AT field. So a root controller that ignores the
> AT field may simply forward the request to the IOMMU as a Normal Memory
> Request, which could end badly. In any case, the endpoint will be
> unusable.
>
> The ats-supported property allows the OS to only enable ATS in endpoints
> if the root controller can handle ATS requests. Only add the property to
> pcie-host-ecam-generic for the moment. For non-generic root controllers,
> availability of ATS can be inferred from the compatible string.
>
> Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org>
> ---
> Documentation/devicetree/bindings/pci/host-generic-pci.yaml | 6 ++++++
> 1 file changed, 6 insertions(+)
>
Reviewed-by: Rob Herring <robh@kernel.org>
_______________________________________________
iommu mailing list
iommu@lists.linux-foundation.org
https://lists.linuxfoundation.org/mailman/listinfo/iommu
WARNING: multiple messages have this Message-ID (diff)
From: Rob Herring <robh@kernel.org>
To: Jean-Philippe Brucker <jean-philippe@linaro.org>
Cc: mark.rutland@arm.com, linux-doc@vger.kernel.org,
linux-pci@vger.kernel.org, liviu.dudau@arm.com,
guohanjun@huawei.com, will@kernel.org, lorenzo.pieralisi@arm.com,
corbet@lwn.net, frowand.list@gmail.com, joro@8bytes.org,
linux-acpi@vger.kernel.org, lenb@kernel.org,
devicetree@vger.kernel.org, robh+dt@kernel.org,
bhelgaas@google.com, linux-arm-kernel@lists.infradead.org,
dwmw2@infradead.org, rjw@rjwysocki.net,
iommu@lists.linux-foundation.org, sudeep.holla@arm.com,
baolu.lu@linux.intel.com, robin.murphy@arm.com,
amurray@thegoodpenguin.co.uk
Subject: Re: [PATCH 01/11] dt-bindings: PCI: generic: Add ats-supported property
Date: Wed, 19 Feb 2020 16:24:54 -0600 [thread overview]
Message-ID: <20200219222454.GA16221@bogus> (raw)
In-Reply-To: <20200213165049.508908-2-jean-philippe@linaro.org>
On Thu, 13 Feb 2020 17:50:39 +0100, Jean-Philippe Brucker wrote:
> Add a way for firmware to tell the OS that ATS is supported by the PCI
> root complex. An endpoint with ATS enabled may send Translation Requests
> and Translated Memory Requests, which look just like Normal Memory
> Requests with a non-zero AT field. So a root controller that ignores the
> AT field may simply forward the request to the IOMMU as a Normal Memory
> Request, which could end badly. In any case, the endpoint will be
> unusable.
>
> The ats-supported property allows the OS to only enable ATS in endpoints
> if the root controller can handle ATS requests. Only add the property to
> pcie-host-ecam-generic for the moment. For non-generic root controllers,
> availability of ATS can be inferred from the compatible string.
>
> Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org>
> ---
> Documentation/devicetree/bindings/pci/host-generic-pci.yaml | 6 ++++++
> 1 file changed, 6 insertions(+)
>
Reviewed-by: Rob Herring <robh@kernel.org>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2020-02-19 22:30 UTC|newest]
Thread overview: 63+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-02-13 16:50 [PATCH 00/10] PCI/ATS: Device-tree support and other improvements Jean-Philippe Brucker
2020-02-13 16:50 ` Jean-Philippe Brucker
2020-02-13 16:50 ` Jean-Philippe Brucker
2020-02-13 16:50 ` [PATCH 01/11] dt-bindings: PCI: generic: Add ats-supported property Jean-Philippe Brucker
2020-02-13 16:50 ` Jean-Philippe Brucker
2020-02-13 16:50 ` Jean-Philippe Brucker
2020-02-19 22:24 ` Rob Herring [this message]
2020-02-19 22:24 ` Rob Herring
2020-02-19 22:24 ` Rob Herring
2020-02-13 16:50 ` [PATCH 02/11] PCI: Add ats_supported host bridge flag Jean-Philippe Brucker
2020-02-13 16:50 ` Jean-Philippe Brucker
2020-02-13 16:50 ` Jean-Philippe Brucker
2020-02-15 21:10 ` Bjorn Helgaas
2020-02-15 21:10 ` Bjorn Helgaas
2020-02-15 21:10 ` Bjorn Helgaas
2020-02-17 15:40 ` Jean-Philippe Brucker
2020-02-17 15:40 ` Jean-Philippe Brucker
2020-02-17 15:40 ` Jean-Philippe Brucker
2020-02-13 16:50 ` [PATCH 03/11] PCI: OF: Check whether the host bridge supports ATS Jean-Philippe Brucker
2020-02-13 16:50 ` Jean-Philippe Brucker
2020-02-13 16:50 ` Jean-Philippe Brucker
2020-02-13 18:26 ` Rob Herring
2020-02-13 18:26 ` Rob Herring
2020-02-13 18:26 ` Rob Herring
2020-02-17 12:40 ` Jean-Philippe Brucker
2020-02-17 12:40 ` Jean-Philippe Brucker
2020-02-17 12:40 ` Jean-Philippe Brucker
2020-02-13 16:50 ` [PATCH 04/11] ACPI/IORT: Check ATS capability in root complex node Jean-Philippe Brucker
2020-02-13 16:50 ` Jean-Philippe Brucker
2020-02-13 16:50 ` Jean-Philippe Brucker
2020-03-06 9:37 ` Hanjun Guo
2020-03-06 9:37 ` Hanjun Guo
2020-03-06 9:37 ` Hanjun Guo
2020-02-13 16:50 ` [PATCH 05/11] PCI/ATS: Gather checks into pci_ats_supported() Jean-Philippe Brucker
2020-02-13 16:50 ` Jean-Philippe Brucker
2020-02-13 16:50 ` Jean-Philippe Brucker
2020-02-13 16:50 ` [PATCH 06/11] iommu/amd: Use pci_ats_supported() Jean-Philippe Brucker
2020-02-13 16:50 ` Jean-Philippe Brucker
2020-02-13 16:50 ` Jean-Philippe Brucker
2020-02-13 16:50 ` [PATCH 07/11] iommu/arm-smmu-v3: " Jean-Philippe Brucker
2020-02-13 16:50 ` Jean-Philippe Brucker
2020-02-13 16:50 ` Jean-Philippe Brucker
2020-02-13 16:50 ` [PATCH 08/11] iommu/vt-d: " Jean-Philippe Brucker
2020-02-13 16:50 ` Jean-Philippe Brucker
2020-02-13 16:50 ` Jean-Philippe Brucker
2020-02-13 16:50 ` [PATCH 09/11] ACPI/IORT: Drop ATS fwspec flag Jean-Philippe Brucker
2020-02-13 16:50 ` Jean-Philippe Brucker
2020-02-13 16:50 ` Jean-Philippe Brucker
2020-03-06 9:42 ` Hanjun Guo
2020-03-06 9:42 ` Hanjun Guo
2020-03-06 9:42 ` Hanjun Guo
2020-02-13 16:50 ` [PATCH 10/11] arm64: dts: fast models: Enable PCIe ATS for Base RevC FVP Jean-Philippe Brucker
2020-02-13 16:50 ` Jean-Philippe Brucker
2020-02-13 16:50 ` Jean-Philippe Brucker
2020-02-13 16:50 ` [PATCH 11/11] Documentation: Generalize the "pci=noats" boot parameter Jean-Philippe Brucker
2020-02-13 16:50 ` Jean-Philippe Brucker
2020-02-13 16:50 ` Jean-Philippe Brucker
2020-02-19 10:42 ` [PATCH 00/10] PCI/ATS: Device-tree support and other improvements Joerg Roedel
2020-02-19 10:42 ` Joerg Roedel
2020-02-19 10:42 ` Joerg Roedel
2020-03-06 9:32 ` Hanjun Guo
2020-03-06 9:32 ` Hanjun Guo
2020-03-06 9:32 ` Hanjun Guo
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-all 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=20200219222454.GA16221@bogus \
--to=robh@kernel.org \
--cc=amurray@thegoodpenguin.co.uk \
--cc=baolu.lu@linux.intel.com \
--cc=bhelgaas@google.com \
--cc=corbet@lwn.net \
--cc=devicetree@vger.kernel.org \
--cc=dwmw2@infradead.org \
--cc=frowand.list@gmail.com \
--cc=guohanjun@huawei.com \
--cc=iommu@lists.linux-foundation.org \
--cc=jean-philippe@linaro.org \
--cc=joro@8bytes.org \
--cc=lenb@kernel.org \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-doc@vger.kernel.org \
--cc=linux-pci@vger.kernel.org \
--cc=liviu.dudau@arm.com \
--cc=lorenzo.pieralisi@arm.com \
--cc=mark.rutland@arm.com \
--cc=rjw@rjwysocki.net \
--cc=robh+dt@kernel.org \
--cc=robin.murphy@arm.com \
--cc=sudeep.holla@arm.com \
--cc=will@kernel.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.