Re: linux-next: build failure after merge of the rcu tree

Paul E. McKenney
2017-05-29T21:15:07Z

On Mon, May 29, 2017 at 04:02:09PM +1000, Stephen Rothwell wrote:
> Hi Paul,
>
> After merging the rcu tree, today's linux-next build (bfin
> BF526-EZBRD_defconfig and several other bfin configs) failed like this:
>
> In file included from include/linux/srcu.h:60:0,
> from include/linux/notifier.h:15,
> from include/linux/memory_hotplug.h:6,
> from include/linux/mmzone.h:777,
> from include/linux/gfp.h:5,
> from include/linux/kmod.h:22,
> from include/linux/module.h:13,
> from include/linux/moduleloader.h:5,
> from arch/blackfin/kernel/module.c:9:
> include/linux/srcutiny.h: In function 'srcu_torture_stats_print':
> include/linux/srcutiny.h:96:2: error: 'mod' undeclared (first use in this function)
>
> Caused by commit
>
> 54ffb22bd841 ("rcutorture: Move SRCU status printing to SRCU implementations")

And of course this is nothing but a printk().

> arch/blackfin/kernel/module.c includes this:
>
> #define pr_fmt(fmt) "module %s: " fmt, mod->name

Hmmm... There are well over 1000 definitions of pr_fmt() in the kernel.
Very popular name there!

> added by commit
>
> b75a9e6bfba5 ("Blackfin: use more standard pr_fmt in the module loader")
>
> before v2.6.39-rc2 ...

I guess I cannot claim to be blindsided by a recent change, at least not
unless you take my age into account. ;-)

So maybe there is some header file that blackfin wants included? There
are only about 100 definitions of "mod", so let's see... Well, none of
them are in blackfin, but perhaps the two in module.h? But these are just
fields in structs. As is the one in moduleparam.h.

At this point, the only option I see is to convert srcu_torture_stats_print()
from a static inline function to a macro.

Anyone see any other options?

Thanx, Paul