This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [ Bug 16340][AARCH64][PATCH] Offset into GOT for TLS relocation computed inconsistently


On 24 January 2014 15:59, Will Newton <will.newton@linaro.org> wrote:
> On 14 January 2014 04:16, Alan Modra <amodra@gmail.com> wrote:
>> On Mon, Jan 13, 2014 at 04:36:41PM +0100, Christophe Lyon wrote:
>>> On 28 December 2013 02:26, Kugan <kugan.vivekanandarajah@linaro.org> wrote:
>>> > On 22/12/13 12:47, Alan Modra wrote:
>>> >> On Fri, Dec 20, 2013 at 07:32:15PM +1100, Kugan wrote:
>>> >>> [Apologies if you get this twice; Noticed a typo with email address]
>>> >>>
>>> >>> Hi,
>>> >>>
>>> >>> mongodb for aarch64 segfaults due to an error in applying GOT for TLS
>>> >>> relocation. Michael Hudson-Doyle tracked this down to wrong usage of
>>> >>> offset for the relocation. Attached patch (by Michael Hudson-Doyle)
>>> >>> fixes this. Regression tested on aarch64-none-linux-gnu with no new
>>> >>> regressions.
>>> >>>
>>> >>> Is this OK for trunk and backport for 2.24?
>>> >>
>>> >> Looks good to me.  There are two more occurrences of
>>> >> output_section->output_offset in elfnn-aarch64.c.  Please fix those
>>> >> too.
>>> >>
>>> >
>>> > Thanks Alan for the review. Please fond the patch attached which changes
>>> > it. make check for binutils, gas and ld has no new regression with the
>>> > patch.
>>> >
>>>
>>> Hi Alan,
>>>
>>> Is this version OK?
>>
>> Yes, it's good.  I meant my previous reply as an OK for the patch as
>> posted plus preapproval for the extra fixes.  Sorry if that wasn't clear.
>
> Is this patch ok for the 2.24 branch too?

Ping for 2.24?

Thanks,

-- 
Will Newton
Toolchain Working Group, Linaro


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]