[v2] Further bringup of SHIFTphone 8

[PATCH v2 3/6] arm64: dts: qcom: qcm6490-shift-otter: Add missing reserved-memory

Posted by Luca Weiss 2 months, 1 week ago

From: Alexander Martinz <amartinz@shiftphones.com>

It seems we also need to reserve a region of 81 MiB called "removed_mem"
otherwise we can easily hit memory errors with higher RAM usage.

Fixes: 249666e34c24 ("arm64: dts: qcom: add QCM6490 SHIFTphone 8")
Signed-off-by: Alexander Martinz <amartinz@shiftphones.com>
Reviewed-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
Signed-off-by: Luca Weiss <luca.weiss@fairphone.com>
---
 arch/arm64/boot/dts/qcom/qcm6490-shift-otter.dts | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/arch/arm64/boot/dts/qcom/qcm6490-shift-otter.dts b/arch/arm64/boot/dts/qcom/qcm6490-shift-otter.dts
index 0d331bda4a82..31650c29b1ca 100644
--- a/arch/arm64/boot/dts/qcom/qcm6490-shift-otter.dts
+++ b/arch/arm64/boot/dts/qcom/qcm6490-shift-otter.dts
@@ -118,6 +118,11 @@ cdsp_mem: cdsp@88f00000 {
 			no-map;
 		};
 
