linux-kernel - Re: [PATCH v4 4/6] media: rockchip: Introduce the rkvdec2 driver

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] [day] [month] [year] [list]

Message-ID: <a66e3d27ee427710ed3de8ce89fa3aca40dced03.camel@collabora.com>
Date: Thu, 19 Jun 2025 11:16:15 -0400
From: Nicolas Dufresne <nicolas.dufresne@...labora.com>
To: Detlev Casanova <detlev.casanova@...labora.com>, 
	20250325213303.826925-5-detlev.casanova@...labora.com
Cc: alchark@...il.com, andrzej.p@...labora.com, cassel@...nel.org, 
	conor+dt@...nel.org, devicetree@...r.kernel.org,
 dmitry.perchanov@...el.com, 	dsimic@...jaro.org,
 ezequiel@...guardiasur.com.ar, gregkh@...uxfoundation.org, 
	heiko@...ech.de, hverkuil@...all.nl, jacopo.mondi@...asonboard.com, 
	jeanmichel.hautbois@...asonboard.com, jonas@...boo.se,
 kernel@...labora.com, 	kieran.bingham@...asonboard.com, krzk+dt@...nel.org,
 	laurent.pinchart@...asonboard.com, linux-arm-kernel@...ts.infradead.org, 
	linux-kernel@...r.kernel.org, linux-media@...r.kernel.org, 
	linux-rockchip@...ts.infradead.org, linux-staging@...ts.linux.dev, Jianfeng
 Liu	 <liujianfeng1994@...il.com>, mchehab@...nel.org,
 naush@...pberrypi.com, 	robh@...nel.org, sakari.ailus@...ux.intel.com,
 sebastian.reichel@...labora.com, 	tomi.valkeinen@...asonboard.com,
 umang.jain@...asonboard.com
Subject: Re: [PATCH v4 4/6] media: rockchip: Introduce the rkvdec2 driver

Le jeudi 19 juin 2025 à 10:39 -0400, Detlev Casanova a écrit :
> Hi Jianfeng,
> 
> This patch set is now a  bit outdated. I will soon send a new version that you 
> can find at [1].
> That being said, you may be right. without an iommu, it would panic here 
> (although the iommu should really be used).

Let's make sure to test it in the next series. One should be able to comment out
the mmu node in the DT, and then it should work with CMA transparently.

thanks,
Nicolas

> 
> [1]: https://gitlab.collabora.com/detlev/linux/-/tree/add-vdpu381-and-383-to-rkvdec
> 
> Regards,
> Detlev
> 
> On Thursday, 19 June 2025 05:19:32 EDT Jianfeng Liu wrote:
> > Hi Detlev,
> > 
> > On Tue, 25 Mar 2025 17:22:20 -0400, Detlev Casanova wrote:
> >  >+        case RKVDEC2_ALLOC_SRAM:
> >  >+            virt_addr = (unsigned long)ctx->rcb_bufs[i].cpu;
> >  >+
> >  >+            iommu_unmap(rkvdec->iommu_domain, virt_addr, rcb_size)
> > 
> > I'm testing your patch with ffmpeg patched with v4l2-request patches[1],
> > and I usually
> > 
> > get kernel panic here. After checking rkvdec->iommu_domain before
> > running iommu_unmap,
> > 
> > I can pass fluster ffmpeg v4l2-request test. Here is my patch based on
> > your commit:
> > 
> > 
> > diff --git a/drivers/media/platform/rockchip/rkvdec2/rkvdec2.c
> > b/drivers/media/platform/rockchip/rkvdec2/rkvdec2.c
> > index 75768561399..122bcdcebd4 100644
> > --- a/drivers/media/platform/rockchip/rkvdec2/rkvdec2.c
> > +++ b/drivers/media/platform/rockchip/rkvdec2/rkvdec2.c
> > @@ -681,8 +681,8 @@ static void rkvdec2_free_rcb(struct rkvdec2_ctx *ctx)
> >                  switch (ctx->rcb_bufs[i].type) {
> >                  case RKVDEC2_ALLOC_SRAM:
> >                          virt_addr = (unsigned long)ctx->rcb_bufs[i].cpu;
> > -
> > -                       iommu_unmap(rkvdec->iommu_domain, virt_addr,
> > rcb_size);
> > +                       if (rkvdec->iommu_domain)
> > + iommu_unmap(rkvdec->iommu_domain, virt_addr, rcb_size);
> >                          gen_pool_free(ctx->dev->sram_pool, virt_addr,
> > rcb_size);
> >                          break;
> >                  case RKVDEC2_ALLOC_DMA:
> > 
> > 
> > [1] https://github.com/amazingfate/FFmpeg/commits/n6.1.1-new-patches/
> > 
> > 
> > Best regards,
> > 
> > Jianfeng
> 
> 
>