linux-kernel - Re: [PATCH v5 4/4] media: chips-media: wave5: Improve performance of decoder

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: <6eed102e2aa739e5026ee545a38ddacf09058bbb.camel@collabora.com>
Date: Wed, 24 Sep 2025 09:20:40 -0400
From: Nicolas Dufresne <nicolas.dufresne@...labora.com>
To: "jackson.lee" <jackson.lee@...psnmedia.com>, Brandon Brnich
	 <b-brnich@...com>, "mchehab@...nel.org" <mchehab@...nel.org>, 
 "hverkuil-cisco@...all.nl"
	 <hverkuil-cisco@...all.nl>, "bob.beckett@...labora.com"
	 <bob.beckett@...labora.com>
Cc: "linux-media@...r.kernel.org" <linux-media@...r.kernel.org>, 
 "linux-kernel@...r.kernel.org"	 <linux-kernel@...r.kernel.org>,
 "lafley.kim" <lafley.kim@...psnmedia.com>,  "hverkuil@...all.nl"	
 <hverkuil@...all.nl>, Nas Chung <nas.chung@...psnmedia.com>
Subject: Re: [PATCH v5 4/4] media: chips-media: wave5: Improve performance
 of decoder

Hi Jackson,

Le mercredi 24 septembre 2025 à 01:14 +0000, jackson.lee a écrit :
> Hi Nicolas
> 
> > -----Original Message-----
> > From: Nicolas Dufresne <nicolas.dufresne@...labora.com>
> > Sent: Tuesday, September 23, 2025 3:00 AM
> > To: Brandon Brnich <b-brnich@...com>; jackson.lee
> > <jackson.lee@...psnmedia.com>; mchehab@...nel.org; hverkuil-
> > cisco@...all.nl; bob.beckett@...labora.com
> > Cc: linux-media@...r.kernel.org; linux-kernel@...r.kernel.org; lafley.kim
> > <lafley.kim@...psnmedia.com>; hverkuil@...all.nl; Nas Chung
> > <nas.chung@...psnmedia.com>
> > Subject: Re: [PATCH v5 4/4] media: chips-media: wave5: Improve performance
> > of decoder
> > 
> > Hi Brandon,
> > 
> > Le lundi 22 septembre 2025 à 12:32 -0500, Brandon Brnich a écrit :
> > > > -	/*
> > > > -	 * During a resolution change and while draining, the firmware
> > > > may
> > > > flush
> > > > -	 * the reorder queue regardless of having a matching decoding
> > > > operation
> > > > -	 * pending. Only terminate the job if there are no more IRQ
> > > > coming.
> > > > -	 */
> > > > -	wave5_vpu_dec_give_command(inst, DEC_GET_QUEUE_STATUS,
> > > > &q_status);
> > > > -	if (q_status.report_queue_count == 0 &&
> > > > -	    (q_status.instance_queue_count == 0 ||
> > > > dec_info.sequence_changed)) {
> > > > -		dev_dbg(inst->dev->dev, "%s: finishing job.\n",
> > > > __func__);
> > > > -		pm_runtime_mark_last_busy(inst->dev->dev);
> > > 
> > > Patch is failing to apply here to linux-next because these redundant
> > > calls have already been removed[0].
> > 
> > Which have not been merged back from the RC into media-committers/next,
> > forcing to skip a cycle. Jackson, feel free to rebase on linux-next like
> > Brandon suggest.
> 
> Then should I make v6 patch series based on Linux-next ?

I've asked advises from the other maintainers, and the answer is no. Basing it
on our next branch for linux-media submission is the correct thing to do. Its
too late for this cycle, but be reassured we will improve our process in future
iterations to reduce the risk of this happening.

Feel free to send a rebased patch to Brandon, having more testing is always
good.

cheers,
Nicolas

> 
> Thanks
> Jackson
> 
> > 
> > regards,
> 
> > Nicolas

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