[-queue] arch/mips/kernel/linux32.c:304: error: implicit declaration of function 'lock_kernel'

[-queue] arch/mips/kernel/linux32.c:304: error: implicit declaration of function 'lock_kernel'

[Wu Zhangjin]

Hi, Ralf

When I compile the queue branch of your -queue git repo, meet this
error:

arch/mips/kernel/linux32.c: In function 'SYSC_32_sysctl':
arch/mips/kernel/linux32.c:304: error: implicit declaration of function
'lock_kernel'
arch/mips/kernel/linux32.c:307: error: implicit declaration of function
'unlock_kernel'

and found you have removed this <linux/smp_lock.h> header file in
linux32.c in this commit: 

commit 0adcf22332d4da33629568fd14e00069cbf002e6
Author: Ralf Baechle <ralf@linux-mips.org>
Date:   Tue Nov 3 23:06:18 2009 +0100

    MIPS: Don't include <linux/smp_lock.h> unnecessarily.
    
    Signed-off-by: Ralf Baechle <ralf@linux-mips.org>

:100644 100644 938b1d0... 3676660... M
arch/mips/basler/excite/excite_iodev.c
:100644 100644 b77fefa... 07cc39e... M  arch/mips/kernel/linux32.c
:100644 100644 364f066... dcaed1b... M  arch/mips/kernel/rtlx.c
:100644 100644 6047752... 2bd2151... M  arch/mips/kernel/vpe.c
:100644 100644 15ea778... ed2453e... M
arch/mips/sibyte/common/sb_tbprof.c

So, revert it for linux32.c?

Regards,
	Wu Zhangjin

Re: [-queue] arch/mips/kernel/linux32.c:304: error: implicit declaration of function 'lock_kernel'

[Ralf Baechle]

On Wed, Nov 04, 2009 at 01:59:09PM +0800, Wu Zhangjin wrote:

> When I compile the queue branch of your -queue git repo, meet this
> error:
> 
> arch/mips/kernel/linux32.c: In function 'SYSC_32_sysctl':
> arch/mips/kernel/linux32.c:304: error: implicit declaration of function
> 'lock_kernel'
> arch/mips/kernel/linux32.c:307: error: implicit declaration of function
> 'unlock_kernel'
> 
> and found you have removed this <linux/smp_lock.h> header file in
> linux32.c in this commit: 
> 
> commit 0adcf22332d4da33629568fd14e00069cbf002e6
> Author: Ralf Baechle <ralf@linux-mips.org>
> Date:   Tue Nov 3 23:06:18 2009 +0100
> 
>     MIPS: Don't include <linux/smp_lock.h> unnecessarily.
>     
>     Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
> 
> :100644 100644 938b1d0... 3676660... M
> arch/mips/basler/excite/excite_iodev.c
> :100644 100644 b77fefa... 07cc39e... M  arch/mips/kernel/linux32.c
> :100644 100644 364f066... dcaed1b... M  arch/mips/kernel/rtlx.c
> :100644 100644 6047752... 2bd2151... M  arch/mips/kernel/vpe.c
> :100644 100644 15ea778... ed2453e... M
> arch/mips/sibyte/common/sb_tbprof.c

There is another patch from Thomas Gleixner pending which will push down
the lock_kernel calls deeper into the sysctl calls.  I also had that
patch applied so missed that build failure.

> So, revert it for linux32.c?

Yes, that's what I'll do for now.

Thanks!

  Ralf