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: <7b281f10-f42c-454b-9c4d-96ea4f66c66f@kernel.org>
Date: Sat, 4 Oct 2025 19:36:08 +0100
From: Srinivas Kandagatla <srini@...nel.org>
To: Jingyi Wang <jingyi.wang@....qualcomm.com>,
 Srinivas Kandagatla <srini@...nel.org>,
 Amol Maheshwari <amahesh@....qualcomm.com>, Arnd Bergmann <arnd@...db.de>,
 Greg Kroah-Hartman <gregkh@...uxfoundation.org>
Cc: aiqun.yu@....qualcomm.com, tingwei.zhang@....qualcomm.com,
 trilok.soni@....qualcomm.com, yijie.yang@....qualcomm.com,
 linux-arm-msm@...r.kernel.org, dri-devel@...ts.freedesktop.org,
 linux-kernel@...r.kernel.org,
 Kumari Pallavi <kumari.pallavi@....qualcomm.com>
Subject: Re: [PATCH 2/2] misc: fastrpc: Update dma_mask for CDSP support on
 Kaanapali SoC



On 9/25/25 12:46 AM, Jingyi Wang wrote:
> From: Kumari Pallavi <kumari.pallavi@....qualcomm.com>
> 
> DSP currently supports 32-bit IOVA (32-bit PA + 4-bit SID) for
> both Q6 and user DMA (uDMA) access. This is being upgraded to
> 34-bit PA + 4-bit SID due to a hardware revision in CDSP for
> Kaanapali SoC, which expands the DMA addressable range.
> Update DMA mask configuration in the driver to support CDSP on
> Kaanapali SoC. Set the default `dma_mask` to 32-bit and update
> it to 34-bit based on CDSP and SoC-specific compatible string.
> 
> Signed-off-by: Kumari Pallavi <kumari.pallavi@....qualcomm.com>
> Signed-off-by: Jingyi Wang <jingyi.wang@....qualcomm.com>
> ---
>  drivers/misc/fastrpc.c | 14 ++++++++++----
>  1 file changed, 10 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/misc/fastrpc.c b/drivers/misc/fastrpc.c
> index db396241b8ce..e019163eb265 100644
> --- a/drivers/misc/fastrpc.c
> +++ b/drivers/misc/fastrpc.c
> @@ -275,6 +275,7 @@ struct fastrpc_session_ctx {
>  	bool used;
>  	bool valid;
>  	u32 sid_pos;
> +	u32 pa_bits;
same comments as in patch 1 move to channel ctx, also why do we even
need this  to be stored in the first place as dma mask is set in
probe,we will not need it after wards.
>  };
>  
>  struct fastrpc_channel_ctx {
> @@ -2179,9 +2180,9 @@ static int fastrpc_cb_probe(struct platform_device *pdev)
>  	sess->used = false;
>  	sess->valid = true;
>  	sess->dev = dev;
> +	sess->pa_bits = cctx->dma_mask;

>  	/* Configure where sid will be prepended to pa */
> -	sess->sid_pos =
> -		(cctx->iova_format ? SID_POS_IN_IOVA : DSP_DEFAULT_BUS_WIDTH);
> +	sess->sid_pos = (cctx->iova_format ? SID_POS_IN_IOVA : sess->pa_bits);
>  
>  	if (of_property_read_u32(dev->of_node, "reg", &sess->sid))
>  		dev_info(dev, "FastRPC Session ID not specified in DT\n");
> @@ -2198,9 +2199,9 @@ static int fastrpc_cb_probe(struct platform_device *pdev)
>  		}
>  	}
>  	spin_unlock_irqrestore(&cctx->lock, flags);
> -	rc = dma_set_mask(dev, DMA_BIT_MASK(32));
> +	rc = dma_set_mask(dev, DMA_BIT_MASK(sess->pa_bits));
>  	if (rc) {
> -		dev_err(dev, "32-bit DMA enable failed\n");
> +		dev_err(dev, "%u-bit DMA enable failed\n", sess->pa_bits);
>  		return rc;
>  	}
>  
> @@ -2287,10 +2288,12 @@ static int fastrpc_get_domain_id(const char *domain)
>  
>  struct fastrpc_soc_data {
>  	u32 dsp_iova_format;
> +	u32 cdsp_dma_mask;
>  };
>  
>  static const struct fastrpc_soc_data kaanapali_soc_data = {
>  	.dsp_iova_format = 1,
> +	.cdsp_dma_mask = 34,
>  };
>  
>  static const struct of_device_id qcom_soc_match_table[] = {
> @@ -2310,6 +2313,7 @@ static int fastrpc_rpmsg_probe(struct rpmsg_device *rpdev)
>  	const struct of_device_id *match;
>  	const struct fastrpc_soc_data *soc_data = NULL;
>  	u32 iova_format = 0;
> +	u32 ubs = DSP_DEFAULT_BUS_WIDTH;
>  
>  	root = of_find_node_by_path("/");
>  	if (!root)
> @@ -2322,6 +2326,7 @@ static int fastrpc_rpmsg_probe(struct rpmsg_device *rpdev)
>  	} else {
>  		soc_data = match->data;
>  		iova_format = soc_data->dsp_iova_format;
> +		ubs = soc_data->cdsp_dma_mask;
>  	}
>  
>  	err = of_property_read_string(rdev->of_node, "label", &domain);
> @@ -2404,6 +2409,7 @@ static int fastrpc_rpmsg_probe(struct rpmsg_device *rpdev)
>  	}
>  	/* determine where sid needs to be prepended to pa based on iova_format */
>  	data->iova_format = iova_format;
> +	data->dma_mask = (domain_id == CDSP_DOMAIN_ID ? ubs : DSP_DEFAULT_BUS_WIDTH);

set the default of 32 and move this inside switch case where we have
domain id checks.


>  	kref_init(&data->refcount);
>  
>  	dev_set_drvdata(&rpdev->dev, data);
> 


Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