Working frv toolchain?

Working frv toolchain?

[Jan Dittmer]

With gcc 4.0.0 and binutils 2.15.94 I get:

   CC      arch/frv/mm/dma-alloc.o
arch/frv/mm/dma-alloc.c: In function 'consistent_alloc':
arch/frv/mm/dma-alloc.c:66: error: impossible constraint in 'asm'
make[2]: *** [arch/frv/mm/dma-alloc.o] Error 1
make[1]: *** [arch/frv/mm] Error 2
make: *** [sub-make] Error 2

http://l4x.org/k/?d=35919 for details

What is a known working toolchain for fr-v? Is there a
mailing list for frv related problems?

Thanks,

Jan

Re: Working frv toolchain?

[David Howells]

Jan Dittmer <jdi@l4x.org> wrote:

> With gcc 4.0.0 and binutils 2.15.94 I get:

I'm using gcc 4.1.2.

David

Re: Working frv toolchain?

[Jan Dittmer]

David Howells wrote:
> Jan Dittmer <jdi@l4x.org> wrote:
> 
>> With gcc 4.0.0 and binutils 2.15.94 I get:
> 
> I'm using gcc 4.1.2.

4.1.2 together with 2.17.50 gives me with a i386 cross
compiler:

   CC      arch/frv/mm/dma-alloc.o
/usr/src/xtest/linux-2.6/arch/frv/mm/dma-alloc.c: In function 
'consistent_alloc':
/usr/src/xtest/linux-2.6/arch/frv/mm/dma-alloc.c:66: error: impossible 
constraint in 'asm'
make[2]: *** [arch/frv/mm/dma-alloc.o] Error 1
make[1]: *** [arch/frv/mm] Error 2
make: *** [sub-make] Error 2

Known bug or toolchain problem?

Jan

Re: Working frv toolchain?

[David Howells]

Jan Dittmer <jdi@l4x.org> wrote:

> Known bug or toolchain problem?

Ah... I'd forgotten about that.  I'm not sure all the ASM constraint changes
are upstream yet, and gcc bz 28583 also gets incurred.  Are you particularly
interested in building your own compiler, or would one of ours do?

David

Re: Working frv toolchain?

[David Howells]

David Howells <dhowells@redhat.com> wrote:

> Ah... I'd forgotten about that.  I'm not sure all the ASM constraint changes
> are upstream yet, and gcc bz 28583 also gets incurred.  Are you particularly
> interested in building your own compiler, or would one of ours do?

Look in:

	ftp://ftp.redhat.com/pub/redhat/gnupro/FRV

David

Re: Working frv toolchain?

[Jan Dittmer]

David Howells wrote:
> David Howells <dhowells@redhat.com> wrote:
> 
>> Ah... I'd forgotten about that.  I'm not sure all the ASM constraint changes
>> are upstream yet, and gcc bz 28583 also gets incurred.  Are you particularly
>> interested in building your own compiler, or would one of ours do?
> 
> Look in:
> 
> 	ftp://ftp.redhat.com/pub/redhat/gnupro/FRV

Hmm, I'm needing it for my pet project kernel compile tests at
http://l4x.org/k/. It would be nice to be as close as possible
to the upstream gcc. So patches against gcc upstream would be
welcome...
If you say that ain't happening, I'll just use the precompiled stuff.

Jan

Re: Working frv toolchain?

[Jan Dittmer]

David Howells wrote:
> David Howells <dhowells@redhat.com> wrote:
> 
>> Ah... I'd forgotten about that.  I'm not sure all the ASM constraint changes
>> are upstream yet, and gcc bz 28583 also gets incurred.  Are you particularly
>> interested in building your own compiler, or would one of ours do?
> 
> Look in:
> 
> 	ftp://ftp.redhat.com/pub/redhat/gnupro/FRV

Hrm, that gets me further, but one of the final stages fail:

   CC      init/version.o
   LD      init/built-in.o
   LD      .tmp_vmlinux1
