This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH][ARM] Fix ARM stub misalignment problem caused by cortex-a8 erratum stubs.
- From: =?big5?b?RG91ZyBLd2FuICjD9q62vHcp?= <dougkwan at google dot com>
- To: Nick Clifton <nickc at redhat dot com>
- Cc: binutils at sourceware dot org, Julian Brown <julian at codesourcery dot com>
- Date: Fri, 31 Jul 2009 02:27:05 -0700
- Subject: Re: [PATCH][ARM] Fix ARM stub misalignment problem caused by cortex-a8 erratum stubs.
- References: <498552560907301508x69d4c2e7s4cd1cd1bf1d94995@mail.gmail.com> <4A72AE84.8010605@redhat.com>
Thanks for the suggestion. I will change that.
-Doug
2009/7/31 Nick Clifton <nickc@redhat.com>:
> Hi Doug,
>
>> 2009-07-30 ?Doug Kwan ?<dougkwan@google.com>
>> bfd/ChangeLog
>>
>> ? ? ? ?* elf32-arm.c (arm_build_one_stub): Align section size increment by
>> ? ? ? ?8 byte.
>> ? ? ? ?(elf32_arm_size_stubs): Align cortex-a8 erratum stubs by 8 byte.
>> ? ? ? ?boundary.
>>
>> ld/testsuite/ChangeLog
>>
>> ? ? ? ?* ld-arm/arm-elf.exp: Add new test cortex-a8-fix-alignment
>> ? ? ? ?* ld-arm/cortex-a8-fix-alignment.d: New file
>> ? ? ? ?* ld-arm/cortex-a8-fix-alignment.s: New file
>> ? ? ? ?* ld-arm/cortex-a8-fix-alignment2.s: New file
>> ? ? ? ?* ld-arm/cortex-a8-fix-b-rel-thumb.d: Adjust test for new stub
>> ? ? ? ?alignment.
>> ? ? ? ?* ld-arm/cortex-a8-fix-b.d: Same.
>> ? ? ? ?* ld-arm/cortex-a8-fix-bcc-rel-thumb.d: Same.
>> ? ? ? ?* ld-arm/cortex-a8-fix-bcc.d: Same.
>> ? ? ? ?* ld-arm/cortex-a8-fix-bl-rel-arm.d: Same.
>> ? ? ? ?* ld-arm/cortex-a8-fix-bl-rel-thumb.d: Same.
>> ? ? ? ?* ld-arm/cortex-a8-fix-bl.d: Same.
>> ? ? ? ?* ld-arm/cortex-a8-fix-blx-rel-arm.d: Same.
>> ? ? ? ?* ld-arm/cortex-a8-fix-blx-rel-thumb.d: Same.
>> ? ? ? ?* ld-arm/cortex-a8-fix-blx.d: Same.
>> ? ? ? ?* ld-arm/farcall-group-size2.d: Same.
>> ? ? ? ?* ld-arm/farcall-group.d: Same.
>> ? ? ? ?* ld-arm/farcall-mix.d: Same.
>> ? ? ? ?* ld-arm/farcall-mix2.d: Same.
>> ? ? ? ?* ld-arm/farcall-mixed-lib.d: Same.
>
> Approved - please apply.
>
> Note - you might find that using the align_power() macro that is defined in
> bfd-in.h will make the code slightly simpler.
>
> Cheers
> ?Nick
>
>
>