[lkml]   [2008]   [Aug]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH 0/5] ftrace: to kill a daemon
    On Sat, Aug 9, 2008 at 9:07 PM, Steven Rostedt <> wrote:
    > One of the things I tried to do was to make ftrace port easily to all
    > archs. David Miller ported it to Sparc64 in 20 minutes and that was mostly
    > compiling. Doing a kprobe fix would require much more knowledge to each
    > arch and more prone to errors.

    I agree with the point about porting. My kprobe reference was simply
    to point out that
    an existing module seems to be overwriting code on the fly and doesn't
    take care of much
    on non-x86 archs. Ftrace has a unique 5-byte call replacement issue
    specific to x86 which kprobes doesn't have to deal with.

    >> This solution indeed would fit all archs well but for some it may be
    >> an overkill (or is it?...I'd like to know that :-\ ).
    > It is not an issue at all. The mcount list is discarded with the init
    > section, and the change to nops is relatively fast. We do make a list of
    > all entries anyway, so that we can pick and choose which functions we want
    > to enable tracing on.


    > I want ftrace_record_ip to be as fast as possible, and also having it call
    > the registered function means we need to test if it was set in the filter
    > too. This is too much for what record_ip does. And as I said, doing it all
    > on boot up would be best.


    Abhishek Sagar

     \ /
      Last update: 2008-08-09 19:17    [W:0.022 / U:5.820 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site