kernel/built-in.o(.text+0x2e684): In function `kallsyms_lookup_name':
: relocation truncated to fit: R_FRV_GPREL12 kallsyms_num_syms
kernel/built-in.o(.text+0x2e6d4): In function `kallsyms_lookup_name':
: relocation truncated to fit: R_FRV_GPREL12 kallsyms_num_syms
kernel/built-in.o(.text+0x2e750): In function `get_symbol_pos':
: relocation truncated to fit: R_FRV_GPREL12 kallsyms_num_syms
kernel/built-in.o(.text+0x2ed00): In function `update_iter':
: relocation truncated to fit: R_FRV_GPREL12 kallsyms_num_syms
make[1]: *** [.tmp_vmlinux1] Error 1
make: *** [sub-make] Error 2

Todays git tree. Is there any known good release I can test
this toolchain against?

Thanks,

Jan

Re: Working frv toolchain?

[David Howells]

Jan Dittmer <jdi@l4x.org> wrote:

> Hrm, that gets me further, but one of the final stages fail:

What's your configuration?

It's possible that there's a bit more bringing up to date that I need to do.

David

Re: Working frv toolchain?

[Jan Dittmer]

David Howells wrote:
> Jan Dittmer <jdi@l4x.org> wrote:
> 
>> Hrm, that gets me further, but one of the final stages fail:
> 
> What's your configuration?

I just do:

make HOSTCC=gcc-4.0 ARCH=frv CROSS_COMPILE=frv-linux-gnu- O=... \
      defconfig
make HOSTCC=gcc-4.0 ARCH=frv CROSS_COMPILE=frv-linux-gnu- O=...

As I said it is for the automatic compile testing at l4x.org/k/

Thanks,

Jan

Re: Working frv toolchain?

[David Howells]

Jan Dittmer <jdi@l4x.org> wrote:

> Hrm, that gets me further, but one of the final stages fail:

Can you try the attached patch?

David
---
[PATCH] FRV: Fix the extern declaration of kallsyms_num_syms

From: David Howells <dhowells@redhat.com>

Fix the extern declaration of kallsyms_num_syms to indicate that the symbol
does not reside in the small-data storage space, and so may not be accessed
relative to the small data base register.

Signed-off-by: David Howells <dhowells@redhat.com>
---

 kernel/kallsyms.c |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/kernel/kallsyms.c b/kernel/kallsyms.c
index 474219a..fa84b83 100644
--- a/kernel/kallsyms.c
+++ b/kernel/kallsyms.c
@@ -32,7 +32,8 @@
 
 /* These will be re-linked against their real values during the second link stage */
 extern const unsigned long kallsyms_addresses[] __attribute__((weak));
-extern const unsigned long kallsyms_num_syms __attribute__((weak));
+extern const unsigned long kallsyms_num_syms
+__nongpreldata __attribute__((weak));
 extern const u8 kallsyms_names[] __attribute__((weak));
 
 extern const u8 kallsyms_token_table[] __attribute__((weak));

Re: Working frv toolchain?

[Jan Dittmer]

David Howells wrote:
> Jan Dittmer <jdi@l4x.org> wrote:
> 
>> Hrm, that gets me further, but one of the final stages fail:
> 
> Can you try the attached patch?

Thanks, that fixes the error in question. Now I have only
a couple of scary looking warnings (see below, sorry for
the word-wrap). So it's no toolchain problem then, after all?
Are there any chances to get a patch for frv support against
some upstream gcc 4.x version?

Thanks,

Jan


  LD      kernel/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_set_robust_list' changed 
