From: Ingo Molnar <mingo@kernel.org>
To: Steven Rostedt <rostedt@goodmis.org>
Cc: Andi Kleen <andi@firstfloor.org>,
x86@kernel.org, linux-kernel@vger.kernel.org,
peterz@infradead.org, Andi Kleen <ak@linux.intel.com>,
fweisbec@gmail.com
Subject: Re: [PATCH 6/6] x86: Allow disabling HW_BREAKPOINTS and PERF_EVENTS
Date: Tue, 8 Oct 2013 21:39:27 +0200 [thread overview]
Message-ID: <20131008193927.GB7315@gmail.com> (raw)
In-Reply-To: <20131008153518.GB15214@home.goodmis.org>
* Steven Rostedt <rostedt@goodmis.org> wrote:
> On Tue, Oct 08, 2013 at 08:59:38AM +0200, Ingo Molnar wrote:
> >
> > Secondly and more importantly, visualized with relative sizes, in a
> > feature matrix, makes it clearer what's going on with vmlinux .text:
> >
> > perf-OFF perf-ON
> >
> > ftrace-OFF 0 +151k
> >
> > ftrace-ON +897k +1142k
> >
> > So basically ftrace is causing a big chunk of the instrumentation size
> > increase. With tons of tracers and lots of kernel subsystems built into
> > your .config that's a lot of nice instrumentation functionality and it's
> > thus also a natural end result IMO.
>
> I'm curious what you turned on for between the "ftrace-OFF" and "ftrace-ON"?
ON: all the tracing bits in Andi's original config - which had most
tracing options enabled AFAICS.
OFF: all of it turned off.
I don't think there's anything inherently wrong with 3000+ tracepoints
built into the kernel using up a fair amount of cold-path space. We should
try to compress it all, but it's a hell of a lot real functionality.
> The one thing that you didn't post was CONFIG_EVENT_TRACING. This is
> where a lot of bloat can come from, as this enables the infrastructure
> for tracepoints. This is also needed by perf to trace most sw events.
Correct.
EVENT_TRACING was on - Andi did not post his .config publicly, but you can
get something fairly close to his config by doing this:
make defconfig
cat config-options.txt >> .config
make oldconfig
where 'config-options.txt' is the short list of options I appended to my
mail.
> The trace event infrastructure that both perf and ftrace uses (as well
> as other tracers and tools) is well known to cause bloat in the kernel.
> There's patches to help make it more bearable, but there's more work to
> be done.
It's 3000+ very nice tracepoints - which is a huge multiplier, so we
should not expect any miracles - it's a whole lot of good functionality.
Subsystems can decide whether they want to expose their tracepoints or
not.
Thanks,
Ingo
next prev parent reply other threads:[~2013-10-08 19:39 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-10-04 21:39 Allow disabling perf on x86 Andi Kleen
2013-10-04 21:39 ` [PATCH 1/6] perf, x86, amd: Move __get_ibs_caps into common amd CPU file Andi Kleen
2013-10-04 21:39 ` [PATCH 2/6] perf, x86: Make perf amd ibs code depend on PERF_EVENTS and SUP_AMD Andi Kleen
2013-10-04 21:39 ` [PATCH 3/6] x86, ptrace: Ifdef HW_BREAKPOINTS code in ptrace Andi Kleen
2013-10-04 21:39 ` [PATCH 4/6] trace: Make UPROBES depend on PERF_EVENTS Andi Kleen
2013-10-05 0:52 ` Steven Rostedt
2013-10-05 3:25 ` Andi Kleen
2013-10-04 21:39 ` [PATCH 5/6] x86, kgdb: Support compiling without hardware break points Andi Kleen
2013-10-04 21:39 ` [PATCH 6/6] x86: Allow disabling HW_BREAKPOINTS and PERF_EVENTS Andi Kleen
2013-10-04 22:27 ` Frederic Weisbecker
2013-10-05 7:08 ` Ingo Molnar
2013-10-05 17:05 ` Andi Kleen
2013-10-08 19:55 ` Frederic Weisbecker
2013-10-08 20:05 ` Peter Zijlstra
2013-10-08 20:34 ` Frederic Weisbecker
2013-10-08 20:35 ` Frederic Weisbecker
2013-10-09 6:02 ` Ingo Molnar
2013-10-06 16:49 ` Ingo Molnar
2013-10-08 6:59 ` Ingo Molnar
2013-10-08 15:35 ` Steven Rostedt
2013-10-08 19:39 ` Ingo Molnar [this message]
2013-10-08 15:42 ` Vince Weaver
2013-10-08 15:51 ` Dave Jones
2013-10-08 16:22 ` David Ahern
2013-10-08 16:24 ` Dave Jones
2013-10-08 19:36 ` Ingo Molnar
2013-10-08 20:19 ` Steven Rostedt
2013-10-09 3:21 ` Andi Kleen
2013-10-09 3:24 ` Andi Kleen
2013-10-09 3:39 ` Andi Kleen
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20131008193927.GB7315@gmail.com \
--to=mingo@kernel.org \
--cc=ak@linux.intel.com \
--cc=andi@firstfloor.org \
--cc=fweisbec@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=peterz@infradead.org \
--cc=rostedt@goodmis.org \
--cc=x86@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).