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,