lkml.org 
[lkml]   [2009]   [Feb]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH -tip] tracing: remove unused code
    On Sat, Feb 28, 2009 at 07:40:04PM +0000, Luis Henriques wrote:
    > On Sat, Feb 28, 2009 at 08:21:37AM +0100, Ingo Molnar wrote:
    > >
    > > * Steven Rostedt <rostedt@goodmis.org> wrote:
    > >
    > > > Thanks Luis,
    > > >
    > > > But before I remove this code, I want to move it to be
    > > > activated by tracing options. The irqsoff tracer (and other
    > > > latency tracers) are pretty useless without it.
    > >
    > > sounds good.
    >
    > Ok, understood. I am sorry for the noise introduced by this patch... I should
    > have got some more context before submitting it -- I am really not that familiar
    > with this code and tried just to remove a compile warning without fully
    > understand the whole picture :-)


    Don't be sorry, the fix was right.
    But nobody can be aware of all background history in a subsystem, unless one read
    all tracing related posts but it is very active these time so it can be a little pain for
    a newcomer.


    > --
    > Luis Henriques
    > --
    > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    > the body of a message to majordomo@vger.kernel.org
    > More majordomo info at http://vger.kernel.org/majordomo-info.html
    > Please read the FAQ at http://www.tux.org/lkml/



    \
     
     \ /
      Last update: 2009-02-28 20:57    [W:0.024 / U:0.424 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site