linux-kernel - Re: [PATCH] ARM: tegra: Register host1x node with iommu binding on tegra124

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Mon, 31 Jul 2017 17:23:06 +0200
From:   Thierry Reding <thierry.reding@...il.com>
To:     Marcel Ziswiler <marcel.ziswiler@...adex.com>
Cc:     "linux-arm-kernel@...ts.infradead.org" 
        <linux-arm-kernel@...ts.infradead.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "contact@...lk.fr" <contact@...lk.fr>,
        "linux-tegra@...r.kernel.org" <linux-tegra@...r.kernel.org>,
        "treding@...dia.com" <treding@...dia.com>,
        "mperttunen@...dia.com" <mperttunen@...dia.com>,
        "swarren@...dia.com" <swarren@...dia.com>,
        "jonathanh@...dia.com" <jonathanh@...dia.com>
Subject: Re: [PATCH] ARM: tegra: Register host1x node with iommu binding on
 tegra124

On Tue, Jul 11, 2017 at 03:17:46PM +0000, Marcel Ziswiler wrote:
> On Tue, 2017-07-11 at 18:05 +0300, Paul Kocialkowski wrote:
> > On Tue, 2017-07-11 at 14:54 +0000, Marcel Ziswiler wrote:
> > > On Tue, 2017-07-11 at 11:50 +0300, Paul Kocialkowski wrote:
> > > > On Sun, 2017-07-09 at 19:36 +0300, Paul Kocialkowski wrote:
> > > > > This registers the host1x node with the SMMU (as HC swgroup) to
> > > > > allow
> > > > > the host1x code to attach to it. It avoid failing the probe
> > > > > sequence,
> > > > > which resulted in the tegra drm driver not probing and thus
> > > > > nothing
> > > > > being displayed on-screen.
> > > > 
> > > > Fixes: 404bfb78daf3 ("gpu: host1x: Add IOMMU support")
> > > > 
> > > > > Signed-off-by: Paul Kocialkowski <contact-
> > > > > W9ppeneeCTY@...lic.gmane.org>
> > > 
> > > Tested-by: Marcel Ziswiler <marcel.ziswiler@...adex.com>
> > > Tested-on: Apalis TK1, Jetson-TK1
> > > 
> > > > > ---
> > > > >  arch/arm/boot/dts/tegra124.dtsi | 1 +
> > > > >  1 file changed, 1 insertion(+)
> > > > > 
> > > > > diff --git a/arch/arm/boot/dts/tegra124.dtsi
> > > > > b/arch/arm/boot/dts/tegra124.dtsi
> > > > > index 187a36c6d0fc..b3b89befffeb 100644
> > > > > --- a/arch/arm/boot/dts/tegra124.dtsi
> > > > > +++ b/arch/arm/boot/dts/tegra124.dtsi
> > > > > @@ -85,6 +85,7 @@
> > > > >  		clocks = <&tegra_car TEGRA124_CLK_HOST1X>;
> > > > >  		resets = <&tegra_car 28>;
> > > > >  		reset-names = "host1x";
> > > > > +		iommus = <&mc TEGRA_SWGROUP_HC>;
> > > > >  
> > > > >  		#address-cells = <2>;
> > > > >  		#size-cells = <2>;
> > > 
> > > So I take it we still will need this one moving forward, correct?
> > 
> > Yes, this one actually enables the IOMMU while the other one handles
> > the
> > failure to attach the IOMMU without brinding the whole drm driver
> > down.
> > 
> > So it's best to actually make use of the feature!
> > 
> > > How about tegra30, I guess the same applies there, isn't it? Should
> > > I
> > > send a patch or are you guys doing that?
> > 
> > I don't have any tegra hardware aside of tegra124, so that's all I
> > can
> > contribute to. I don't know if others have looked into the same type
> > of
> > issue happening on either newer or older platforms.
> 
> OK, as we happen to still selling Apalis/Colibri T30 and me having
> Beaver as well as Cardhu available for testing I can cook something up,
> test and send it along. First tries indicate it working on 4.12 but
> somehow crashing during memory allocation on -next. We'll see.

I think we're going to need the same change on Tegra30 and Tegra114. And
Tegra132 and Tegra210 for that matter. With the gpu/host1x fix applied
things should keep working on all generations, but we need DT changes if
we want to make use of the SMMU.

Thierry

Download attachment "signature.asc" of type "application/pgp-signature" (834 bytes)