Re: [PATCH 2.6.34-rc2] sh: Fix FDPIC binary loader

[Thomas Schwinge <thomas@codesourcery.com>]

[-- Attachment #1.1: Type: text/plain, Size: 1071 bytes --]

Hi!

On Mon, 29 Mar 2010 12:15:23 +0900, Paul Mundt <lethal@linux-sh.org> wrote:
> On Fri, Mar 26, 2010 at 09:55:38AM +0000, Andrew Stubbs wrote:
> > Ensure that the aux table is properly initialized, even when optional 
> > features are missing. Without this, the FDPIC loader did not work.

Andrew's patch became commit d5ab780305bb6d60a7b5a74f18cf84eb6ad153b1.

> This was whitespace damaged and didn't apply, so I did it by hand.

Unfortunately, in Paul's manual application, one part of the patch was
lost, and that happened to be the functional part.  I'm attaching a patch
to address that.  The patch is for 3.5.2, but likewise applies to the
current master branch.


If you add that patch to, for example, »SH7269 Linux BSP«,
<https://oss.renesas.com/modules/download/index.php?cid=80&lid=231>, you
can re-enable dynamic linking, and can remove the following note from its
readme.txt:

| 1. The toolchain used can only be used with static builds. Do not use this with
|    dynamically loaded shared libraries.


Grüße,
 Thomas



[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1.2: 0001-sh-Fix-FDPIC-binary-loader.patch --]
[-- Type: text/x-diff, Size: 1119 bytes --]

From 23dc6096a678259662224de60470fababed35d2d Mon Sep 17 00:00:00 2001
From: Thomas Schwinge <thomas@codesourcery.com>
Date: Fri, 16 Nov 2012 10:46:20 +0100
Subject: [PATCH] sh: Fix FDPIC binary loader

Ensure that the aux table is properly initialized, even when optional features
are missing.  Without this, the FDPIC loader did not work.  This was meant to
be included in commit d5ab780305bb6d60a7b5a74f18cf84eb6ad153b1.

Signed-off-by: Thomas Schwinge <thomas@codesourcery.com>
---
 arch/sh/include/asm/elf.h |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/sh/include/asm/elf.h b/arch/sh/include/asm/elf.h
index f38112b..978b7fd 100644
--- a/arch/sh/include/asm/elf.h
+++ b/arch/sh/include/asm/elf.h
@@ -202,9 +202,9 @@ extern void __kernel_vsyscall;
 	if (vdso_enabled)					\
 		NEW_AUX_ENT(AT_SYSINFO_EHDR, VDSO_BASE);	\
 	else							\
-		NEW_AUX_ENT(AT_IGNORE, 0);
+		NEW_AUX_ENT(AT_IGNORE, 0)
 #else
-#define VSYSCALL_AUX_ENT
+#define VSYSCALL_AUX_ENT	NEW_AUX_ENT(AT_IGNORE, 0)
 #endif /* CONFIG_VSYSCALL */
 
 #ifdef CONFIG_SH_FPU
-- 
1.7.4.1


[-- Attachment #2: Type: application/pgp-signature, Size: 489 bytes --]