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: <0dbde87b-593f-4b14-8929-b78e189549ad@ti.com>
Date: Sun, 22 Sep 2024 16:57:07 +0530
From: Beleswar Prasad Padhi <b-padhi@...com>
To: Mathieu Poirier <mathieu.poirier@...aro.org>,
        Lu Baolu
	<baolu.lu@...ux.intel.com>, <andersson@...nel.org>,
        <afd@...com>, <nm@...com>, <hnagalla@...com>
CC: Joerg Roedel <joro@...tes.org>, Will Deacon <will@...nel.org>,
        Robin
 Murphy <robin.murphy@....com>, Jason Gunthorpe <jgg@...pe.ca>,
        Kevin Tian
	<kevin.tian@...el.com>, <linux-remoteproc@...r.kernel.org>,
        <iommu@...ts.linux.dev>, <linux-kernel@...r.kernel.org>,
        Jason Gunthorpe
	<jgg@...dia.com>
Subject: Re: [PATCH 1/1] remoteproc: Use iommu_paging_domain_alloc()


On 29-08-2024 11:47, Beleswar Prasad Padhi wrote:
> Hi All,
>
> On 22/08/24 21:47, Mathieu Poirier wrote:
>> Hi Baolu,
>>
>> Sorry for the late reply, this slipped through the cracks.
>>
>> On Mon, Aug 12, 2024 at 03:28:11PM +0800, Lu Baolu wrote:
>> > An iommu domain is allocated in rproc_enable_iommu() and is 
>> attached to
>> > rproc->dev.parent in the same function.
>> > > Use iommu_paging_domain_alloc() to make it explicit.
>> > > Signed-off-by: Lu Baolu <baolu.lu@...ux.intel.com>
>> > Reviewed-by: Jason Gunthorpe <jgg@...dia.com>
>> > Link: 
>> https://lore.kernel.org/r/20240610085555.88197-13-baolu.lu@linux.intel.com
>> > ---
>> >  drivers/remoteproc/remoteproc_core.c | 6 +++---
>> >  1 file changed, 3 insertions(+), 3 deletions(-)
>> > > diff --git a/drivers/remoteproc/remoteproc_core.c 
>> b/drivers/remoteproc/remoteproc_core.c
>> > index f276956f2c5c..eb66f78ec8b7 100644
>> > --- a/drivers/remoteproc/remoteproc_core.c
>> > +++ b/drivers/remoteproc/remoteproc_core.c
>> > @@ -109,10 +109,10 @@ static int rproc_enable_iommu(struct rproc 
>> *rproc)
>> >          return 0;
>> >      }
>> >  > -    domain = iommu_domain_alloc(dev->bus);
>> > -    if (!domain) {
>> > +    domain = iommu_paging_domain_alloc(dev);
>>
>> I'm a little hesitant here.  Function iommu_domain_alloc() passes 
>> NULL two the
>> second argument of __iommu_domain_alloc() while 
>> iommu_paging_domain_alloc()
>> provides a 'dev'.  I don't have any HW to test on and I am not 
>> familiar enough
>> with the IOMMU subsystem to confidently more forward.
>>
>> I am asking the Qualcomm (Bjorn and friends) and TI crew (Beleswar, 
>> Andrew,
>> Nishanth and Hari) to test this patch on their IOMMU devices and get 
>> back to me
>> with a "Tested-by".
>
>
> Just a heads-up. Currently, I am seeing boot failures while booting 
> remotecores in TI's IOMMU devices with mainline kernel. Working on the 
> the fix, once it is done, will apply the above patch and test it ASAP.


Since commit 17de3f5fdd35, the IOMMU framework has changed how it 
handles callback ops, moving away from using bus->iommu_ops. The 
omap-iommu driver has not yet been updated to align with these framework 
changes. Therefore, omap-iommu, and hence, omap-remoteproc have been 
broken since 17de3f5fdd35.

This patch is a step in the right direction for adapting the 
remoteproc_core framework to the newer IOMMU framework by looking for 
iommu_ops attached to the dev structure instead of dev->bus. Due to time 
constraints, I was unable to update the omap-iommu driver to accommodate 
these changes, so, I could not test this patch. However, logically, this 
patch LGTM. So,

Acked-by: Beleswar Padhi <b-padhi@...com>

Thanks,
Beleswar

>
> Thanks,
> Beleswar
>
>>
>> Thanks,
>> Mathieu
>>
>> > +    if (IS_ERR(domain)) {
>> >          dev_err(dev, "can't alloc iommu domain\n");
>> > -        return -ENOMEM;
>> > +        return PTR_ERR(domain);
>> >      }
>> >  >      iommu_set_fault_handler(domain, rproc_iommu_fault, rproc);
>> > -- > 2.34.1
>> >

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