from 8 in kernel/sys_ni.o to 32 in kernel/futex.o
frv-linux-gnu-ld: Warning: size of symbol `sys_get_robust_list' changed 
from 8 in kernel/sys_ni.o to 252 in kernel/futex.o
frv-linux-gnu-ld: Warning: size of symbol `sys_futex' changed from 8 in 
kernel/sys_ni.o to 476 in kernel/futex.o
frv-linux-gnu-ld: Warning: size of symbol `sys_chown16' changed from 8 
in kernel/sys_ni.o to 56 in kernel/uid16.o
frv-linux-gnu-ld: Warning: size of symbol `sys_lchown16' changed from 8 
in kernel/sys_ni.o to 56 in kernel/uid16.o
frv-linux-gnu-ld: Warning: size of symbol `sys_fchown16' changed from 8 
in kernel/sys_ni.o to 56 in kernel/uid16.o
frv-linux-gnu-ld: Warning: size of symbol `sys_setregid16' changed from 
8 in kernel/sys_ni.o to 56 in kernel/uid16.o
frv-linux-gnu-ld: Warning: size of symbol `sys_setgid16' changed from 8 
in kernel/sys_ni.o to 40 in kernel/uid16.o
frv-linux-gnu-ld: Warning: size of symbol `sys_setreuid16' changed from 
8 in kernel/sys_ni.o to 56 in kernel/uid16.o
frv-linux-gnu-ld: Warning: size of symbol `sys_setuid16' changed from 8 
in kernel/sys_ni.o to 40 in kernel/uid16.o
frv-linux-gnu-ld: Warning: size of symbol `sys_setresuid16' changed from 
8 in kernel/sys_ni.o to 156 in kernel/uid16.o
frv-linux-gnu-ld: Warning: size of symbol `sys_getresuid16' changed from 
8 in kernel/sys_ni.o to 312 in kernel/uid16.o
frv-linux-gnu-ld: Warning: size of symbol `sys_setresgid16' changed from 
8 in kernel/sys_ni.o to 156 in kernel/uid16.o
frv-linux-gnu-ld: Warning: size of symbol `sys_getresgid16' changed from 
8 in kernel/sys_ni.o to 312 in kernel/uid16.o
frv-linux-gnu-ld: Warning: size of symbol `sys_setfsuid16' changed from 
8 in kernel/sys_ni.o to 40 in kernel/uid16.o
frv-linux-gnu-ld: Warning: size of symbol `sys_setfsgid16' changed from 
8 in kernel/sys_ni.o to 40 in kernel/uid16.o
frv-linux-gnu-ld: Warning: size of symbol `sys_getgroups16' changed from 
8 in kernel/sys_ni.o to 368 in kernel/uid16.o
frv-linux-gnu-ld: Warning: size of symbol `sys_setgroups16' changed from 
8 in kernel/sys_ni.o to 452 in kernel/uid16.o
frv-linux-gnu-ld: Warning: size of symbol `sys_getuid16' changed from 8 
in kernel/sys_ni.o to 36 in kernel/uid16.o
frv-linux-gnu-ld: Warning: size of symbol `sys_geteuid16' changed from 8 
in kernel/sys_ni.o to 36 in kernel/uid16.o
frv-linux-gnu-ld: Warning: size of symbol `sys_getgid16' changed from 8 
in kernel/sys_ni.o to 36 in kernel/uid16.o
frv-linux-gnu-ld: Warning: size of symbol `sys_getegid16' changed from 8 
in kernel/sys_ni.o to 36 in kernel/uid16.o
   GEN     .version
   CHK     include/linux/compile.h
   UPD     include/linux/compile.h
   CC      init/version.o
   LD      init/built-in.o
   LD      .tmp_vmlinux1
   KSYM    .tmp_kallsyms1.S
   AS      .tmp_kallsyms1.o
   LD      .tmp_vmlinux2
   KSYM    .tmp_kallsyms2.S
   AS      .tmp_kallsyms2.o
   LD      vmlinux.o
frv-linux-gnu-ld: Warning: size of symbol `sys_munlockall' changed from 
8 in kernel/built-in.o to 72 in mm/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_swapoff' changed from 8 
in kernel/built-in.o to 2568 in mm/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_remap_file_pages' changed 
from 8 in kernel/built-in.o to 1008 in mm/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_munlock' changed from 8 
in kernel/built-in.o to 116 in mm/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_mincore' changed from 8 
in kernel/built-in.o to 1056 in mm/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_msync' changed from 8 in 
kernel/built-in.o to 448 in mm/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_swapon' changed from 8 in 
kernel/built-in.o to 2428 in mm/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_madvise' changed from 8 
in kernel/built-in.o to 1400 in mm/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_mlockall' changed from 8 
in kernel/built-in.o to 196 in mm/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_mprotect' changed from 8 
in kernel/built-in.o to 576 in mm/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_mlock' changed from 8 in 
kernel/built-in.o to 200 in mm/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_mremap' changed from 8 in 
kernel/built-in.o to 132 in mm/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_eventfd' changed from 8 
in kernel/built-in.o to 176 in fs/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_epoll_create' changed 
from 8 in kernel/built-in.o to 252 in fs/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_ioprio_set' changed from 
8 in kernel/built-in.o to 584 in fs/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_epoll_pwait' changed from 
8 in kernel/built-in.o to 356 in fs/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_epoll_ctl' changed from 8 
in kernel/built-in.o to 1424 in fs/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_epoll_wait' changed from 
8 in kernel/built-in.o to 1392 in fs/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_inotify_add_watch' 
changed from 8 in kernel/built-in.o to 496 in fs/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_signalfd' changed from 8 
in kernel/built-in.o to 476 in fs/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_inotify_init' changed 
from 8 in kernel/built-in.o to 524 in fs/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_bdflush' changed from 8 
in kernel/built-in.o to 124 in fs/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_inotify_rm_watch' changed 
from 8 in kernel/built-in.o to 132 in fs/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_ioprio_get' changed from 
8 in kernel/built-in.o to 512 in fs/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_msgget' changed from 8 in 
kernel/built-in.o to 132 in ipc/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_mq_getsetattr' changed 
from 8 in kernel/built-in.o to 532 in ipc/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_mq_unlink' changed from 8 
in kernel/built-in.o to 272 in ipc/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_shmdt' changed from 8 in 
kernel/built-in.o to 412 in ipc/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_msgrcv' changed from 8 in 
kernel/built-in.o to 148 in ipc/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_mq_open' changed from 8 
in kernel/built-in.o to 840 in ipc/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_semget' changed from 8 in 
kernel/built-in.o to 176 in ipc/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_mq_timedsend' changed 
from 8 in kernel/built-in.o to 752 in ipc/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_semctl' changed from 8 in 
kernel/built-in.o to 2704 in ipc/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_mq_notify' changed from 8 
in kernel/built-in.o to 964 in ipc/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_semtimedop' changed from 
8 in kernel/built-in.o to 1820 in ipc/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_shmctl' changed from 8 in 
kernel/built-in.o to 2332 in ipc/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_mq_timedreceive' changed 
from 8 in kernel/built-in.o to 612 in ipc/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_msgsnd' changed from 8 in 
kernel/built-in.o to 124 in ipc/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_semop' changed from 8 in 
kernel/built-in.o to 16 in ipc/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_shmat' changed from 8 in 
kernel/built-in.o to 56 in ipc/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_msgctl' changed from 8 in 
kernel/built-in.o to 1804 in ipc/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_shmget' changed from 8 in 
kernel/built-in.o to 144 in ipc/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_recv' changed from 8 in 
kernel/built-in.o to 20 in net/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_setsockopt' changed from 
8 in kernel/built-in.o to 204 in net/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_recvfrom' changed from 8 
in kernel/built-in.o to 304 in net/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_socket' changed from 8 in 
kernel/built-in.o to 88 in net/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_socketcall' changed from 
8 in kernel/built-in.o to 452 in net/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_listen' changed from 8 in 
kernel/built-in.o to 124 in net/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_recvmsg' changed from 8 
in kernel/built-in.o to 592 in net/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_bind' changed from 8 in 
kernel/built-in.o to 168 in net/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_send' changed from 8 in 
kernel/built-in.o to 20 in net/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_accept' changed from 8 in 
kernel/built-in.o to 404 in net/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_sendto' changed from 8 in 
kernel/built-in.o to 300 in net/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_getsockname' changed from 
8 in kernel/built-in.o to 176 in net/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_getsockopt' changed from 
8 in kernel/built-in.o to 188 in net/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_shutdown' changed from 8 
in kernel/built-in.o to 112 in net/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_sendmsg' changed from 8 
in kernel/built-in.o to 680 in net/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_connect' changed from 8 
in kernel/built-in.o to 176 in net/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_socketpair' changed from 
8 in kernel/built-in.o to 524 in net/built-in.o
frv-linux-gnu-ld: Warning: size of symbol `sys_getpeername' changed from 
8 in kernel/built-in.o to 176 in net/built-in.o
   MODPOST vmlinux.o
WARNING: vmlinux.o(.text+0x40): Section mismatch: reference to 
.init.text:free_initmem (between 'init_post' and 'try_name')
WARNING: vmlinux.o(.text+0x54fc): Section mismatch: reference to 
.init.text:start_kernel (between 'debug_stub' and 'debug_stub_exit')
WARNING: vmlinux.o(.text+0x5500): Section mismatch: reference to 
.init.text:start_kernel (between 'debug_stub' and 'debug_stub_exit')
WARNING: vmlinux.o(.text+0xf7b48): Section mismatch: reference to 
.init.text:pcibios_fixup_bus (between 'pci_scan_child_bus' and 
'pci_scan_bridge')
   LD      vmlinux
   SYSMAP  System.map
   SYSMAP  .tmp_System.map
frv-linux-gnu-objcopy -O binary -R .note -R .comment -S vmlinux 
arch/frv/boot/Image

Re: Working frv toolchain?

[David Howells]

Jan Dittmer <jdi@l4x.org> wrote:

> > Can you try the attached patch?
> 
> Thanks, that fixes the error in question.

Excellent.  I'll post it on upstream.

> Are there any chances to get a patch for frv support against
> some upstream gcc 4.x version?

Do you mean a patch to gcc?  Apparently gcc 4.3 should be fine, but I haven't
been able to build a toolchain that includes it.  I'm currently stuck on
trying to get through the glibc build phase.


> Now I have only a couple of scary looking warnings (see below, sorry for the
> word-wrap).
> ...
> frv-linux-gnu-ld: Warning: size of symbol `sys_set_robust_list' changed from 8
> in kernel/sys_ni.o to 32 in kernel/futex.o

