From: Tim Bird <tim.bird@am.sony.com>
To: Chris Snook <csnook@redhat.com>
Cc: linux-embedded <linux-embedded@vger.kernel.org>,
linux kernel <linux-kernel@vger.kernel.org>
Subject: Re: RFC - size tool for kernel build system
Date: Wed, 8 Oct 2008 12:32:11 -0700 [thread overview]
Message-ID: <48ED0ABB.2040607@am.sony.com> (raw)
In-Reply-To: <48ED0575.4060005@redhat.com>
Chris Snook wrote:
> The kernel build system is supposed to be stateless, and integrating
> this with make would mess that up.
> If your goal is to get more people
> to use Bloatwatch
Not really. Bloatwatch is a means to an end. It's
only the end I care about, which is more visibility
into kernel size growth over time.
> so they don't make your job quite as difficult, it
> would probably be more appropriate to put a size analysis script in
> scripts/ (like checkpatch.pl) that looks at only the kernel you just
> built and generates thorough statistics in a format readable by both
> humans and Bloatwatch, preferably something easily diffed.
The intent would be to do as you describe (although I don't
really care if the format is readable by Bloatwatch). I'm a
Bloatwatch proponent (I'm not the author), but this suggestion
doesn't really have anything to do with Bloatwatch. Maybe I should
have avoided mentioning it in my initial post.
Having a make target to run the script is just
a method of making it as easy as possible to run it.
(Kind of like 'make checkstack' or 'make cscope')
checkpatch.pl is different in that it operates on something
not in the tree yet. It doesn't make sense as a make target
so I don't think it's comparable.
Maybe it would be simpler to omit the baseline comparison
capability at first, and just focus on a canonical size
report (script and report format) for the kernel.
> Then
> developers could use that output in mailing list discussions without
> having to use Bloatwatch, but embedded developers who care about this
> enough to use Bloatwatch can be confident that they're working with the
> same numbers that the rest of us are discussing with the plain text on
> the lists.
Discussing the number with the plain text on the lists is
the ultimate goal. My thinking is that I'd like it to
be used kind of like 'powertop' (except not as a runtime tool
but as a development-time tool). The main intent is just
to give developers more information about how their changes
affect the kernel size.
Thanks for the feedback.
-- Tim
=============================
Tim Bird
Architecture Group Chair, CE Linux Forum
Senior Staff Engineer, Sony Corporation of America
=============================
next prev parent reply other threads:[~2008-10-08 19:32 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-10-07 21:19 RFC - size tool for kernel build system Tim Bird
2008-10-08 19:09 ` Chris Snook
2008-10-08 19:32 ` Tim Bird [this message]
2008-10-09 15:21 ` Adrian Bunk
2008-10-09 16:03 ` Jörn Engel
2008-10-09 18:34 ` Robin Getz
2008-10-09 23:56 ` Tim Bird
2008-10-10 9:42 ` Geert Uytterhoeven
2008-10-13 4:17 ` Robin Getz
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=48ED0ABB.2040607@am.sony.com \
--to=tim.bird@am.sony.com \
--cc=csnook@redhat.com \
--cc=linux-embedded@vger.kernel.org \
--cc=linux-kernel@vger.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).