Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1327623
| From | Dmitry Safonov <0x7f454c46@gmail.com> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [RFC] trace_irqsoff: remove redundant reset per-CPU buff |
| Date | 2016-02-05 13:00 +0100 |
| Message-ID | <qYNdp-85k-29@gated-at.bofh.it> (permalink) |
| References | <qln2H-3Iv-29@gated-at.bofh.it> <qYED8-2lp-31@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
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 Cc'd myself on this to make sure I see it again.
>
> -- Steve
--
Regards,
Safonov Dmitry.
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
Re: [RFC] trace_irqsoff: remove redundant reset per-CPU buff Steven Rostedt <rostedt@goodmis.org> - 2016-02-05 03:50 +0100
Re: [RFC] trace_irqsoff: remove redundant reset per-CPU buff Dmitry Safonov <0x7f454c46@gmail.com> - 2016-02-05 13:00 +0100
Re: [RFC] trace_irqsoff: remove redundant reset per-CPU buff Steven Rostedt <rostedt@goodmis.org> - 2016-02-05 15:50 +0100
Re: [RFC] trace_irqsoff: remove redundant reset per-CPU buff Steven Rostedt <rostedt@goodmis.org> - 2016-02-05 16:10 +0100
Re: [RFC] trace_irqsoff: remove redundant reset per-CPU buff Dmitry Safonov <0x7f454c46@gmail.com> - 2016-02-05 17:00 +0100
csiph-web