[lkml]   [2008]   [Nov]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH 2/5] x86: ret_from_fork - get rid of jump back
    On Fri, Nov 28, 2008 at 02:47:54PM +0100, Ingo Molnar wrote:

    Hi Ingo,

    > * Andi Kleen <> wrote:
    > > > But it gets worse than that: checking for _TIF_SYSCALL_TRACE is
    > > > completely unnecessary here because we clear that flag for every
    > >
    > > That's true. I found your 2005 changeset which did that, [...]
    > that's irrelevant, because all the necessary TIF_ flag processing is
    > already done in ret_from_sys_call.

    Yes, that's true (as I wrote in my email), but it's just not dead because
    the audit code will take it (or used to handle both audit and fork before
    2005), just unnecessary because the later code checks it as well@). Ok it's
    only a very subtle difference given.

    It would have been nice if you had pruned at least the fork case
    when you changed fork.c to always clear it, but that can be also
    done now.

    > The unnecessary TIF_SYSCALL_TRACE code in the 64-bit ret_from_fork was
    > apparently added by you in 2002:

    Yes, to be honest I don't remember why I did it this way. Most likely
    the additional check was needed in some earlier iteration and then not pruned
    away when it became unnecessary. I remember I had some ordering problems
    with strace very early on it might have been related to that. Or perhaps
    I just goofed up. In the current form it's certainly not great code.

    Anyways I have no objections to removing it now, just one has to be
    careful that the changed processing order (signals vs trace) -- which
    is likely user visible -- won't break anything.


     \ /
      Last update: 2008-11-28 19:47    [W:0.020 / U:83.660 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site