linux-kernel - Re: [RFC] trace_irqsoff: remove redundant reset per-CPU buff

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAJwJo6a+sgGRCaPQMyYy7o2QtS3BhuAuT=ZbBYYLx77WkbY2Pw@mail.gmail.com>
Date:	Fri, 5 Feb 2016 14:52:53 +0300
From:	Dmitry Safonov <0x7f454c46@...il.com>
To:	Steven Rostedt <rostedt@...dmis.org>
Cc:	mingo@...hat.com, linux-kernel@...r.kernel.org
Subject: Re: [RFC] trace_irqsoff: remove redundant reset per-CPU buff

2016-02-05 5:40 GMT+03:00 Steven Rostedt <rostedt@...dmis.org>:
> On Mon, 19 Oct 2015 21:10:26 +0300
> Dmitry Safonov <0x7f454c46@...il.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.