Re: [PATCH v3 1/2] arm64: refactor the rodata=xxx

[Shijie Huang]

On 2024/12/10 1:05, Christoph Lameter (Ampere) wrote:
> > --- a/Documentation/admin-guide/kernel-parameters.txt
> > +++ b/Documentation/admin-guide/kernel-parameters.txt
> > @@ -5901,7 +5901,7 @@
> >   	rodata=		[KNL,EARLY]
> >   		on	Mark read-only kernel memory as read-only (default).
> >   		off	Leave read-only kernel memory writable for debugging.
> > -		full	Mark read-only kernel memory and aliases as read-only
> > +		noalias	Use more block mappings,may have better performance.
> >   		        [arm64]
> Maybe use
> 		noalias Do not check aliases in order to allow larger kernel page
>                          sizes on a platforms without FEAT_BBM2 support[arm64])
>
> instead?

The "rodata=noalias" here is equal to original "rodata=on".

But Yang Shi's patch set works with original "rodata=full" :

https://lists.infradead.org/pipermail/linux-arm-kernel/2024-November/979770.html


IMHO, it's not proper to mention the FEAT_BBM in "rodata=noalias".



Thanks

Huang Shijie