[parisc-linux] spinlock 2.4 re-organise a la 2.6 [was: [RFC] rewrite kernel spinlock code to work better with gcc]

Grant Grundler grundler@parisc-linux.org
Sun, 30 Nov 2003 14:10:14 -0700


On Sun, Nov 30, 2003 at 04:31:49PM +0000, Joel Soete wrote:
> No, i think it is right but I forgot to embrace spinlock_t declaration 
> with "#ifdef CONFIG_SMP ...#endif" in system.h. Done in the new attched 
> patch (tested and run fine on c100 (32bit up) with kernel up and smp)
> My bad in previous test: forgot make distclean ; make mrproper (to be sure).

Use of "make distclean" means you don't have the dependencies correct.
ie whereever "ifdef CONFIG_*" is used, linux/config.h needs to be included.

> If you find some interest can you ci (I would like to test Randolph 
> patch on n4k just to be sure).

I don't. I have a basic problem with this patch.

...
> -#include <asm/spinlock_t.h>
...
> diff -Naur linux-2.4.23-rc5-pa17/include/asm-parisc/spinlock_t.h linux-2.4.23-rc5-pa17-bp/include/asm-parisc/spinlock_t.h
> --- linux-2.4.23-rc5-pa17/include/asm-parisc/spinlock_t.h	2003-11-29 16:13:04.000000000 +0100
> +++ linux-2.4.23-rc5-pa17-bp/include/asm-parisc/spinlock_t.h	1970-01-01 01:00:00.000000000 +0100
> @@ -1,97 +0,0 @@
> -#ifndef __PARISC_SPINLOCK_T_H
> -#define __PARISC_SPINLOCK_T_H
...

Why delete spinlock_t.h?

I added spinlock_t.h to resolve the circular inter-dependency between
asm/system.h, asm/bitops.h, and asm/spinlock.h.  spinlock_t.h depends
on nothing and defines spinlock primitives used by the others.

grant