Re: [PATCH v4 2/4] x86: simplify _TIF_SYSCALL_EMU handling
From: Thomas Gleixner
Date: Tue Jun 11 2019 - 10:43:20 EST
On Thu, 23 May 2019, Sudeep Holla wrote:
$Subject: Please use the proper prefix and start the sentence with an upper
case letter.
x86/entry: Simplify _TIF_SYSCALL_EMU handling
> The usage of emulated/_TIF_SYSCALL_EMU flags in syscall_trace_enter
> seems to be bit overcomplicated than required. Let's simplify it.
s/seems to be bit overcomplicated/is more complicated/
Either you are sure that it is overengineered, then say so. If not, then
you should not touch the code at all.
s/Let's simplify it.//
'Let's do X.' is a popular, but technically useless phrase.
> Cc: Andy Lutomirski <luto@xxxxxxxxxx>
> Cc: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
> Cc: Ingo Molnar <mingo@xxxxxxxxxx>
> Cc: Borislav Petkov <bp@xxxxxxxxx>
> Acked-by: Oleg Nesterov <oleg@xxxxxxxxxx>
> Signed-off-by: Sudeep Holla <sudeep.holla@xxxxxxx>
This is a nice simplification indeed! With the changelog fixed:
Reviewed-by: Thomas Gleixner <tglx@xxxxxxxxxxxxx>