linux-kernel - Re: [RESEND][PATCH v1] dmaengine: sh: Discard pm_runtime

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: <aWBq-Rfu1yez6EjK@vaman>
Date: Fri, 9 Jan 2026 08:12:01 +0530
From: Vinod Koul <vkoul@...nel.org>
To: "Rafael J. Wysocki" <rafael@...nel.org>
Cc: Linux PM <linux-pm@...r.kernel.org>, dmaengine@...r.kernel.org,
	LKML <linux-kernel@...r.kernel.org>,
	Ulf Hansson <ulf.hansson@...aro.org>,
	Brian Norris <briannorris@...omium.org>
Subject: Re: [RESEND][PATCH v1] dmaengine: sh: Discard pm_runtime_put()
 return value

On 08-01-26, 16:28, Rafael J. Wysocki wrote:
> From: Rafael J. Wysocki <rafael.j.wysocki@...el.com>
> 
> Clobbering an error value to be returned from shdma_tx_submit() with
> a pm_runtime_put() return value is not particularly useful, especially
> if the latter is 0, so stop doing that.
> 
> This will facilitate a planned change of the pm_runtime_put() return
> type to void in the future.

Hey Rafael,

This is commit b442377c0ea2044a8f50ffa3fe59448f9ed922c in my tree.

> 
> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@...el.com>
> ---
> 
> This is requisite for converting pm_runtime_put() into a void function.
> 
> If you decide to pick it up, please let me know.
> 
> Otherwise, an ACK or equivalent will be appreciated, but also the lack
> of specific criticism will be eventually regarded as consent.
> 
> Originally posted here:
> 
> https://lore.kernel.org/linux-pm/9626129.rMLUfLXkoz@rafael.j.wysocki/
> 
> ---
>  drivers/dma/sh/shdma-base.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> --- a/drivers/dma/sh/shdma-base.c
> +++ b/drivers/dma/sh/shdma-base.c
> @@ -143,7 +143,7 @@ static dma_cookie_t shdma_tx_submit(stru
>  				}
>  
>  				schan->pm_state = SHDMA_PM_ESTABLISHED;
> -				ret = pm_runtime_put(schan->dev);
> +				pm_runtime_put(schan->dev);
>  
>  				spin_unlock_irq(&schan->chan_lock);
>  				return ret;
> 
> 

-- 
~Vinod