linux-kernel - Re: [PATCH] arm64: dts: imx8qxp: shrink dsp reserved memory

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: <Z+xX9/nwhfYt0Zik@lizhi-Precision-Tower-5810>
Date: Tue, 1 Apr 2025 17:17:43 -0400
From: Frank Li <Frank.li@....com>
To: Laurentiu Mihalcea <laurentiumihalcea111@...il.com>
Cc: Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
	Shawn Guo <shawnguo@...nel.org>,
	Sascha Hauer <s.hauer@...gutronix.de>,
	Iuliana Prodan <iuliana.prodan@....com>,
	Shengjiu Wang <shengjiu.wang@....com>,
	Fabio Estevam <festevam@...il.com>,
	Conor Dooley <conor+dt@...nel.org>,
	Pengutronix Kernel Team <kernel@...gutronix.de>,
	devicetree@...r.kernel.org, imx@...ts.linux.dev,
	linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] arm64: dts: imx8qxp: shrink dsp reserved memory

On Tue, Apr 01, 2025 at 11:52:44AM -0400, Laurentiu Mihalcea wrote:
> From: Laurentiu Mihalcea <laurentiu.mihalcea@....com>

subject suggest because avoid overlap is critcal part.

"shrink dsp reserved memory to avoid overlap with 'dsp_vdev0vring0'"

>
> Shrink the size of the memory reserved for the dsp to half of its original
> size to avoid an overlap with the 'dsp_vdev0vring0' memory region. 16MB of

       to fix overlap error...,

It'd better add a fail log here.

> reserved memory should be enough anyways.

I perfer provide reason why 16MB is enough or just remove it.

>

Add fixes tag here and Cc stable.

Frank

> Signed-off-by: Laurentiu Mihalcea <laurentiu.mihalcea@....com>
> ---
>  arch/arm64/boot/dts/freescale/imx8qxp.dtsi | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/freescale/imx8qxp.dtsi b/arch/arm64/boot/dts/freescale/imx8qxp.dtsi
> index 05138326f0a5..a65c5a2c4a81 100644
> --- a/arch/arm64/boot/dts/freescale/imx8qxp.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx8qxp.dtsi
> @@ -185,7 +185,7 @@ decoder_rpc: decoder-rpc@...00000 {
>  		};
>
>  		dsp_reserved: dsp@...00000 {
> -			reg = <0 0x92400000 0 0x2000000>;
> +			reg = <0 0x92400000 0 0x1000000>;
>  			no-map;
>  			status = "disabled";
>  		};
> --
> 2.34.1
>