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 >