[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] ppc-linux-user: Fix missing symbols in .rel/.re
From: |
Alexander Graf |
Subject: |
Re: [Qemu-devel] [PATCH] ppc-linux-user: Fix missing symbols in .rel/.rela.plt sections |
Date: |
Tue, 10 Jan 2012 00:11:33 +0100 |
On 10.01.2012, at 00:06, Aurelien Jarno wrote:
> On Mon, Jan 09, 2012 at 02:31:15PM +0100, Alexander Graf wrote:
>>
>> On 07.01.2012, at 21:16, Aurelien Jarno wrote:
>>
>>> Fix .rel.plt sections in the output to not only include .rel.plt
>>> sections from the input but also the .rel.iplt sections and to define
>>> the hidden symbols __rel_iplt_start and __rel_iplt_end around
>>> .rel.iplt as otherwise we get undefined references to these when
>>> linking statically to a multiarch enabled libc (using STT_GNU_IFUNC).
>>
>> Please see:
>>
>> http://patchwork.ozlabs.org/patch/130932/
>>
>> I'd still say that we should fix the linker scripts regardless. Also for 1.0.
>
> I agree that we should either remove them or fix them. Not fixing them
> because they should not be there, but at the same time not removing
> them is a bad idea.
>
> On my side I have also tested that it can work with PROVIDE instead of
> PROVIDE_HIDDEN, so we end-up with the same patches. Now how should we
> proceed to merge that to master, which is the first step before getting
> that into stable?
Either apply yours and rebase mine on top, so ppc64 is fixed too, or apply mine
:). I'm rather indifferent on that one.
Alex