Re: [patch 04/18] loongarch/syscall: Use syscall_enter_from_user_mode_randomize_stack()
Jinjie Ruan <[email protected]> Thu, 9 Jul 2026 10:37:44 +0800
| Newsgroups | gmane.linux.ports.alpha,gmane.linux.kernel,gmane.linux.ports.ppc64.devel,gmane.linux.ports.riscv,gmane.linux.ports.arm.kernel,gmane.linux.ports.mips,gmane.linux.ports.parisc,gmane.linux.ports.sh.devel,gmane.linux.uml.devel,gmane.linux.kernel.arc,gmane.linux.ports.hexagon,gmane.linux.ports.sparc,gmane.linux.kernel.cross-arch,gmane.linux.documentation |
|---|---|
| Message-ID | <[email protected]> |
On 7/8/2026 3:06 AM, Thomas Gleixner wrote: > syscall_enter_from_user_mode_randomize_stack() replaces > syscall_enter_from_user_mode() and the subsequent invocation of > add_random_kstack_offset(). > > The advantage is that it applies the stack randomization right after > enter_from_user_mode() and thereby avoids the overhead of get/put_cpu_var() > as that code is invoked with interrupts disabled. > > No functional change. > > Signed-off-by: Thomas Gleixner <[email protected]> > Cc: Huacai Chen <[email protected]> > Cc: [email protected] > --- > arch/loongarch/kernel/syscall.c | 5 +---- > 1 file changed, 1 insertion(+), 4 deletions(-) > > --- a/arch/loongarch/kernel/syscall.c > +++ b/arch/loongarch/kernel/syscall.c > @@ -11,7 +11,6 @@ > #include <linux/linkage.h> > #include <linux/nospec.h> > #include <linux/objtool.h> > -#include <linux/randomize_kstack.h> > #include <linux/syscalls.h> > #include <linux/unistd.h> > > @@ -70,9 +69,7 @@ void noinstr __no_stack_protector do_sys > regs->orig_a0 = regs->regs[4]; > regs->regs[4] = -ENOSYS; > > - nr = syscall_enter_from_user_mode(regs, nr); > - > - add_random_kstack_offset(); > + nr = syscall_enter_from_user_mode_randomize_stack(regs, nr); Reviewed-by: Jinjie Ruan <[email protected]> > > if (nr < NR_syscalls) { > syscall_fn = sys_call_table[array_index_nospec(nr, NR_syscalls)]; >