Re: 2.6.17-rc5-mm2 link issues on s390

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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 [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux