Re: [PATCH v4 1/2 fix-v2 ] arm64: refactor the rodata=xxx

[Shijie Huang]

On 2025/6/27 23:44, Will Deacon wrote:
> Digging up an old thread...
>
> On Tue, Dec 17, 2024 at 03:17:15PM +0800, Huang Shijie wrote:
> > As per admin guide documentation, "rodata=on" should be the default on
> > platforms. Documentation/admin-guide/kernel-parameters.txt describes
> > these options as
> >
> >     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
> >                     [arm64]
> >
> > But on arm64 platform, "rodata=full" is the default instead. This patch
> > implements the following changes.
> >
> >   - Make "rodata=on" behaviour same as the original "rodata=full"
> >   - Make "rodata=noalias" (new) behaviour same as the original "rodata=on"
> >   - Drop the original "rodata=full"
> >   - Add comment for arch_parse_debug_rodata()
> >   - Update kernel-parameters.txt as required
> >
> > After this patch, the "rodata=on" will be the default on arm64 platform
> > as well.
> >
> > Signed-off-by: Huang Shijie <shijie@xxxxxxxxxxxxxxxxxxxxxx>
> > ---
> > Add more comment for "rodata=noalias" in arch_parse_debug_rodata() from Ard.
> > ---
> >   .../admin-guide/kernel-parameters.txt         |  2 +-
> >   arch/arm64/include/asm/setup.h                | 28 +++++++++++++++++--
> >   2 files changed, 27 insertions(+), 3 deletions(-)
> Sorry, but I'd missed this as you'd sent it as a reply to an existing
> series. When you send a new version of a patch, please can you post it
> as a new thread with an updated version?

Okay, I will rebase this patch set and send out it later..


Thanks

Huang Shijie