Re: [PATCH] Documentation: Add ARM64 for reserving ZONE_MOVABLE

[Dev Jain <dev.jain@arm.com>]

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 <dev.jain@arm.com>
>> ---
>>   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