This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc 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: [ARM] Fix ld.so crash when built using Binutils 2.29


On 07/13/2017 07:41 AM, Phil Blundell wrote:
> On Wed, 2017-07-12 at 17:13 +0100, Jiong Wang wrote:
>>
> 
>> 2017-07-12  Jiong Wang  <jiong.wang@arm.com>
>>
>>          * sysdeps/arm/dl-machine.h (elf_machine_load_address):  Also
>> strip bit 0
>>          of pcrel_address under Thumb mode.
> 
> It seems a bit unfortunate that the semantics of the ADR instruction
> have changed in this way (after nearly two decades of the old
> behaviour) but having reviewed the GAS bug report again I do agree with
> Nick's rationale for doing that.  And this patch seems like a
> reasonable way of dealing with it in glibc.  So, assuming you have
> tested this with both old and new binutils, it is OK.

$0.02.

I'm already running with this patch in Fedora Rawhide for our Fedora 27
32-bit ARM builds and it fixes the issue with no regressions.

-- 
Cheers,
Carlos.


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