[U-Boot] [PATCH V3 1/3] arm926ejs: fix linker file for newer ld support

[Albert ARIBAUD]

Le 04/11/2010 13:47, Alexander Holler a écrit :
> Am 04.11.2010 13:40, schrieb Albert ARIBAUD:
>> Le 04/11/2010 11:46, Alexander Holler a écrit :
>>> So the suggested change from Steve Sakoman (reordered fix from Albert
>>> Aribaud) still seems to be the one to prefer.
>>
>> Something is wrong here:
>>
>>> - .rel.dyn : { *(.rel.dyn) }
>>> + .rel.dyn : { *(.rel.*) }
>>
>> This change is the same as the one I submitted, replacing any .rel.dyn
>> input sections references with .rel* input sections (this includes
>> .rel.* sections), so I fail to see the difference.
>
> The difference is the place. Your patch v1 puts that after the end which
> results in a larger binary.

V3 doesn't result in a larger binary any more, at least as far as my own 
tests have shown, and that is why I asked people to test V3.

> This one just leaves the place as before and
> only changes what is included. In regard to your v3 I don't know what is
> the difference, I haven't looked at what it (v3) really does because I
> don't know much about the linker sections and (currently) don't want to
> drive deeper into that.

Then please do test this V3 patch and verify if the issues you raise 
still apply or are fixed.

>> Can you please perform a build and wrap the u-boot, u-boot.lds,
>> u-boot.map files and build standard and error outputs in a tarball and
>> send that to me?
>
> Will do so.

Please make sure to indicate which source commit you're testing on and 
which toolchain you're using, and to compare with V3 results.

> Regards,
>
> Alexander

Amicalement,
-- 
Albert.