From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH v4 2015.08 1/4] pkg-generic: add step_pkg_size global instrumentation hook
Date: Sat, 11 Jul 2015 13:09:14 +0200 [thread overview]
Message-ID: <20150711130914.40d5622e@free-electrons.com> (raw)
In-Reply-To: <55A0F41E.10900@openwide.fr>
Dear Romain Naour,
On Sat, 11 Jul 2015 12:46:54 +0200, Romain Naour wrote:
> > This commit also adds a Config.in option to enable the collection of
> > this data, as calculating the md5 of all installed files at the
> > beginning and end of the installation of each package can be
> > considered a time-consuming process which maybe some users will not be
> > willing to suffer from.
>
> The part for Config.in is not in this patch, I seems that you dropped it wile
> reworking this series. So the size-stat option can't be disabled actually.
It's actually the commit log that is incorrect: removing the option is
intentional. From the cover letter:
- Remove the BR2_COLLECT_FILE_SIZE_STATS option, and collect the
necessary data unconditionally. This was suggested by Yann
E. Morin, who did some testing of the performance impact, and found
that it was so insignificant that it is negligible. See Yann's
analysis at
http://lists.busybox.net/pipermail/buildroot/2015-February/119431.html.
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
next prev parent reply other threads:[~2015-07-11 11:09 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-05-25 21:56 [Buildroot] [PATCH v4 2015.08 0/4] Generate package size statistics Thomas Petazzoni
2015-05-25 21:56 ` [Buildroot] [PATCH v4 2015.08 1/4] pkg-generic: add step_pkg_size global instrumentation hook Thomas Petazzoni
2015-05-28 3:29 ` Ryan Barnett
2015-07-11 10:46 ` Romain Naour
2015-07-11 11:09 ` Thomas Petazzoni [this message]
2015-05-25 21:56 ` [Buildroot] [PATCH v4 2015.08 2/4] support/scripts: add size-stats script Thomas Petazzoni
2015-05-28 3:18 ` Ryan Barnett
2015-05-28 14:55 ` Matthew Weber
2015-09-02 21:08 ` Thomas Petazzoni
2015-06-03 15:50 ` Clayton Shotwell
2015-07-11 11:46 ` Romain Naour
2015-05-25 21:56 ` [Buildroot] [PATCH v4 2015.08 3/4] Makefile: implement a size-stats target Thomas Petazzoni
2015-05-28 3:32 ` Ryan Barnett
2015-08-20 12:42 ` Ryan Barnett
2015-08-20 20:43 ` Thomas Petazzoni
2015-08-21 14:07 ` Ryan Barnett
2015-08-23 14:46 ` Thomas Petazzoni
2015-05-25 21:56 ` [Buildroot] [PATCH v4 2015.08 4/4] docs/manual: add section about size graphing Thomas Petazzoni
2015-05-28 3:42 ` Ryan Barnett
2015-07-30 21:54 ` [Buildroot] [PATCH v4 2015.08 0/4] Generate package size statistics Ryan Barnett
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=20150711130914.40d5622e@free-electrons.com \
--to=thomas.petazzoni@free-electrons.com \
--cc=buildroot@busybox.net \
/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