[lkml]   [2011]   [Jan]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [RFC] [PATCH 2.6.37-rc5-tip 10/20] 10: uprobes: task specific information.
    > On 01/25/2011 05:56 AM, Peter Zijlstra wrote:
    > > Ah, I think I found it while reading patch 13, you need the pre/post_xol
    > > callbacks, can't you simply synthesize their effect into the replacement
    > > sequence?
    > >
    > > push %rax
    > > mov $vaddr, %rax
    > > $INSN
    > > pop %rax
    > > jmp $next_insn
    > >
    > > like replacements would obviate the need for the pre/post callbacks and
    > > allow you to run straight through.
    > For this particular example, you'd better be sure that $INSN doesn't
    > need %rsp intact.

    In general it is quite bad form to touch the user's stack at all for
    instrumentation purposes. Unexpected stack usage might be what you are
    trying to debug, after all.

    On x86-64 in particular, it is strictly verboten to touch the user's stack
    immediately below the SP. In the x86-64 ABI, the 128 bytes below %rsp are
    a scratch area for leaf functions that normal compiled user code will use
    to store data that must not be clobbered. (Normal signal handler frames
    start 128 bytes below %rsp for this reason.)

    That's aside from the more obvious issues Josh mentioned, where the
    instruction itself is a push/pop/call/ret or uses an addressing mode
    relative to %rsp.


     \ /
      Last update: 2011-01-25 19:59    [W:0.021 / U:0.708 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site