From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: Mark Hatle <mark.hatle@windriver.com>,
"Burton, Ross" <ross.burton@intel.com>
Cc: "openembedded-core@lists.openembedded.org"
<openembedded-core@lists.openembedded.org>
Subject: Re: Prelink problem
Date: Tue, 08 Jan 2019 21:46:37 +0000 [thread overview]
Message-ID: <fa30e51479e77bed2b7ca05049bed3c65f3efe2b.camel@linuxfoundation.org> (raw)
In-Reply-To: <7497c039-7951-ee51-7506-300a74d0b558@windriver.com>
On Tue, 2019-01-08 at 14:50 -0600, Mark Hatle wrote:
> On 1/8/19 2:37 PM, Burton, Ross wrote:
> > On Tue, 8 Jan 2019 at 20:15, Mark Hatle <mark.hatle@windriver.com>
> > wrote:
> > > > No idea why the opkg rootfs code is doing prelink operations
> > > > when RPM
> > > > or dpkg don't. CCing Mark who may have an idea here. I
> > > > thought the
> > > > autobuilder exerised multilib-on-opkg, but maybe not.
> > >
> > > They all should be doing prelink operations. The operation
> > > SHOULD be
> > > generically implemented as part of the image-prelink class, which
> > > is where I
> > > would have expected that copy to exist.
> > >
> > > If any of the package types of specifically doing something, that
> > > sounds
> > > broken... but the generic ones (last I looked) said to copy in
> > > the config file
> > > [if it didn't exist], run prelink, remove the file [if it wasn't
> > > there to start
> > > with].
> >
> > Note that it's part of the incremental code, so needs to be in the
> > rootfs code directly I suspect. Frankly I'd love to see incremental
> > images removed. It makes promises it can't keep (the moment a
> > rootfs
> > postprocess command is used, all bets are off) and massively
> > complicates things.
>
> We assume the post process command is what an 'admin' would do. So
> the various
> package managers should be able to deal with it in most
> cases. (Note, obviously
> it's more freeform, but I wouldn't expect everything to work in you
> removed a
> large part of the filesystem for instance.)
>
> As for prelink, I'm surprised this is in the incremental code. I'm
> not sure why
> it would be necessary unless the incremental work wants to UNPRELINK
> the rootfs
> before performing the upgrade?
>
> Prelink itself should still be run as a postprocess command that
> takes the
> output of the filesystem and reprocesses it.
>
> So something seems out of sync here.. (at a minimum probably should
> be better
> commented on why it's needed..)
The code is there for incremental opkg multilib image support. Its
trying to compare whether binaries are identical. To make it work, it
has to "unprelink" the files first before comparing.
I'm not convinced this is a good idea :/. I'm wondering if incremental
image generation makes sense at all in this context to be honest.
Cheers,
Richard
next prev parent reply other threads:[~2019-01-08 21:46 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-01-08 7:14 Prelink problem Andrej Valek
2019-01-08 12:27 ` Burton, Ross
2019-01-08 20:14 ` Mark Hatle
2019-01-08 20:37 ` Burton, Ross
2019-01-08 20:50 ` Mark Hatle
2019-01-08 21:46 ` Richard Purdie [this message]
2019-01-16 7:23 ` Andrej Valek
2019-01-16 15:57 ` Mark Hatle
2019-01-22 9:49 ` Andrej Valek
2019-02-05 8:54 ` [PATCH] lib/oe/rootfs: prelink only when image-prelink is inherited Andrej Valek
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=fa30e51479e77bed2b7ca05049bed3c65f3efe2b.camel@linuxfoundation.org \
--to=richard.purdie@linuxfoundation.org \
--cc=mark.hatle@windriver.com \
--cc=openembedded-core@lists.openembedded.org \
--cc=ross.burton@intel.com \
/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