On Fri, Nov 13 2020 at 22:29, Gabriel Krisman Bertazi wrote: "kernel: entry:" is not the right subsystem prefix.
git log kernel/entry/ might give you a hint. > Prepares the common entry code to use the SYSCALL_WORK flags. They > will s/Prepares/Prepare/ > be defined in subsequent patches for each type of syscall > work. SYSCALL_WORK_ENTRY/EXIT are defined for the transition, as they > will replace the TIF_ equivalent defines. > > Signed-off-by: Gabriel Krisman Bertazi <[email protected]> > --- > include/linux/entry-common.h | 3 +++ > kernel/entry/common.c | 15 +++++++++------ > 2 files changed, 12 insertions(+), 6 deletions(-) > > diff --git a/include/linux/entry-common.h b/include/linux/entry-common.h > index 1a128baf3628..cbc5c702ee4d 100644 > --- a/include/linux/entry-common.h > +++ b/include/linux/entry-common.h > @@ -64,6 +64,9 @@ > (_TIF_SYSCALL_TRACE | _TIF_SYSCALL_AUDIT | \ > _TIF_SYSCALL_TRACEPOINT | ARCH_SYSCALL_EXIT_WORK) > > +#define SYSCALL_WORK_ENTER (0) > +#define SYSCALL_WORK_EXIT (0) > + > /* > * TIF flags handled in exit_to_user_mode_loop() > */ > diff --git a/kernel/entry/common.c b/kernel/entry/common.c > index bc75c114c1b3..5a4bb72ff28e 100644 > --- a/kernel/entry/common.c > +++ b/kernel/entry/common.c > @@ -42,7 +42,7 @@ static inline void syscall_enter_audit(struct pt_regs > *regs, long syscall) > } > > static long syscall_trace_enter(struct pt_regs *regs, long syscall, > - unsigned long ti_work) > + unsigned long ti_work, unsigned long work) > { > long ret = 0; > > @@ -75,10 +75,11 @@ static __always_inline long > __syscall_enter_from_user_work(struct pt_regs *regs, long syscall) > { > unsigned long ti_work; > + unsigned long work = READ_ONCE(current_thread_info()->syscall_work); Even if this is temporary this code uses reverse fir tree ordering of variable declarations: unsigned long work = READ_ONCE(current_thread_info()->syscall_work); unsigned long ti_work; Thanks, tglx
