Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-16 Thread Peter Maydell
On 15 November 2010 15:09, Loïc Minier loic.min...@linaro.org wrote: On Mon, Nov 15, 2010, Albert ARIBAUD wrote: One last question:     qemu: fatal: Trying to execute code outside RAM or ROM at 0x3400 Does this mean that qemu does not simulate data or instruction aborts? Not that it is

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-15 Thread Albert ARIBAUD
Le 14/11/2010 22:22, Wolfgang Denk a écrit : Dear Albert ARIBAUD, In message4cdba515.3050...@free.fr you wrote: Alright. I'll prepare a V5 patch set with fixes to all ARM cpus. Wolfgang, is a single *patch* for all cpus ok or do you want a single *patchset* with one patch per cpu? Please

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-15 Thread Andreas Bießmann
Dear Albert ARIBAUD, Am 15.11.2010 12:01, schrieb Albert ARIBAUD: I therefore propose that V5 of my patch fixes arm926ejs, armv7, arm1136, arm1176 and pxa only, and leave the ten other ARM cpus untouched. Board maintainers who want to fix their board support for ELF relocs may have to

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-15 Thread Wolfgang Denk
Dear Albert ARIBAUD, In message 4ce11314.50...@free.fr you wrote: ATM on mainline master branch only five ARM cpus (arm926ejs, arm1136, arm1176, armv7 and pxa) have ELF relocation support. Others still use GOT relocation (or the older fixed-location scheme maybe), which IIUC means all

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-15 Thread Sebastien Carlier
Dear Andreas, On 2010-11-15 12:09:58, Andreas Bießmann wrote: arm920t/at91 is ongoing. I may need your help these days but first I need some time to get my last linker related issues fixed properly (segfault when linking empty weak symbol with -pie). Can you please try the attached patch?

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-15 Thread Albert ARIBAUD
Le 15/11/2010 12:13, Wolfgang Denk a écrit : Dear Albert ARIBAUD, In message4ce11314.50...@free.fr you wrote: ATM on mainline master branch only five ARM cpus (arm926ejs, arm1136, arm1176, armv7 and pxa) have ELF relocation support. Others still use GOT relocation (or the older

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-15 Thread Loïc Minier
On Mon, Nov 15, 2010, Albert ARIBAUD wrote: I can make the changes for other cpus, but I cannot perform even basic testing beyond making sure it still builds. Would qemu be of any help there? You could use e.g. versatile or integrator, and pretend that they use any CPU by changing their

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-15 Thread Reinhard Meyer
Dear Albert ARIBAUD, ATM on mainline master branch only five ARM cpus (arm926ejs, arm1136, arm1176, armv7 and pxa) have ELF relocation support. Others still use GOT relocation (or the older fixed-location scheme maybe), which IIUC means all boards using these other cpus are broken unless a

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-15 Thread Wolfgang Denk
Dear Albert ARIBAUD, In message 4ce11e66.5060...@free.fr you wrote: Do you think you could convert the remaining CPUs to ELF relocation as well? That would probably provide the best base for any further work. I can make the changes for other cpus, but I cannot perform even basic

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-15 Thread Wolfgang Denk
Dear =?iso-8859-1?Q?Lo=EFc?= Minier, In message 20101115115556.ga13...@bee.dooz.org you wrote: On Mon, Nov 15, 2010, Albert ARIBAUD wrote: I can make the changes for other cpus, but I cannot perform even basic = testing beyond making sure it still builds. Would qemu be of any help

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-15 Thread Albert ARIBAUD
Le 15/11/2010 13:02, Wolfgang Denk a écrit : Besides, while I can reasonably provide a V5 patch by this evening for the five cpus which already support ELF, I won't be able to provide fixes for the whole of arch/arm/cpu/* until thursday evening/friday morning. Is that ok? That's OK.

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-15 Thread Bas Mevissen
On Mon, 15 Nov 2010 12:49:58 +0100, Albert ARIBAUD albert.arib...@free.fr wrote: Do you think you could convert the remaining CPUs to ELF relocation as well? That would probably provide the best base for any further work. I can make the changes for other cpus, but I cannot perform even

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-15 Thread Loïc Minier
On Mon, Nov 15, 2010, Wolfgang Denk wrote: Do you have a working qemu setup and/or documentation how to run U-Boot on such an ARM config? Sometimes I do ;-) I use either: * Ubuntu QEMU packages - qemu-kvm-extras package has qemu-system-arm; it's usually fairly old because it's built

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-15 Thread Daniel Hobi
Hi Eric, On 10.11.2010 00:43, Eric Cooper wrote: I have been periodically rebasing my patches for the Seagate DockStar on master. But ever since the elf_reloc changes were merged, I have been unable to get a working build. First I saw symptoms similar to what Alexander Holler reported

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-15 Thread Albert ARIBAUD
Le 15/11/2010 15:06, Loïc Minier a écrit : So I'm afraid I couldn't find a fully working combination of U-Boot supported board + QEMU, but I didn't try QEMU tip; I'm Cc:ing Peter who cares for QEMU in Linaro, maybe he has more idea or is tempted to try a tip QEMU :-) Not sure I get

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-15 Thread Albert ARIBAUD
Le 15/11/2010 15:15, Daniel Hobi a écrit : @Albert: If your patch is applied without fixing the accesses to BSS before relocation, some ARM ports will be broken even when compiling with ELDK 4.2. Thanks for pointing this out. Indeed, my patch is actively based on the assumption that BSS is

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-15 Thread Loïc Minier
On Mon, Nov 15, 2010, Albert ARIBAUD wrote: I assume that the -kernel option indicates the executable to run. Correct, IOW u-boot.bin Would it take an ELF executable? a binary? You can pass an ELF as well How would one go about testing running

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-15 Thread Eric Cooper
On Mon, Nov 15, 2010 at 03:15:53PM +0100, Daniel Hobi wrote: Did you locally fix arch/arm/cpu/arm926ejs/kirkwood/timer.c to not access uninitialized data (variables timestamp and lastdec) before relocation? I didn't before, but now I have made that change (moved the static vars in timer.c to

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-15 Thread Andreas Bießmann
Dear Sebastien, Am 15.11.2010 um 12:43 schrieb Sebastien Carlier: Dear Andreas, On 2010-11-15 12:09:58, Andreas Bießmann wrote: arm920t/at91 is ongoing. I may need your help these days but first I need some time to get my last linker related issues fixed properly (segfault when linking

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-14 Thread Wolfgang Denk
Dear Albert ARIBAUD, In message 4cdba515.3050...@free.fr you wrote: Alright. I'll prepare a V5 patch set with fixes to all ARM cpus. Wolfgang, is a single *patch* for all cpus ok or do you want a single *patchset* with one patch per cpu? Please tell me the single patch approach is ok. :)

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-11 Thread Albert ARIBAUD
Le 10/11/2010 14:24, Daniel Hobi a écrit : On 10.11.2010 13:48, Albert ARIBAUD wrote: Le 10/11/2010 13:31, Daniel Hobi a écrit : But shouldn't this change be applied to all ARM linker scripts, ie arch/arm/cpu/*/u-boot.lds? Yes, it should. :) Can you please provide such a patch? I could,

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-10 Thread Daniel Hobi
Hi Albert, On 09.11.2010 19:47, Albert ARIBAUD wrote: Le 09/11/2010 19:24, Daniel Hobi a écrit : Thank you. This patch is required to get Kirkwood-based boards working again when using the CodeSourcery 2009q3 toolchain. (can't find the 2010q3 Lite toolchain on CodeSourcery's site, latest is

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-10 Thread Albert ARIBAUD
Le 10/11/2010 13:31, Daniel Hobi a écrit : Since all ARM boards are broken when using a recent toolchain, the patch should go in as fast as possible. But shouldn't this change be applied to all ARM linker scripts, ie arch/arm/cpu/*/u-boot.lds? Yes, it should. :) Can you please provide

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-10 Thread Daniel Hobi
On 10.11.2010 13:48, Albert ARIBAUD wrote: Le 10/11/2010 13:31, Daniel Hobi a écrit : But shouldn't this change be applied to all ARM linker scripts, ie arch/arm/cpu/*/u-boot.lds? Yes, it should. :) Can you please provide such a patch? I could, but I tend to provide patches only for

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-10 Thread Eric Cooper
On Wed, Nov 10, 2010 at 08:53:03AM +0100, Albert ARIBAUD wrote: Can you make the ELF images available? I'll do the readlf/objdump analysis and feed back on what I find. I've uploaded them here: http://www.cs.cmu.edu/~ecc/u-boot.{good,bad} Thanks. -- Eric Cooper e c c @ c m u

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-10 Thread Albert ARIBAUD
Le 10/11/2010 15:20, Eric Cooper a écrit : On Wed, Nov 10, 2010 at 08:53:03AM +0100, Albert ARIBAUD wrote: Can you make the ELF images available? I'll do the readlf/objdump analysis and feed back on what I find. I've uploaded them here: http://www.cs.cmu.edu/~ecc/u-boot.{good,bad}

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-09 Thread Albert ARIBAUD
Le 04/11/2010 23:27, Albert ARIBAUD a écrit : Le 04/11/2010 23:22, Albert Aribaud a écrit : older ld emitted all ELF relocations in input sections named ..rel.dyn, whereas newer ld uses names of the form .rel*. The linker script only collected .rel.dyn input sections. Rewrite to collect all

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-09 Thread Albert ARIBAUD
Le 09/11/2010 19:24, Daniel Hobi a écrit : Hi Albert, On 04.11.2010 23:22, Albert Aribaud wrote: older ld emitted all ELF relocations in input sections named .rel.dyn, whereas newer ld uses names of the form .rel*. The linker script only collected .rel.dyn input sections. Rewrite to collect

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-09 Thread Albert ARIBAUD
Le 09/11/2010 19:47, Albert ARIBAUD a écrit : I think this V4 of my patchset could be now committed to u-boot-arm/master, and if possible even on u-boot-arm for the december release of u-boot, as it is a bugfix. ... even on u-boot, not u-boot-arm, of course. Amicalement, -- Albert.

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-09 Thread Andreas Bießmann
Dear Daniel Hobi, Am 09.11.2010 um 19:24 schrieb Daniel Hobi: And on many ARM platforms (including Kirkwood), the timer implementation is still accessing BSS variables before relocation. Is someone working on this? Candidates are: $ git grep static ulong timestamp

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-09 Thread Andreas Bießmann
Dear Albert Ardibaud, Am 09.11.2010 um 19:47 schrieb Albert ARIBAUD: Thank you. This patch is required to get Kirkwood-based boards working again when using the CodeSourcery 2009q3 toolchain. (can't find the 2010q3 Lite toolchain on CodeSourcery's site, latest is 2010q1 apparently... Can

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-09 Thread Albert ARIBAUD
Le 09/11/2010 20:27, Andreas Bießmann a écrit : BTW couldn't the stuff in #ifdef CONFIG_AT91FAMILY in global_data generalized in some way to be used for all arm boards? Don't know for others, but regarding the timer, there is no need for timestamp before relocation, so it needs not join gd

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-09 Thread Eric Cooper
I have been periodically rebasing my patches for the Seagate DockStar on master. But ever since the elf_reloc changes were merged, I have been unable to get a working build. First I saw symptoms similar to what Alexander Holler reported (failing during NAND initialization due to incorrect BSS

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-09 Thread Albert ARIBAUD
Le 10/11/2010 00:43, Eric Cooper a écrit : I have been periodically rebasing my patches for the Seagate DockStar on master. But ever since the elf_reloc changes were merged, I have been unable to get a working build. First I saw symptoms similar to what Alexander Holler reported (failing

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-05 Thread Reinhard Meyer
Dear Albert Aribaud, older ld emitted all ELF relocations in input sections named .rel.dyn, whereas newer ld uses names of the form .rel*. The linker script only collected .rel.dyn input sections. Rewrite to collect all .rel* input sections and overlay with .bss. Signed-off-by: Albert

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-05 Thread Albert ARIBAUD
Le 05/11/2010 09:38, Reinhard Meyer a écrit : Dear Albert Aribaud, older ld emitted all ELF relocations in input sections named .rel.dyn, whereas newer ld uses names of the form .rel*. The linker script only collected .rel.dyn input sections. Rewrite to collect all .rel* input sections and

Re: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support

2010-11-04 Thread Albert ARIBAUD
Le 04/11/2010 23:22, Albert Aribaud a écrit : older ld emitted all ELF relocations in input sections named ..rel.dyn, whereas newer ld uses names of the form .rel*. The linker script only collected .rel.dyn input sections. Rewrite to collect all .rel* input sections and overlay with .bss.