Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [pull request] Pull request for branch for-2012.05/ccache-fixes
Date: Wed, 7 Mar 2012 20:40:03 +0100	[thread overview]
Message-ID: <20120307204003.3a02b08b@skate> (raw)
In-Reply-To: <cover.1331148403.git.thomas.petazzoni@free-electrons.com>

Hello,

Le Wed,  7 Mar 2012 20:26:48 +0100,
Thomas Petazzoni <thomas.petazzoni@free-electrons.com> a ?crit :

>       ccache: set COMPILERCHECK to 'none'

Some numbers about this.

Without the patch, a build of Qt with an empty cache takes the
following time:

real    8m17.607s
user    31m30.830s
sys     1m8.990s

Without the patch, a build of Qt with a cache full takes the following
time:

real    8m12.630s
user    31m17.340s
sys     1m8.520s

So basically, no improvement.

With the patch, a build of Qt with a cache full takes the following
time:

real    4m37.559s
user    5m13.390s
sys     0m31.840s

Here, there is clearly a nice impact of ccache.

I have decided to take a simpler approach than the one explored by
Arnout. Arnout approach was, IMO, too complicated, and only applicable
to external toolchains.

My approach is simpler, but indeed requires attention from the
developer on whether the cache should be invalidated or not. However, I
am not sure it is possible to provide a solution that is both safe (in
terms of cache contents validity) *and* efficient (in terms of cache
usage). Also, the approach taken by my patch is basically the same as
what was happening with previous versions of ccache, so this is not a
new behaviour.

Best regards,

Thomas
-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

      parent reply	other threads:[~2012-03-07 19:40 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-03-07 19:26 [Buildroot] [pull request] Pull request for branch for-2012.05/ccache-fixes Thomas Petazzoni
2012-03-07 19:26 ` [Buildroot] [PATCH 1/2] ccache: Force ccache to use its internal zlib Thomas Petazzoni
2012-03-08 21:57   ` Peter Korsgaard
2012-03-07 19:26 ` [Buildroot] [PATCH 2/2] ccache: set COMPILERCHECK to 'none' Thomas Petazzoni
2012-03-08 21:57   ` Peter Korsgaard
2012-03-10 11:39   ` Arnout Vandecappelle
2012-03-07 19:40 ` Thomas Petazzoni [this message]

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=20120307204003.3a02b08b@skate \
    --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