lkml.org 
[lkml]   [2016]   [Feb]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [RFC] trace_irqsoff: remove redundant reset per-CPU buff
On Fri, 5 Feb 2016 14:52:53 +0300
Dmitry Safonov <0x7f454c46@gmail.com> wrote:

> 2016-02-05 5:40 GMT+03:00 Steven Rostedt <rostedt@goodmis.org>:
> > On Mon, 19 Oct 2015 21:10:26 +0300
> > Dmitry Safonov <0x7f454c46@gmail.com> wrote:
> >
> >> There is no reason to do it twice: from commit b6f11df26fdc28
> >> ("trace: Call tracing_reset_online_cpus before tracer->init()")
> >> resetting of per-CPU buffers done before tracer->init() call.
> >>
> >> tracer->init() calls {irqs,preempt,preemptirqs}off_tracer_init() and it
> >> calls __irqsoff_tracer_init(), which resets per-CPU ringbuffer second
> >> time.
> >> It's slowpath, but anyway.
> >
> > Bah, I said I'd look at this and I did. But never added it to my queue.
> > It's not a big deal as this is the slow path but it makes sense. I knew
> > there was a patch from you to get but saw the TP_ARGS redefining patch
> > and thought that was it.
>
> Thanks, Steve!
> Yeah, I know it may take time to get to unimportant patches
> that are also mostly cleanups.
> Don't worry, please, take a glance when you'll have time on these
> small also not important cleanings (I'm fine if you'll drop them all):
> https://lkml.org/lkml/2015/11/20/290
> https://lkml.org/lkml/2015/11/20/291
> https://lkml.org/lkml/2015/10/30/580

I'm slowing going through my INBOX (doing clean up to see what I
missed). I'm currently at 11/10. I marked your 10/30 one already as a
"go back and look at later".

-- Steve

\
 
 \ /
  Last update: 2016-02-05 16:01    [W:0.405 / U:0.036 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site