+		removed_mem: removed@c0000000 {
+			reg = <0x0 0xc0000000 0x0 0x5100000>;
+			no-map;
+		};
+
 		rmtfs_mem: rmtfs@f8500000 {
 			compatible = "qcom,rmtfs-mem";
 			reg = <0x0 0xf8500000 0x0 0x600000>;

-- 
2.51.0

Re: [PATCH v2 3/6] arm64: dts: qcom: qcm6490-shift-otter: Add missing reserved-memory

Posted by Bjorn Andersson 1 month, 3 weeks ago

On Thu, Oct 09, 2025 at 11:06:33AM +0200, Luca Weiss wrote:
> From: Alexander Martinz <amartinz@shiftphones.com>
> 
> It seems we also need to reserve a region of 81 MiB called "removed_mem"
> otherwise we can easily hit memory errors with higher RAM usage.
> 

If you make sure CONFIG_MEMTEST is enabled, you can boot with memtest=1
on the command line to catch such issues, without relying on "higher RAM
usage" (or randomness).

Regards,
Bjorn

> Fixes: 249666e34c24 ("arm64: dts: qcom: add QCM6490 SHIFTphone 8")
> Signed-off-by: Alexander Martinz <amartinz@shiftphones.com>
> Reviewed-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
> Signed-off-by: Luca Weiss <luca.weiss@fairphone.com>
> ---
>  arch/arm64/boot/dts/qcom/qcm6490-shift-otter.dts | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/qcom/qcm6490-shift-otter.dts b/arch/arm64/boot/dts/qcom/qcm6490-shift-otter.dts
> index 0d331bda4a82..31650c29b1ca 100644
> --- a/arch/arm64/boot/dts/qcom/qcm6490-shift-otter.dts
> +++ b/arch/arm64/boot/dts/qcom/qcm6490-shift-otter.dts
> @@ -118,6 +118,11 @@ cdsp_mem: cdsp@88f00000 {
>  			no-map;
>  		};
>  
> +		removed_mem: removed@c0000000 {
> +			reg = <0x0 0xc0000000 0x0 0x5100000>;
> +			no-map;
> +		};
> +
>  		rmtfs_mem: rmtfs@f8500000 {
>  			compatible = "qcom,rmtfs-mem";
>  			reg = <0x0 0xf8500000 0x0 0x600000>;
> 
> -- 
> 2.51.0
>

Re: [PATCH v2 3/6] arm64: dts: qcom: qcm6490-shift-otter: Add missing reserved-memory

Posted by Luca Weiss 1 month, 1 week ago

On Mon Oct 27, 2025 at 5:45 PM CET, Bjorn Andersson wrote:
> On Thu, Oct 09, 2025 at 11:06:33AM +0200, Luca Weiss wrote:
>> From: Alexander Martinz <amartinz@shiftphones.com>
>> 
>> It seems we also need to reserve a region of 81 MiB called "removed_mem"
>> otherwise we can easily hit memory errors with higher RAM usage.
>> 
>
> If you make sure CONFIG_MEMTEST is enabled, you can boot with memtest=1
> on the command line to catch such issues, without relying on "higher RAM
> usage" (or randomness).

Good idea, ran it now and no issue seen.

[    0.000000] Machine model: SHIFT SHIFTphone 8
     <...>
[    0.000000] early_memtest: # of tests: 1
[    0.000000]   0x0000000080880000 - 0x0000000080884000 pattern 0000000000000000
[    0.000000]   0x0000000080894000 - 0x00000000808ff000 pattern 0000000000000000
[    0.000000]   0x0000000081800000 - 0x0000000086700000 pattern 0000000000000000
[    0.000000]   0x000000008ad00000 - 0x000000008b200000 pattern 0000000000000000
[    0.000000]   0x000000008b710000 - 0x000000008b71a000 pattern 0000000000000000
[    0.000000]   0x000000008b71c000 - 0x000000008b800000 pattern 0000000000000000
[    0.000000]   0x000000009c700000 - 0x00000000a0080000 pattern 0000000000000000
[    0.000000]   0x00000000a18b0000 - 0x00000000af41e000 pattern 0000000000000000
[    0.000000]   0x00000000af43fe53 - 0x00000000af61f000 pattern 0000000000000000
[    0.000000]   0x00000000affff000 - 0x00000000b7100000 pattern 0000000000000000
[    0.000000]   0x00000000c5100000 - 0x00000000e1000000 pattern 0000000000000000
[    0.000000]   0x00000000e3300000 - 0x00000000f8500000 pattern 0000000000000000
[    0.000000]   0x00000000f8b00000 - 0x000000037e857c78 pattern 0000000000000000
[    0.000000]   0x000000037e857ca7 - 0x000000037e857ca8 pattern 0000000000000000
[    0.000000]   0x000000037e857cd7 - 0x000000037e857cd8 pattern 0000000000000000
[    0.000000]   0x000000037e857d04 - 0x000000037e857d08 pattern 0000000000000000
[    0.000000]   0x000000037e857d34 - 0x000000037e857d38 pattern 0000000000000000
[    0.000000]   0x000000037e857d64 - 0x000000037e857d68 pattern 0000000000000000
[    0.000000]   0x000000037e857d94 - 0x000000037e857d98 pattern 0000000000000000
[    0.000000]   0x000000037e857dc4 - 0x000000037e857dc8 pattern 0000000000000000
[    0.000000]   0x000000037e857df4 - 0x000000037e857df8 pattern 0000000000000000
[    0.000000]   0x000000037e857e24 - 0x000000037e857e28 pattern 0000000000000000
[    0.000000]   0x000000037e857e54 - 0x000000037e857e58 pattern 0000000000000000
[    0.000000]   0x000000037e857e84 - 0x000000037e857e88 pattern 0000000000000000
[    0.000000]   0x000000037e857eb4 - 0x000000037e857eb8 pattern 0000000000000000
[    0.000000]   0x000000037e857ee4 - 0x000000037e857ee8 pattern 0000000000000000
[    0.000000]   0x000000037e857f14 - 0x000000037e857f18 pattern 0000000000000000
[    0.000000]   0x000000037e857f44 - 0x000000037e857f48 pattern 0000000000000000
[    0.000000]   0x000000037e857f74 - 0x000000037e857f78 pattern 0000000000000000
[    0.000000]   0x000000037e857fa4 - 0x000000037e857fa8 pattern 0000000000000000
[    0.000000]   0x000000037e857fd4 - 0x000000037e857fd8 pattern 0000000000000000
[    0.000000]   0x000000037e858004 - 0x000000037e858008 pattern 0000000000000000
[    0.000000]   0x000000037e858034 - 0x000000037e858038 pattern 0000000000000000
[    0.000000]   0x000000037e858064 - 0x000000037e858068 pattern 0000000000000000
[    0.000000]   0x000000037e858094 - 0x000000037e858098 pattern 0000000000000000
[    0.000000]   0x000000037e8580c4 - 0x000000037e8580c8 pattern 0000000000000000
[    0.000000]   0x000000037e8580f4 - 0x000000037e8580f8 pattern 0000000000000000
[    0.000000]   0x000000037e858124 - 0x000000037e858128 pattern 0000000000000000
[    0.000000]   0x000000037e858154 - 0x000000037e858158 pattern 0000000000000000
[    0.000000]   0x000000037e858184 - 0x000000037e858188 pattern 0000000000000000
[    0.000000]   0x000000037e8581b4 - 0x000000037e8581b8 pattern 0000000000000000
[    0.000000]   0x000000037e8581e4 - 0x000000037e8581e8 pattern 0000000000000000
[    0.000000]   0x000000037e858214 - 0x000000037e858218 pattern 0000000000000000
[    0.000000]   0x000000037e858244 - 0x000000037e858248 pattern 0000000000000000
[    0.000000]   0x000000037e858274 - 0x000000037e858278 pattern 0000000000000000
[    0.000000]   0x000000037e8582a4 - 0x000000037e8582a8 pattern 0000000000000000
[    0.000000]   0x000000037e8582d4 - 0x000000037e8582d8 pattern 0000000000000000
[    0.000000]   0x000000037e858304 - 0x000000037e858308 pattern 0000000000000000
[    0.000000]   0x000000037e858334 - 0x000000037e858338 pattern 0000000000000000

Regards
Luca

>
> Regards,
> Bjorn
>
>> Fixes: 249666e34c24 ("arm64: dts: qcom: add QCM6490 SHIFTphone 8")
>> Signed-off-by: Alexander Martinz <amartinz@shiftphones.com>
>> Reviewed-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
>> Signed-off-by: Luca Weiss <luca.weiss@fairphone.com>
>> ---
>>  arch/arm64/boot/dts/qcom/qcm6490-shift-otter.dts | 5 +++++
>>  1 file changed, 5 insertions(+)
>> 
>> diff --git a/arch/arm64/boot/dts/qcom/qcm6490-shift-otter.dts b/arch/arm64/boot/dts/qcom/qcm6490-shift-otter.dts
>> index 0d331bda4a82..31650c29b1ca 100644
>> --- a/arch/arm64/boot/dts/qcom/qcm6490-shift-otter.dts
>> +++ b/arch/arm64/boot/dts/qcom/qcm6490-shift-otter.dts
>> @@ -118,6 +118,11 @@ cdsp_mem: cdsp@88f00000 {
>>  			no-map;
>>  		};
>>  
>> +		removed_mem: removed@c0000000 {
>> +			reg = <0x0 0xc0000000 0x0 0x5100000>;
>> +			no-map;
>> +		};
>> +
>>  		rmtfs_mem: rmtfs@f8500000 {
>>  			compatible = "qcom,rmtfs-mem";
>>  			reg = <0x0 0xf8500000 0x0 0x600000>;
>> 
>> -- 
>> 2.51.0
>>

Re: [PATCH v2 3/6] arm64: dts: qcom: qcm6490-shift-otter: Add missing reserved-memory

Posted by Dmitry Baryshkov 2 months, 1 week ago

On Thu, Oct 09, 2025 at 11:06:33AM +0200, Luca Weiss wrote:
> From: Alexander Martinz <amartinz@shiftphones.com>
> 
> It seems we also need to reserve a region of 81 MiB called "removed_mem"
> otherwise we can easily hit memory errors with higher RAM usage.
> 
> Fixes: 249666e34c24 ("arm64: dts: qcom: add QCM6490 SHIFTphone 8")
> Signed-off-by: Alexander Martinz <amartinz@shiftphones.com>
> Reviewed-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
> Signed-off-by: Luca Weiss <luca.weiss@fairphone.com>
> ---
>  arch/arm64/boot/dts/qcom/qcm6490-shift-otter.dts | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/qcom/qcm6490-shift-otter.dts b/arch/arm64/boot/dts/qcom/qcm6490-shift-otter.dts

Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>


-- 
With best wishes
Dmitry