Yeah.  Dunno what to do about that.  I suspect I need to do something about
cond_syscall(), but I'm not sure what.

> WARNING: vmlinux.o(.text+0x40): Section mismatch: reference to
> .init.text:free_initmem (between 'init_post' and 'try_name')

It's entirely possible that free_initmem() should not itself be marked __init.

> WARNING: vmlinux.o(.text+0x54fc): Section mismatch: reference to
> .init.text:start_kernel (between 'debug_stub' and 'debug_stub_exit')
> WARNING: vmlinux.o(.text+0x5500): Section mismatch: reference to
> .init.text:start_kernel (between 'debug_stub' and 'debug_stub_exit')

debug_stub() needs to arrange for start_kernel() to be invoked on its first
entry, but can't itself be __init because it gets invoked on every debugging
event (single step, h/w breakpoints, break instruction, etc).  However, its
first call is right at the end of head.S, so it isn't actually a problem.

Note that it isn't possible to jump directly into the debugger.  You have to
take an debug-level exception to enter CPU debug mode - which is a level higher
than kernel mode.

> WARNING: vmlinux.o(.text+0xf7b48): Section mismatch: reference to
> .init.text:pcibios_fixup_bus (between 'pci_scan_child_bus' and
> 'pci_scan_bridge')

Hmmm... pci_scan_child_bus() is not __*init, but it calls pcibios_fixup_bus()
which is.  That smells a trifle fishy.

David