From: Dave Martin <Dave.Martin-5wv7dgnIgG8@public.gmane.org>
To: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org>
Cc: "t.figa-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org"
<t.figa-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>,
Will Deacon <will.deacon-5wv7dgnIgG8@public.gmane.org>,
"tomasz.figa-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org"
<tomasz.figa-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
"joshi-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org"
<joshi-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>,
Thierry Reding
<thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
"s.nawrocki-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org"
<s.nawrocki-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>,
"Varun.Sethi-KZfg59tc24xl57MIdRCFDg@public.gmane.org"
<Varun.Sethi-KZfg59tc24xl57MIdRCFDg@public.gmane.org>,
"linux-samsung-soc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
<linux-samsung-soc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
"prathyush.k-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org"
<prathyush.k-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>,
"sachin.kamat-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org"
<sachin.kamat-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
"devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
<devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>,
"grundler-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org"
<grundler-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>,
"kgene.kim-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org"
<kgene.kim-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>,
"a.motakis-lrHrjnjw1UfHK3s98zE1ajGjJy/sRE9J@public.gmane.org"
<a.motakis-lrHrjnjw1UfHK3s98zE1ajGjJy/sRE9J@public.gmane.org>,
"pullip.cho-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org"
<pullip.cho-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
rahul.sharma-Sze3O3UU22KIwRZHo2/mJg@public.gmane.org
Subject: Re: [PATCH v12 11/31] documentation: iommu: add binding document of Exynos System MMU
Date: Thu, 1 May 2014 12:15:35 +0100 [thread overview]
Message-ID: <20140501111527.GA3732@e103592.cambridge.arm.com> (raw)
In-Reply-To: <6607258.MzzLbm2Afe@wuerfel>
On Tue, Apr 29, 2014 at 10:46:18PM +0200, Arnd Bergmann wrote:
> On Tuesday 29 April 2014 19:16:02 Dave Martin wrote:
[...]
> > For example, suppose devices can post MSIs to an interrupt controller
> > via a mailbox accessed through the IOMMU. Suppose also that the IOMMU
> > generates MSIs itself in order to signal management events or faults
> > to a host OS. Linux (as host) will need to configure the interrupt
> > controller separately for the IOMMU and for the IOMMU clients. This
> > means that Linux needs to know which IDs may travel to the interrupt
> > controller for which purpose, and they must be distinct.
>
> I don't understand. An MSI controller is just an address that acts
> as a DMA slave for a 4-byte inbound data packet. It has no way of
> knowing who is sending data, other than by the address or the data
> sent to it. Are you talking of something else?
Oops, looks like there are a few points I failed to respond to here...
I'm not an expert on PCI -- I'm prepared to believe it works that way.
GICv3 can descriminate between different MSI senders based on ID
signals on the bus.
>
> > I'm not sure whether there is actually a SoC today that is MSI-capable
> > and contains an IOMMU, but all the components to build one are out
> > there today. GICv3 is also explicitly designed to support such
> > systems.
>
> A lot of SoCs have MSI integrated into the PCI root complex, which
> of course is pointless from MSI perspective, as well as implying that
> the MSI won't go through the IOMMU.
>
> We have briefly mentioned MSI in the review of the Samsung GH7 PCI
> support. It's possible that this one can either use the built-in
> MSI or the one in the GICv2m.
We are likely to get non-PCI MSIs in future SoC systems too, and there
are no standards governing how such systems should look.
> > In the future, it is likely that "HSA"-style GPUs and other high-
> > throughput virtualisable bus mastering devices will have capabilities
> > of this sort, but I don't think there's anything concrete yet.
>
> Wouldn't they just have IOMMUs with multiple contexts?
Who knows? A management component of the GPU that is under exclusive
control of the host or hypervisor might be wired up to bypass the IOMMU
completely.
I'm not saying this kind of thing definitely will happen, but I can't
say confidently that it won't.
> > > how it might be wired up in hardware, but I don't know what it's good for,
> > > or who would actually do it.
> > >
> > > > > A variation would be to not use #iommu-cells at all, but provide a
> > > > > #address-cells / #size-cells pair in the IOMMU, and have a translation
> > > > > as we do for dma-ranges. This is probably most flexible.
> > > >
> > > > That would also allow us to describe ranges of master IDs, which we need for
> > > > things like PCI RCs on the ARM SMMU. Furthermore, basic transformations of
> > > > these ranges could also be described like this, although I think Dave (CC'd)
> > > > has some similar ideas in this area.
> >
> > Ideally, we would reuse the ePAPR "ranges" concept and describe the way
> > sideband ID signals propagate down the bus hierarchy in a similar way.
>
> It would be 'dma-ranges'. Unfortunately that would imply that each DMA
> master is connected to only one IOMMU, which you say is not necessarily
> the case. The simpler case of a device is only a master on a single IOMMU
> but can use multiple contexts would however work fine with dma-ranges.
Partly, yes. The concept embodied by "dma-ranges" is correct, but the
topological relationship is not: the assumption that a master device
always masters onto its parent node doesn't work for non-tree-like
topologies.
Cheers
---Dave
next prev parent reply other threads:[~2014-05-01 11:15 UTC|newest]
Thread overview: 78+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-04-27 7:37 [PATCH v12 00/31] iommu/exynos: Fixes and Enhancements of System MMU driver with DT Shaik Ameer Basha
2014-04-27 7:37 ` [PATCH v12 02/31] iommu/exynos: add missing cache flush for removed page table entries Shaik Ameer Basha
2014-04-27 7:37 ` [PATCH v12 04/31] iommu/exynos: fix L2TLB invalidation Shaik Ameer Basha
2014-04-27 7:37 ` [PATCH v12 06/31] iommu/exynos: allocate lv2 page table from own slab Shaik Ameer Basha
2014-04-27 7:37 ` [PATCH v12 07/31] iommu/exynos: always enable runtime PM Shaik Ameer Basha
2014-04-27 7:37 ` [PATCH v12 08/31] iommu/exynos: handle one instance of sysmmu with a device descriptor Shaik Ameer Basha
2014-04-27 7:37 ` [PATCH v12 10/31] iommu/exynos: use managed device helper functions Shaik Ameer Basha
2014-04-27 7:37 ` [PATCH v12 11/31] documentation: iommu: add binding document of Exynos System MMU Shaik Ameer Basha
2014-04-27 18:23 ` Arnd Bergmann
2014-04-28 10:39 ` Thierry Reding
2014-04-28 10:56 ` Arnd Bergmann
2014-04-28 11:18 ` Thierry Reding
2014-04-28 12:05 ` Arnd Bergmann
2014-04-28 12:49 ` Thierry Reding
2014-04-28 19:30 ` Will Deacon
[not found] ` <20140428193056.GD22135-5wv7dgnIgG8@public.gmane.org>
2014-04-28 19:55 ` Arnd Bergmann
2014-04-29 18:16 ` Dave Martin
[not found] ` <20140429181601.GE3582-M5GwZQ6tE7x5pKCnmE3YQBJ8xKzm50AiAL8bYrjMMd8@public.gmane.org>
2014-04-29 20:07 ` Grant Grundler
[not found] ` <CANEJEGs6TXNzE8cWYgEKfFSsD2w5XiYvwSbhQ_+gtfzfs+6udA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-04-29 21:00 ` Arnd Bergmann
2014-04-30 15:14 ` Dave Martin
2014-05-01 14:02 ` Cho KyongHo
[not found] ` <20140501230214.ed53cd0fc977225f37b14e29-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
2014-05-01 14:12 ` Arnd Bergmann
2014-05-01 14:50 ` Dave Martin
2014-05-01 17:41 ` Stephen Warren
[not found] ` <53628751.9000609-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2014-05-02 11:41 ` Dave Martin
2014-04-29 20:46 ` Arnd Bergmann
2014-05-01 11:15 ` Dave Martin [this message]
[not found] ` <20140501111527.GA3732-M5GwZQ6tE7x5pKCnmE3YQBJ8xKzm50AiAL8bYrjMMd8@public.gmane.org>
2014-05-01 13:29 ` Arnd Bergmann
2014-05-01 14:36 ` Dave Martin
[not found] ` <20140501143654.GB3732-M5GwZQ6tE7x5pKCnmE3YQBJ8xKzm50AiAL8bYrjMMd8@public.gmane.org>
2014-05-01 15:11 ` Marc Zyngier
[not found] ` <53626434.8000807-5wv7dgnIgG8@public.gmane.org>
2014-05-01 15:53 ` Arnd Bergmann
2014-05-01 16:24 ` Marc Zyngier
2014-05-01 15:46 ` Arnd Bergmann
2014-05-01 16:42 ` Grant Grundler
2014-05-15 20:37 ` Thierry Reding
2014-05-16 0:39 ` Cho KyongHo
2014-04-28 17:52 ` Stephen Warren
2014-04-29 5:55 ` Hiroshi Doyu
2014-04-27 7:37 ` [PATCH v12 12/31] iommu/exynos: support for device tree Shaik Ameer Basha
2014-04-27 7:37 ` [PATCH v12 13/31] iommu/exynos: gating clocks of master H/W Shaik Ameer Basha
2014-04-27 7:37 ` [PATCH v12 15/31] iommu/exynos: handle 'mmu-masters' property of DT and improve handling sysmmu Shaik Ameer Basha
2014-04-27 18:17 ` Arnd Bergmann
2014-05-01 14:08 ` Cho KyongHo
2014-04-27 7:37 ` [PATCH v12 16/31] iommu/exynos: turn on useful configuration options Shaik Ameer Basha
2014-04-27 7:37 ` [PATCH v12 17/31] iommu/exynos: add support for power management subsystems Shaik Ameer Basha
2014-04-27 7:37 ` [PATCH v12 19/31] iommu/exynos: change rwlock to spinlock Shaik Ameer Basha
[not found] ` <1398584283-22846-1-git-send-email-shaik.ameer-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
2014-04-27 7:37 ` [PATCH v12 01/31] iommu/exynos: do not include removed header Shaik Ameer Basha
2014-04-27 7:37 ` [PATCH v12 03/31] iommu/exynos: change error handling when page table update is failed Shaik Ameer Basha
2014-04-27 7:37 ` [PATCH v12 05/31] iommu/exynos: remove prefetch buffer setting Shaik Ameer Basha
2014-04-27 7:37 ` [PATCH v12 09/31] iommu/exynos: remove dbgname from drvdata of a System MMU Shaik Ameer Basha
2014-04-27 7:37 ` [PATCH v12 14/31] iommu/exynos: remove custom fault handler Shaik Ameer Basha
2014-04-27 7:37 ` [PATCH v12 18/31] iommu/exynos: allow having multiple System MMUs for a master H/W Shaik Ameer Basha
2014-04-28 10:38 ` Tushar Behera
[not found] ` <535E2F96.908-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2014-05-01 14:10 ` Cho KyongHo
2014-05-06 18:05 ` Tomasz Figa
[not found] ` <5369245A.1060001-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2014-05-09 10:54 ` Cho KyongHo
2014-04-27 7:37 ` [PATCH v12 20/31] iommu/exynos: add devices attached to the System MMU to an IOMMU group Shaik Ameer Basha
2014-04-27 7:37 ` [PATCH v12 21/31] iommu/exynos: fix address handling Shaik Ameer Basha
2014-04-27 7:37 ` [PATCH v12 24/31] iommu/exynos: apply workaround of caching fault page table entries Shaik Ameer Basha
2014-04-27 7:37 ` [PATCH v12 25/31] iommu/exynos: enhanced error messages Shaik Ameer Basha
2014-04-27 7:37 ` [PATCH v12 26/31] clk: exynos: add gate clock descriptions of System MMU Shaik Ameer Basha
2014-04-27 7:37 ` [PATCH v12 27/31] ARM: dts: add System MMU nodes of exynos4 series Shaik Ameer Basha
2014-04-27 7:38 ` [PATCH v12 28/31] ARM: dts: add System MMU nodes of exynos4210 Shaik Ameer Basha
2014-04-27 7:38 ` [PATCH v12 29/31] ARM: dts: add System MMU nodes of exynos4x12 Shaik Ameer Basha
2014-04-27 7:38 ` [PATCH v12 30/31] ARM: dts: add System MMU nodes of exynos5250 Shaik Ameer Basha
[not found] ` <1398584283-22846-31-git-send-email-shaik.ameer-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
2014-04-27 17:39 ` Vikas Sajjan
2014-04-28 23:13 ` Doug Anderson
[not found] ` <CAD=FV=UCpQRg9nWu5EfuzWmBpee9N3X6yCmtpRaNQxitfFZkMQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-05-01 14:16 ` Cho KyongHo
2014-04-27 7:38 ` [PATCH v12 31/31] ARM: dts: add System MMU nodes of exynos5420 Shaik Ameer Basha
2014-04-28 8:34 ` [PATCH v12 00/31] iommu/exynos: Fixes and Enhancements of System MMU driver with DT Arnd Bergmann
2014-04-30 4:50 ` Shaik Ameer Basha
2014-04-30 10:57 ` Shaik Ameer Basha
2014-05-06 17:59 ` Joerg Roedel
[not found] ` <20140506175904.GB12376-zLv9SwRftAIdnm+yROfE0A@public.gmane.org>
2014-05-06 18:08 ` Tomasz Figa
[not found] ` <5369252F.4070402-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2014-05-07 0:44 ` Cho KyongHo
2014-05-06 18:21 ` Arnd Bergmann
2014-04-27 7:37 ` [PATCH v12 22/31] iommu/exynos: use exynos-iommu specific typedef Shaik Ameer Basha
2014-04-27 7:37 ` [PATCH v12 23/31] iommu/exynos: use simpler function to get MMU version Shaik Ameer Basha
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=20140501111527.GA3732@e103592.cambridge.arm.com \
--to=dave.martin-5wv7dgnigg8@public.gmane.org \
--cc=Varun.Sethi-KZfg59tc24xl57MIdRCFDg@public.gmane.org \
--cc=a.motakis-lrHrjnjw1UfHK3s98zE1ajGjJy/sRE9J@public.gmane.org \
--cc=arnd-r2nGTMty4D4@public.gmane.org \
--cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=grundler-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org \
--cc=joshi-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org \
--cc=kgene.kim-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org \
--cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
--cc=linux-samsung-soc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=prathyush.k-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org \
--cc=pullip.cho-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org \
--cc=rahul.sharma-Sze3O3UU22KIwRZHo2/mJg@public.gmane.org \
--cc=s.nawrocki-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org \
--cc=sachin.kamat-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org \
--cc=swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org \
--cc=t.figa-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org \
--cc=thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
--cc=tomasz.figa-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
--cc=will.deacon-5wv7dgnIgG8@public.gmane.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).