2024-06-04 05:30:00

by Dev Jain

[permalink] [raw]
Subject: [PATCH] Documentation: Add ARM64 for reserving ZONE_MOVABLE

kernelcore and movablecore kernel command line works for ARM64. Update
the Documentation to reflect the same.

Signed-off-by: Dev Jain <[email protected]>
---
Documentation/admin-guide/kernel-parameters.txt | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/Documentation/admin-guide/kernel-parameters.txt b/Documentation/admin-guide/kernel-parameters.txt
index b600df82669d..7282d6057e32 100644
--- a/Documentation/admin-guide/kernel-parameters.txt
+++ b/Documentation/admin-guide/kernel-parameters.txt
@@ -2544,7 +2544,7 @@

keepinitrd [HW,ARM] See retain_initrd.

- kernelcore= [KNL,X86,IA-64,PPC,EARLY]
+ kernelcore= [KNL,X86,IA-64,PPC,ARM64,EARLY]
Format: nn[KMGTPE] | nn% | "mirror"
This parameter specifies the amount of memory usable by
the kernel for non-movable allocations. The requested
@@ -3612,7 +3612,7 @@
mousedev.yres= [MOUSE] Vertical screen resolution, used for devices
reporting absolute coordinates, such as tablets

- movablecore= [KNL,X86,IA-64,PPC,EARLY]
+ movablecore= [KNL,X86,IA-64,PPC,ARM64,EARLY]
Format: nn[KMGTPE] | nn%
This parameter is the complement to kernelcore=, it
specifies the amount of memory used for migratable
--
2.34.1



2024-06-07 11:34:56

by Ryan Roberts

[permalink] [raw]
Subject: Re: [PATCH] Documentation: Add ARM64 for reserving ZONE_MOVABLE

On 04/06/2024 06:28, Dev Jain wrote:
> kernelcore and movablecore kernel command line works for ARM64. Update
> the Documentation to reflect the same.
>
> Signed-off-by: Dev Jain <[email protected]>
> ---
> Documentation/admin-guide/kernel-parameters.txt | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/Documentation/admin-guide/kernel-parameters.txt b/Documentation/admin-guide/kernel-parameters.txt
> index b600df82669d..7282d6057e32 100644
> --- a/Documentation/admin-guide/kernel-parameters.txt
> +++ b/Documentation/admin-guide/kernel-parameters.txt
> @@ -2544,7 +2544,7 @@
>
> keepinitrd [HW,ARM] See retain_initrd.
>
> - kernelcore= [KNL,X86,IA-64,PPC,EARLY]
> + kernelcore= [KNL,X86,IA-64,PPC,ARM64,EARLY]

Given these are both arch-agnostic parameters, wouldn't the correct change be to
remove all the arch-specific labels, i.e. [KNL,EARLY]? In fact, wasn't IA-64
removed from the kernel?

> Format: nn[KMGTPE] | nn% | "mirror"
> This parameter specifies the amount of memory usable by
> the kernel for non-movable allocations. The requested
> @@ -3612,7 +3612,7 @@
> mousedev.yres= [MOUSE] Vertical screen resolution, used for devices
> reporting absolute coordinates, such as tablets
>
> - movablecore= [KNL,X86,IA-64,PPC,EARLY]
> + movablecore= [KNL,X86,IA-64,PPC,ARM64,EARLY]
> Format: nn[KMGTPE] | nn%
> This parameter is the complement to kernelcore=, it
> specifies the amount of memory used for migratable


2024-06-11 10:17:09

by Dev Jain

[permalink] [raw]
Subject: Re: [PATCH] Documentation: Add ARM64 for reserving ZONE_MOVABLE


On 6/7/24 17:04, Ryan Roberts wrote:
> On 04/06/2024 06:28, Dev Jain wrote:
>> kernelcore and movablecore kernel command line works for ARM64. Update
>> the Documentation to reflect the same.
>>
>> Signed-off-by: Dev Jain <[email protected]>
>> ---
>> Documentation/admin-guide/kernel-parameters.txt | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/Documentation/admin-guide/kernel-parameters.txt b/Documentation/admin-guide/kernel-parameters.txt
>> index b600df82669d..7282d6057e32 100644
>> --- a/Documentation/admin-guide/kernel-parameters.txt
>> +++ b/Documentation/admin-guide/kernel-parameters.txt
>> @@ -2544,7 +2544,7 @@
>>
>> keepinitrd [HW,ARM] See retain_initrd.
>>
>> - kernelcore= [KNL,X86,IA-64,PPC,EARLY]
>> + kernelcore= [KNL,X86,IA-64,PPC,ARM64,EARLY]
> Given these are both arch-agnostic parameters, wouldn't the correct change be to
> remove all the arch-specific labels, i.e. [KNL,EARLY]? In fact, wasn't IA-64
> removed from the kernel?


Indeed, the handling of these parameters completely resides in

generic mm; I'll send a separate patch and CC Andrew. And yes,

IA-64 was removed, but still has some mentions remaining

throughout the tree :)

>
>> Format: nn[KMGTPE] | nn% | "mirror"
>> This parameter specifies the amount of memory usable by
>> the kernel for non-movable allocations. The requested
>> @@ -3612,7 +3612,7 @@
>> mousedev.yres= [MOUSE] Vertical screen resolution, used for devices
>> reporting absolute coordinates, such as tablets
>>
>> - movablecore= [KNL,X86,IA-64,PPC,EARLY]
>> + movablecore= [KNL,X86,IA-64,PPC,ARM64,EARLY]
>> Format: nn[KMGTPE] | nn%
>> This parameter is the complement to kernelcore=, it
>> specifies the amount of memory used for migratable