Re: [PATCH 3/3] LoongArch: Skip relocation-time KASLR if it has already been applied

Huacai Chen <[email protected]>
Newsgroups org.kernel.vger.linux-efi,dev.linux.lists.loongarch,org.kernel.vger.linux-kernel
Message-ID <CAAhV-H7+B6=Uf1ZDy9SLvWkR-ggTi3k9kFU8+B+JefWqPu4Uvw@mail.gmail.com>
Hi, Rui,

On Mon, Apr 27, 2026 at 6:47 PM WANG Rui <[email protected]> wrote:
>
> When the kernel is relocated during early boot, a randomized load
> address may already have been selected and applied. In this case,
> performing KASLR again in relocate.c is unnecessary.
>
> Signed-off-by: WANG Rui <[email protected]>
> ---
>  arch/loongarch/kernel/relocate.c | 4 ++++
>  1 file changed, 4 insertions(+)
>
> diff --git a/arch/loongarch/kernel/relocate.c b/arch/loongarch/kernel/relocate.c
> index 16f6a9b39659..c36604a81d08 100644
> --- a/arch/loongarch/kernel/relocate.c
> +++ b/arch/loongarch/kernel/relocate.c
> @@ -139,6 +139,10 @@ static inline __init bool kaslr_disabled(void)
>         char *str;
>         const char *builtin_cmdline = CONFIG_CMDLINE;
>
> +       /* KASLR is performed during early boot. */
> +       if (kaslr_offset())
> +               return true;
Maybe the "kexec_file" case is also handled by this?

Huacai

> +
>         str = strstr(builtin_cmdline, "nokaslr");
>         if (str == builtin_cmdline || (str > builtin_cmdline && *(str - 1) == ' ')) {
>                 pr_info(KASLR_DISABLED_MESSAGE, "\'nokaslr\'", "built-in");
> --
> 2.54.0
>
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.