Re: BKL: remove extraneous #include <smp_lock.h>

From: Randy Dunlap
Date: Wed Nov 17 2010 - 17:07:31 EST


On Wed, 17 Nov 2010 21:59:01 GMT Linux Kernel Mailing List wrote:

> Gitweb: http://git.kernel.org/linus/451a3c24b0135bce54542009b5fde43846c7cf67
> Commit: 451a3c24b0135bce54542009b5fde43846c7cf67
> Parent: 55f6561c6941713ab5ae9180525b026dd40b7d14
> Author: Arnd Bergmann <arnd@xxxxxxxx>
> AuthorDate: Wed Nov 17 16:26:55 2010 +0100
> Committer: Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx>
> CommitDate: Wed Nov 17 08:59:32 2010 -0800
>
> BKL: remove extraneous #include <smp_lock.h>
>
> The big kernel lock has been removed from all these files at some point,
> leaving only the #include.
>
> Remove this too as a cleanup.
>
> Signed-off-by: Arnd Bergmann <arnd@xxxxxxxx>
> Signed-off-by: Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx>
> ---



causes build errors in linux-next of NOV. 17.

see http://lkml.org/lkml/2010/11/17/339
and http://lkml.org/lkml/2010/11/17/365


smp_lock.h was removed from hardirq.h. smp_lock.h provided the function prototype
for kernel_locked(). Should source files now #include <linux/smp_lock.h> ?
even when not being built for SMP?


---
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/