linux-kernel - RE: [PATCH RFC 0/2] iommu/arm-smmu-v3: bypass streamid zero on i.MX95

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]

Message-ID:
 <PAXPR04MB8459AB654150BED2FB5FFDE188462@PAXPR04MB8459.eurprd04.prod.outlook.com>
Date: Wed, 16 Oct 2024 00:56:54 +0000
From: Peng Fan <peng.fan@....com>
To: Robin Murphy <robin.murphy@....com>, "Peng Fan (OSS)"
	<peng.fan@....nxp.com>, Will Deacon <will@...nel.org>, Joerg Roedel
	<joro@...tes.org>, Rob Herring <robh@...nel.org>, Krzysztof Kozlowski
	<krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>
CC: Joy Zou <joy.zou@....com>, "linux-arm-kernel@...ts.infradead.org"
	<linux-arm-kernel@...ts.infradead.org>, "iommu@...ts.linux.dev"
	<iommu@...ts.linux.dev>, "devicetree@...r.kernel.org"
	<devicetree@...r.kernel.org>, "linux-kernel@...r.kernel.org"
	<linux-kernel@...r.kernel.org>, Jason Gunthorpe <jgg@...pe.ca>
Subject: RE: [PATCH RFC 0/2] iommu/arm-smmu-v3: bypass streamid zero on i.MX95

Hi Robin,

> Subject: Re: [PATCH RFC 0/2] iommu/arm-smmu-v3: bypass streamid
> zero on i.MX95
> 
> On 2024-10-15 4:14 am, Peng Fan (OSS) wrote:
> > i.MX95 eDMA3 connects to DSU ACP, supporting dma coherent
> memory to
> > memory operations. However TBU is in the path between eDMA3
> and ACP,
> > need to bypass the default SID 0 to make eDMA3 work properly.
> 
> I'm confused, why not just describe that the device owns this StreamID
> in the DT the normal way, i.e, "iommus = <&smmu 0>;"?

Current SMMU-v3 driver not support streamID sharing, but we have
limited streamIDs(i.MX95 max supports 64 SIDs). However the eDMA
supports 64 channels, each channel may have a SID, but not supported
by current smmuv3 driver.

We see smmu would degrade the performance in somecases, so
we wanna to give user an option to bypass SMMU for eDMA.

Thanks,
Peng.

> 
> Thanks,
> Robin.
> 
> > I was also thinking to introduce "bypass-sids = <0xA 0xB 0xC ...>" to
> > make this reusable for others, but not sure. I could switch to
> > "bypass-sids" if you prefer.
> >
> > Signed-off-by: Peng Fan <peng.fan@....com>
> > ---
> > Peng Fan (2):
> >        dt-bindings: iommu: arm,smmu-v3: introduce nxp,imx95-bypass-
> sid-zero
> >        iommu/arm-smmu-v3: Bypass SID0 for NXP i.MX95
> >
> >   .../devicetree/bindings/iommu/arm,smmu-v3.yaml        |  4 ++++
> >   drivers/iommu/arm/arm-smmu-v3/arm-smmu-v3.c           | 19
> ++++++++++++++++---
> >   drivers/iommu/arm/arm-smmu-v3/arm-smmu-v3.h           |  1 +
> >   3 files changed, 21 insertions(+), 3 deletions(-)
> > ---
> > base-commit: d61a00525464bfc5fe92c6ad713350988e492b88
> > change-id: 20241014-smmuv3-120b24bc4659
> >
> > Best regards,