On Mon, 23 Sep 2019, Peter Zijlstra wrote:
> On Fri, Sep 20, 2019 at 04:41:03PM -0700, Andy Lutomirski wrote:
> > On Thu, Sep 19, 2019 at 8:09 AM Thomas Gleixner <tglx@xxxxxxxxxxxxx> wrote:
> > >
> > > Replace the syscall entry work handling with the generic version, Provide
> > > the necessary helper inlines to handle the real architecture specific
> > > parts, e.g. audit and seccomp invocations.
> >
> > > - if (work & (_TIF_SYSCALL_TRACE | _TIF_SYSCALL_EMU)) {
> > > - ret = tracehook_report_syscall_entry(regs);
> > > - if (ret || (work & _TIF_SYSCALL_EMU))
> > > - return -1L;
> > > - }
> >
> > Unless I missed something, you lost the _TIF_SYSCALL_EMU abomination.
>
> IIUC that's actually in patch #1.
Correct:
+ if (ti_work & (_TIF_SYSCALL_TRACE | _TIF_SYSCALL_EMU)) {
+ ret = arch_syscall_enter_tracehook(regs);
+ if (ret || (ti_work & _TIF_SYSCALL_EMU))
+ return -1L;
+ }
