All of lore.kernel.org
 help / color / mirror / Atom feed
From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: Patches and discussions about the oe-core layer
	<openembedded-core@lists.openembedded.org>
Subject: Re: BB_SIGNATURE_HANDLER = "basichash" unusable strict?
Date: Wed, 09 Nov 2011 12:06:23 +0000	[thread overview]
Message-ID: <1320840383.10843.173.camel@ted> (raw)
In-Reply-To: <20111109115126.GG3641@jama.jama.net>

On Wed, 2011-11-09 at 12:51 +0100, Martin Jansa wrote:
> I have talked with kergoth on IRC yesterday and he had very nice remark:
> 
> 16:40:50 < kergoth_> JaMa: heh, the biggest weakness of the sstate
> signature bits, in my opinion, is that it only tracks inputs, not
> outputs. If task A depends on B, and the metadata input to B changes,
> then A will be rebuilt, even if the *output* of B didn't change as a 
> result of the change to its metadata.
> 
> And with this idea applied on those 2 changes I think that PR change in
> libxml2 should of course invalidate checksum for 
> sstate-libxml2-native-x86_64-linux-2.7.8-r*populate-sysroot.tgz.siginfo
> and probably wont hurt so much when neon-native is also rebuilt, but then 
> if the output of neon build is the same with new sstate checksum as it was 
> with older one (I know it's hard to detect ie if some file in build has 
> "generation timestamp inside"), then we won't continue to rebuild
> subversion, gcc, ... all (just because neon was rebuilt due to libxml2 PR 
> change which didn't influence neon output).
> 
> The same with openssl PR change.. which can cause python-native rebuild,
> but as long as python-native build output is "the same" we don't need to
> rebuild everything which (even transitively) depends on python-native.

In an ideal world it would be nice to track the output. I've never seen
a proposal for how we could make this work in practise though. There are
at least two big problems that spring to mind:

a) How do you compare two sets of output and decide whether they're the
same? Same list of files? Same contents? How to deal with timestamps?

b) You can't know in advance that the output will or won't match and its
near impossible to calculate any kind of checksum without having the
output available to perform that calculation on. This breaks a lot of
the way bitbake runs the builds and makes it hard to compare two
configurations. Is A compatible with B? You'd have to build them both to
find out.

Whilst output tracking sounds nice, I think its trading one set of
problems for another and in the end, I'm not sure its the perfect
solution it might look like from our current position.

Cheers,

Richard




  reply	other threads:[~2011-11-09 12:12 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-11-08 14:37 BB_SIGNATURE_HANDLER = "basichash" unusable strict? Martin Jansa
2011-11-09 10:32 ` Richard Purdie
2011-11-09 11:51   ` Martin Jansa
2011-11-09 12:06     ` Richard Purdie [this message]
2011-11-09 12:45       ` Martin Jansa
2011-11-09 14:13         ` Richard Purdie
2011-11-09 14:48           ` Martin Jansa

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=1320840383.10843.173.camel@ted \
    --to=richard.purdie@linuxfoundation.org \
    --cc=openembedded-core@lists.openembedded.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.