diff mbox series

[08/11] ARM: dts: qcom: sdx65-mtp: Increase the vmalloc size

Message ID 1654080312-5408-9-git-send-email-quic_rohiagar@quicinc.com (mailing list archive)
State Queued
Headers show
Series Devicetree Changes SDX65 | expand

Commit Message

Rohit Agarwal June 1, 2022, 10:45 a.m. UTC
Increase the size of vmalloc using the bootargs in sdx65
mtp board.
It is failing to alloacte the size needed for firmware and
giving the error logs due to actual vmalloc region lesser than requested.

cat /proc/meminfo shows the size of VmallocTotal as 245760 kB.

[ 10.980356] vmap allocation for size 268439552 failed: use vmalloc=<size> to increase size
[ 10.980505] qcom_q6v5_pas 4080000.remoteproc: unable to map memory region: 0x90800000+10000000
[ 10.988542] In adsp alloc memory: adsp_probe 482
[ 10.988592] remoteproc remoteproc0: releasing 4080000.remoteproc
[ 11.001598] qcom_q6v5_pas: probe of 4080000.remoteproc failed with error -16

Thus, increasing the size to 300000000 as modem is not the only one using vmalloc region.

Signed-off-by: Rohit Agarwal <quic_rohiagar@quicinc.com>
---
 arch/arm/boot/dts/qcom-sdx65-mtp.dts | 1 +
 1 file changed, 1 insertion(+)

Comments

Krzysztof Kozlowski June 1, 2022, 11:06 a.m. UTC | #1
On 01/06/2022 12:45, Rohit Agarwal wrote:
> Increase the size of vmalloc using the bootargs in sdx65
> mtp board.
> It is failing to alloacte the size needed for firmware and
> giving the error logs due to actual vmalloc region lesser than requested.
> 
> cat /proc/meminfo shows the size of VmallocTotal as 245760 kB.
> 
> [ 10.980356] vmap allocation for size 268439552 failed: use vmalloc=<size> to increase size
> [ 10.980505] qcom_q6v5_pas 4080000.remoteproc: unable to map memory region: 0x90800000+10000000
> [ 10.988542] In adsp alloc memory: adsp_probe 482
> [ 10.988592] remoteproc remoteproc0: releasing 4080000.remoteproc
> [ 11.001598] qcom_q6v5_pas: probe of 4080000.remoteproc failed with error -16
> 
> Thus, increasing the size to 300000000 as modem is not the only one using vmalloc region.
> 
> Signed-off-by: Rohit Agarwal <quic_rohiagar@quicinc.com>
> ---
>  arch/arm/boot/dts/qcom-sdx65-mtp.dts | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/arch/arm/boot/dts/qcom-sdx65-mtp.dts b/arch/arm/boot/dts/qcom-sdx65-mtp.dts
> index 85ea02d..966385f 100644
> --- a/arch/arm/boot/dts/qcom-sdx65-mtp.dts
> +++ b/arch/arm/boot/dts/qcom-sdx65-mtp.dts
> @@ -21,6 +21,7 @@
>  
>  	chosen {
>  		stdout-path = "serial0:115200n8";
> +		bootargs = "vmalloc=300000000";

I understand that issue comes from SoC-related driver but this is not a
property of the hardware. This could be added by bootloader if needed,
but in regular case this is a system setup parameter, so it does not
belong to DTS.

Best regards,
Krzysztof
diff mbox series

Patch

diff --git a/arch/arm/boot/dts/qcom-sdx65-mtp.dts b/arch/arm/boot/dts/qcom-sdx65-mtp.dts
index 85ea02d..966385f 100644
--- a/arch/arm/boot/dts/qcom-sdx65-mtp.dts
+++ b/arch/arm/boot/dts/qcom-sdx65-mtp.dts
@@ -21,6 +21,7 @@ 
 
 	chosen {
 		stdout-path = "serial0:115200n8";
+		bootargs = "vmalloc=300000000";
 	};
 
 	reserved-memory {