[PATCH 0/3] Re-enable prelink for IA32 and MIPS

[PATCH 0/3] Re-enable prelink for IA32 and MIPS

[Mark Hatle]

*** BLURB HERE ***
The following changes since commit ecc8d346223030ee06aa6d83427f757982e948e4:

  sanity: Require bitbake 1.31.1 for multi-config changes (2016-08-18 10:07:18 +0100)

are available in the git repository at:

  git://git.openembedded.org/openembedded-core-contrib mhatle/prelink
  http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=mhatle/prelink

Mark Hatle (3):
  prelink: Move to latest version of prelink
  local.conf.sample: Disable ARM and PPC due to prelink test case
    failures
  Revert "local.conf.sample: Disable prelink by default"

 meta/conf/local.conf.sample                  | 12 ++++++++++--
 meta/recipes-devtools/prelink/prelink_git.bb |  2 +-
 2 files changed, 11 insertions(+), 3 deletions(-)

-- 
2.5.5

[PATCH 1/3] prelink: Move to latest version of prelink

[Mark Hatle]

* Uprev rtld emulation to glibc-2.23
* Fix compilation warnings
* Add additional debug scopes
* Change rtld build_local_scope to be breadth-first
* Fix LD_PRELOAD emulation
* Change function reordering to work with latest binutils

Signed-off-by: Mark Hatle <mark.hatle@windriver.com>
---
 meta/recipes-devtools/prelink/prelink_git.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-devtools/prelink/prelink_git.bb b/meta/recipes-devtools/prelink/prelink_git.bb
index e223ef6..5d9f260 100644
--- a/meta/recipes-devtools/prelink/prelink_git.bb
+++ b/meta/recipes-devtools/prelink/prelink_git.bb
@@ -8,7 +8,7 @@ and executables, so that far fewer relocations need to be resolved at \
 runtime and thus programs come up faster."
 LICENSE = "GPLv2"
 LIC_FILES_CHKSUM = "file://COPYING;md5=c93c0550bd3173f4504b2cbd8991e50b"
-SRCREV = "927979bbd115eeb8a75db3231906ef6aca4c4eb6"
+SRCREV = "ef20628dd78b92e1a3123afc67b64cf010bdd9e4"
 PV = "1.0+git${SRCPV}"
 
 #
-- 
2.5.5

[PATCH 2/3] local.conf.sample: Disable ARM and PPC due to prelink test case failures

[Mark Hatle]

Internal prelink test cases reloc8 and reloc9 are failing on both ARM
and PPC systems.  Disable them by removing the prelink from the
IMAGE_CLASSES setting.

Signed-off-by: Mark Hatle <mark.hatle@windriver.com>
---
 meta/conf/local.conf.sample | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/meta/conf/local.conf.sample b/meta/conf/local.conf.sample
index 2949ffb..8879a08 100644
--- a/meta/conf/local.conf.sample
+++ b/meta/conf/local.conf.sample
@@ -132,6 +132,15 @@ EXTRA_IMAGE_FEATURES ?= "debug-tweaks"
 # image-prelink disabled for now due to issues with IFUNC symbol relocation
 USER_CLASSES ?= "buildstats image-mklibs"
 
+# Prelinker tests are currently failing on ARM and PPC.  Prevent us from
+# running on those architectures until the failure is explained.
+USER_CLASSES_remove_arm = 'image-prelink'
+USER_CLASSES_remove_armeb = 'image-prelink'
+USER_CLASSES_remove_aarch64 = 'image-prelink'
+USER_CLASSES_remove_aarch64eb = 'image-prelink'
+USER_CLASSES_remove_powerpc = 'image-prelink'
+USER_CLASSES_remove_powerpc64 = 'image-prelink'
+
 
 #
 # Runtime testing of images
-- 
2.5.5

Re: [PATCH 2/3] local.conf.sample: Disable ARM and PPC due to prelink test case failures

[Mark Hatle]

Just an FYI - I've investigated the ARM and PPC failures further.  It looks like
it may be a binutils problem.

Both of the failed tests link applications using the arguments "-Wl,-z,nocopyreloc"

Once this has been done, the executable will not run.  (Aborts very early.)

I'm currently verifying if this did work in the past.

--Mark

On 8/18/16 12:04 PM, Mark Hatle wrote:
> Internal prelink test cases reloc8 and reloc9 are failing on both ARM
> and PPC systems.  Disable them by removing the prelink from the
> IMAGE_CLASSES setting.
> 
> Signed-off-by: Mark Hatle <mark.hatle@windriver.com>
> ---
>  meta/conf/local.conf.sample | 9 +++++++++
>  1 file changed, 9 insertions(+)
> 
> diff --git a/meta/conf/local.conf.sample b/meta/conf/local.conf.sample
> index 2949ffb..8879a08 100644
> --- a/meta/conf/local.conf.sample
> +++ b/meta/conf/local.conf.sample
> @@ -132,6 +132,15 @@ EXTRA_IMAGE_FEATURES ?= "debug-tweaks"
>  # image-prelink disabled for now due to issues with IFUNC symbol relocation
>  USER_CLASSES ?= "buildstats image-mklibs"
>  
> +# Prelinker tests are currently failing on ARM and PPC.  Prevent us from
> +# running on those architectures until the failure is explained.
> +USER_CLASSES_remove_arm = 'image-prelink'
> +USER_CLASSES_remove_armeb = 'image-prelink'
> +USER_CLASSES_remove_aarch64 = 'image-prelink'
> +USER_CLASSES_remove_aarch64eb = 'image-prelink'
> +USER_CLASSES_remove_powerpc = 'image-prelink'
> +USER_CLASSES_remove_powerpc64 = 'image-prelink'
> +
>  
>  #
>  # Runtime testing of images
> 

[PATCH 3/3] Revert "local.conf.sample: Disable prelink by default"

[Mark Hatle]

This reverts commit 300f858ba07c938427ccd05a3d7220027a03d461.

Reenable prelink

Signed-off-by: Mark Hatle <mark.hatle@windriver.com>
---
 meta/conf/local.conf.sample | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/meta/conf/local.conf.sample b/meta/conf/local.conf.sample
index 8879a08..b35db51 100644
--- a/meta/conf/local.conf.sample
+++ b/meta/conf/local.conf.sample
@@ -129,8 +129,7 @@ EXTRA_IMAGE_FEATURES ?= "debug-tweaks"
 #   - 'image-swab' to perform host system intrusion detection
 # NOTE: if listing mklibs & prelink both, then make sure mklibs is before prelink
 # NOTE: mklibs also needs to be explicitly enabled for a given image, see local.conf.extended
-# image-prelink disabled for now due to issues with IFUNC symbol relocation
-USER_CLASSES ?= "buildstats image-mklibs"
+USER_CLASSES ?= "buildstats image-mklibs image-prelink"
 
 # Prelinker tests are currently failing on ARM and PPC.  Prevent us from
 # running on those architectures until the failure is explained.
-- 
2.5.5