lkml.org 
[lkml]   [2006]   [Jun]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: 2.6.17-rc5-mm2 link issues on s390
From
Date
On Thu, 2006-06-01 at 15:03 +0200, Cedric Le Goater wrote:
> here are small link issues on s390.
>
> ...
> CHK include/linux/compile.h
> UPD include/linux/compile.h
> CC init/version.o
> LD init/built-in.o
> LD .tmp_vmlinux1
> init/built-in.o(.init.text+0x564): In function `start_kernel':
> : undefined reference to `early_init_irq_lock_type'
> lib/built-in.o(.text+0xaf6): In function `__iowrite64_copy':
> : undefined reference to `__raw_writeq'
> make: *** [.tmp_vmlinux1] Error 1

I will try.

> I think the early_init_irq_lock_type() undef is related to the
> lock-validator patch. Shall I just :
>
> --- 2.6.17-rc5-mm2.orig/init/main.c
> +++ 2.6.17-rc5-mm2/init/main.c
> @@ -473,7 +473,9 @@
>
> local_irq_disable();
> early_boot_irqs_off();
> +#ifdef CONFIG_GENERIC_HARDIRQS
> early_init_irq_lock_type();
> +#endif
>
>
> too easy ... the lock-validator should be ported to s390 I guess. What are
> the steps to follow ?

Can't comment on the port issues yet but we plan to add the s390 support
to the lock-validator. It is just a cool debugging tool.

> As for the `__iowrite64_copy', shall we :
>
> #define writeq(b,addr) (*(volatile unsigned long *) __io_virt(addr) = (b))
> #define __raw_writeq writeq

Seems reasonable. __io_wirt() is a nop, so you are simply writing to the
specified address.

> For the moment, it boots but I'd like to make sure i'm in the right
> direction before sending patches. However, the following one is safe and
> fixes a very small compil issue on s390 in klibc.

Patches that fix compile problems are always fine with me. If it turns
out wrong for some obscure reason we can fix it up later.

--
blue skies,
Martin.

Martin Schwidefsky
Linux for zSeries Development & Services
IBM Deutschland Entwicklung GmbH

"Reality continues to ruin my life." - Calvin.


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2006-06-01 15:30    [W:0.210 / U:0.752 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site