public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Ingo Molnar <mingo@elte.hu>
To: KOSAKI Motohiro <m-kosaki@ceres.dti.ne.jp>
Cc: Mathieu Desnoyers <mathieu.desnoyers@polymtl.ca>,
	akpm@linux-foundation.org, linux-kernel@vger.kernel.org,
	Linus Torvalds <torvalds@linux-foundation.org>,
	"Frank Ch. Eigler" <fche@redhat.com>
Subject: Re: [patch for 2.6.26 0/7] Architecture Independent Markers
Date: Fri, 28 Mar 2008 11:15:00 +0100	[thread overview]
Message-ID: <20080328101500.GF30863@elte.hu> (raw)
In-Reply-To: <2f11576a0803271008t7574ead1s9ea285d8df57ef1@mail.gmail.com>


* KOSAKI Motohiro <m-kosaki@ceres.dti.ne.jp> wrote:

> >      text    data     bss     dec     hex filename
> >     37642    7014     384   45040    aff0 sched.o.before
> >     37930    7134     384   45448    b188 sched.o.after
> >
> >  288 bytes for only 3 markers in the scheduler - that's 96 bytes added
> >  per every marker (44 bytes of that in the fastpath).
> >
> >  and the tracer codepath is now slower and more complex as well.
> >
> >  A classic lose-lose situation, so let's apply this patch. Not!
> >
> >  NACK-ed-by: Ingo Molnar <mingo@elte.hu>
> 
> I wonder why nobody doesn't fix this serious problem until now. and I 
> am interesting to what difference to ftrace and marker.

i warned and moaned about it ad nauseum.

furthermore, and because it's Friday again, let me remind folks that 
SystemTap has an even more significant bloat problem: the fact that it 
needs a huge download:

 Installing:
   kernel-debuginfo         x86_64     2.6.25-0.163.rc7.git1.fc9  
   development-debuginfo  198 M
 Installing for dependencies:
   kernel-debuginfo-common  x86_64     2.6.25-0.163.rc7.git1.fc9  
   development-debuginfo   30 M

 Total download size: 229 M

for _every_ updated kernel. That 229 MB size reduces the user base of 
SystemTap (which is otherwise a totally brilliant and cool tool) to 1% 
of its potential userbase, to those truly desperate persons who really 
_need_ to get their problem debugged somehow. But it's nowhere near 
usable as an easy, ad-hoc kernel instrumentation tool, just due to the 
sheer size it brings.

for heaven's sake, we can have 3 years of _full Linux kernel history_, 
with all 87875 commits, with full changelog and dependencies and full 
source code included, packed into a rather tight, 180 MB git repository 
...

[ and then i havent even begun about the on-disk footprint of this 
  monstrum, after the packages have been installed: an additional 850 MB.
  Puh-lease ... ]

there's a huge disconnect with reality here.

	Ingo

  reply	other threads:[~2008-03-28 10:16 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-03-27 13:20 [patch for 2.6.26 0/7] Architecture Independent Markers Mathieu Desnoyers
2008-03-27 13:20 ` [patch for 2.6.26 1/7] Markers - define non optimized marker Mathieu Desnoyers
2008-03-27 13:20 ` [patch for 2.6.26 2/7] LTTng instrumentation fs Mathieu Desnoyers
2008-03-27 13:21 ` [patch for 2.6.26 3/7] LTTng instrumentation ipc Mathieu Desnoyers
2008-03-27 13:21 ` [patch for 2.6.26 4/7] LTTng instrumentation kernel Mathieu Desnoyers
2008-03-27 13:21 ` [patch for 2.6.26 5/7] LTTng instrumentation mm Mathieu Desnoyers
2008-03-27 13:21 ` [patch for 2.6.26 6/7] LTTng instrumentation net Mathieu Desnoyers
2008-03-27 13:21 ` [patch for 2.6.26 7/7] LTTng instrumentation - lib Mathieu Desnoyers
2008-03-27 15:40 ` [patch for 2.6.26 0/7] Architecture Independent Markers Ingo Molnar
2008-03-27 17:08   ` KOSAKI Motohiro
2008-03-28 10:15     ` Ingo Molnar [this message]
2008-03-28 13:34       ` [OT] " Masami Hiramatsu
2008-04-01  1:43         ` Denys Vlasenko
2008-04-01 14:30           ` Masami Hiramatsu
2008-03-28 13:40       ` Frank Ch. Eigler
2008-03-28 14:18         ` Ingo Molnar
2008-03-28 14:41         ` Ingo Molnar
2008-03-28 15:31           ` Frank Ch. Eigler
2008-03-27 20:39   ` Mathieu Desnoyers
2008-03-28  9:43     ` Ingo Molnar
2008-03-28 11:22       ` Ingo Molnar
2008-03-28 11:38       ` Mathieu Desnoyers
2008-03-28 13:33     ` Ingo Molnar
2008-03-29 17:16       ` Mathieu Desnoyers
2008-03-27 21:49   ` Frank Ch. Eigler
2008-03-28  0:01 ` Denys Vlasenko
2008-03-28  1:02   ` [PATCH] Markers - remove extra format argument Mathieu Desnoyers
2008-03-28  5:35     ` Masami Hiramatsu
2008-03-28  1:04   ` [patch for 2.6.26 1/7] Markers - define non optimized marker (updated) Mathieu Desnoyers

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=20080328101500.GF30863@elte.hu \
    --to=mingo@elte.hu \
    --cc=akpm@linux-foundation.org \
    --cc=fche@redhat.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=m-kosaki@ceres.dti.ne.jp \
    --cc=mathieu.desnoyers@polymtl.ca \
    --cc=torvalds@linux-foundation.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