lkml.org 
[lkml]   [2020]   [Nov]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH 03/10] kernel: entry: Wire up syscall_work in common entry code
Date
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 <krisman@collabora.com>
> ---
> 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

\
 
 \ /
  Last update: 2020-11-15 19:41    [W:0.076 / U:6.676 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site