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

[Dev Jain]

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@xxxxxxx>
> > ---
> >   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