From: thierry.reding@gmail.com (Thierry Reding)
To: linux-arm-kernel@lists.infradead.org
Subject: Report from 2013 ARM kernel summit
Date: Wed, 20 Nov 2013 14:53:09 +0100 [thread overview]
Message-ID: <20131120135308.GC8279@ulmo.nvidia.com> (raw)
In-Reply-To: <20131120103111.GB19352@mudshark.cambridge.arm.com>
On Wed, Nov 20, 2013 at 10:31:11AM +0000, Will Deacon wrote:
> On Tue, Nov 19, 2013 at 08:45:02PM +0000, Rob Herring wrote:
> > On 11/19/2013 11:35 AM, Will Deacon wrote:
> > > Adding Andreas and Rob for input on potential binding additions to the SMMU.
> >
> > The above proposal would be an incompatible change. However, I think we
> > could still deal with a change in this binding at this stage.
> >
> > One way approach to handle this without changing the binding would be to
> > scan the DT for all iommu's up front and create a list of all nodes and
> > their iommu parent. The fact that the hierarchy is described in a way
> > that doesn't fit Linux well is really a Linux implementation detail.
> >
> > If changing the binding, a simple approach would be to allow
> > 'smmu-parent' to be a bus and/or device property and not just for
> > chained iommu's. This could be a global or bus property that is
> > inherited. Like interrupt-parent, you would have to deal with the parent
> > being itself. Also, perhaps iommu-parent would be a better name. In any
> > case, I'd like to see this all be a generic iommu binding.
>
> I like that idea. I've recently been toying with removing the chained IOMMU
> support, since I don't think anybody is using it who is interested in
> mainline. However, making it more general sounds like a better idea.
>
> One potential issue is that I think the nvidia guys want to describe masters
> that master via multiple SMMUs (which I believe was the motivation for
> moving the stream-ids out into the master nodes, rather than keeping them in
> the SMMU). Again, that's not something we can easily add to the arm-smmu,
> because the incoming stream-ids are a property of the SMMU node.
If I remember correctly, one of the reasons for the proposal was also
that the interrupt-parent property turned out to be insufficient for
some use-cases, which lead to Grant's proposal of the new interrupts-
extended property. Since that comparison has already been drawn, I think
we can agree that both are used in similar ways. Therefore we should
consider what we've learned from interrupt-parent when designing this
generic IOMMU binding to avoid having to introduce iommu-extended at
some point.
> So the question is: do we actually need to describe masters that master
> through multiple SMMUs as a single node in the devicetree?
I would think so, yes. The alternative would be to have several nodes
that describe the same device, and that conflicts on a different level.
Perhaps it could be done by having separate sub-nodes that each use a
different IOMMU, but that sounds like a much grosser solution. That
pretty much boils down to interrupt-parent/interrupt-map.
Thierry
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20131120/58474dce/attachment.sig>
next prev parent reply other threads:[~2013-11-20 13:53 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-11-08 13:20 Report from 2013 ARM kernel summit Grant Likely
2013-11-11 9:47 ` Hiroshi Doyu
2013-11-15 7:06 ` Grant Likely
2013-11-19 9:40 ` Hiroshi Doyu
2013-11-19 17:35 ` Will Deacon
2013-11-19 20:45 ` Rob Herring
2013-11-20 6:39 ` Hiroshi Doyu
2013-11-20 20:02 ` Rob Herring
2013-11-20 20:22 ` Stephen Warren
2013-11-21 8:49 ` Hiroshi Doyu
2013-11-21 11:00 ` Will Deacon
2013-11-21 18:40 ` Stephen Warren
2013-11-25 18:03 ` Will Deacon
2013-11-20 10:31 ` Will Deacon
2013-11-20 13:53 ` Thierry Reding [this message]
2013-11-20 19:47 ` Rob Herring
2013-11-20 20:22 ` Rob Herring
-- strict thread matches above, loose matches on Subject: below --
2013-11-07 23:29 Grant Likely
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=20131120135308.GC8279@ulmo.nvidia.com \
--to=thierry.reding@gmail.com \
--cc=linux-arm-kernel@lists.infradead.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).