linux-kernel - Re: [PATCH] dmaengine: ti: k3-udma: Use cap_mask directly from dma

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: <f9396cec-bc07-4304-a57c-76311d95f62c@ti.com>
Date: Mon, 3 Feb 2025 19:02:23 +0530
From: "Kumar, Udit" <u-kumar1@...com>
To: Yemike Abhilash Chandra <y-abhilashchandra@...com>,
        <peter.ujfalusi@...il.com>, <vkoul@...nel.org>
CC: <vaishnav.a@...com>, <dmaengine@...r.kernel.org>,
        <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] dmaengine: ti: k3-udma: Use cap_mask directly from
 dma_device structure instead of a local copy


On 2/3/2025 1:19 PM, Yemike Abhilash Chandra wrote:
> Currently, a local dma_cap_mask_t variable is used to store device
> cap_mask within udma_of_xlate(). However, the DMA_PRIVATE flag in
> the device cap_mask can get cleared when the last channel is released.
> This can happen right after storing the cap_mask locally in
> udma_of_xlate(), and subsequent dma_request_channel() can fail due to
> mismatch in the cap_mask. Fix this by removing the local dma_cap_mask_t
> variable and directly using the one from the dma_device structure.
>
> Fixes: 25dcb5dd7b7c ("dmaengine: ti: New driver for K3 UDMA")
> Cc: stable@...r.kernel.org

I don't see  patch is sent to stable in cc , please check if you 
suppressed cc list.

For rest

Reviewed-by: Udit Kumar <u-kumar1@...com>

> Signed-off-by: Vaishnav Achath <vaishnav.a@...com>
> Signed-off-by: Yemike Abhilash Chandra <y-abhilashchandra@...com>
> Acked-by: Peter Ujfalusi <peter.ujfalusi@...il.com>
> ---
> RFC: https://lore.kernel.org/all/20250117121728.203452-1-y-abhilashchandra@ti.com/
>
>   drivers/dma/ti/k3-udma.c | 3 +--
>   1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/dma/ti/k3-udma.c b/drivers/dma/ti/k3-udma.c
> index 7ed1956b4642..c775a2284e86 100644
> --- a/drivers/dma/ti/k3-udma.c
> +++ b/drivers/dma/ti/k3-udma.c
> @@ -4246,7 +4246,6 @@ static struct dma_chan *udma_of_xlate(struct of_phandle_args *dma_spec,
>   				      struct of_dma *ofdma)
>   {
>   	struct udma_dev *ud = ofdma->of_dma_data;
> -	dma_cap_mask_t mask = ud->ddev.cap_mask;
>   	struct udma_filter_param filter_param;
>   	struct dma_chan *chan;
>   
> @@ -4278,7 +4277,7 @@ static struct dma_chan *udma_of_xlate(struct of_phandle_args *dma_spec,
>   		}
>   	}
>   
> -	chan = __dma_request_channel(&mask, udma_dma_filter_fn, &filter_param,
> +	chan = __dma_request_channel(&ud->ddev.cap_mask, udma_dma_filter_fn, &filter_param,
>   				     ofdma->of_node);
>   	if (!chan) {
>   		dev_err(ud->dev, "get channel fail in %s.\n", __func__);