Re: [PATCH 1/1] bug: fix problem including <linux.bug.h> from linux/kernel.h

[kbuild test robot <lkp@intel.com>]

[-- Attachment #1: Type: text/plain, Size: 6499 bytes --]

Hi Ian,

[auto build test ERROR on linus/master]
[also build test ERROR on v4.12-rc2 next-20170524]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Ian-Abbott/bug-fix-problem-including-linux-bug-h-from-linux-kernel-h/20170525-081427
config: powerpc-allmodconfig (attached as .config)
compiler: powerpc64-linux-gnu-gcc (Debian 6.1.1-9) 6.1.1 20160705
reproduce:
        wget https://raw.githubusercontent.com/01org/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        make.cross ARCH=powerpc 

All error/warnings (new ones prefixed by >>):

   In file included from arch/powerpc/include/asm/bug.h:127:0,
                    from include/linux/bug.h:4,
                    from arch/powerpc/include/asm/mmu.h:125,
                    from arch/powerpc/include/asm/lppaca.h:36,
                    from arch/powerpc/include/asm/paca.h:21,
                    from arch/powerpc/include/asm/current.h:16,
                    from include/linux/sched.h:11,
                    from include/linux/utsname.h:5,
                    from drivers/char/random.c:238:
   include/asm-generic/bug.h:101:13: warning: 'struct pt_regs' declared inside parameter list will not be visible outside of this definition or declaration
         struct pt_regs *regs, struct warn_args *args);
                ^~~~~~~
   In file included from arch/powerpc/include/asm/lppaca.h:36:0,
                    from arch/powerpc/include/asm/paca.h:21,
                    from arch/powerpc/include/asm/current.h:16,
                    from include/linux/sched.h:11,
                    from include/linux/utsname.h:5,
                    from drivers/char/random.c:238:
   arch/powerpc/include/asm/mmu.h: In function 'mmu_has_feature':
>> arch/powerpc/include/asm/mmu.h:174:3: error: implicit declaration of function 'printk' [-Werror=implicit-function-declaration]
      printk("Warning! mmu_has_feature() used prior to jump label init!\n");
      ^~~~~~
>> arch/powerpc/include/asm/mmu.h:175:3: error: implicit declaration of function 'dump_stack' [-Werror=implicit-function-declaration]
      dump_stack();
      ^~~~~~~~~~
   In file included from include/linux/kernel.h:13:0,
                    from arch/powerpc/include/asm/page.h:15,
                    from arch/powerpc/include/asm/book3s/64/mmu-hash.h:16,
                    from arch/powerpc/include/asm/book3s/64/mmu.h:29,
                    from arch/powerpc/include/asm/mmu.h:304,
                    from arch/powerpc/include/asm/lppaca.h:36,
                    from arch/powerpc/include/asm/paca.h:21,
                    from arch/powerpc/include/asm/current.h:16,
                    from include/linux/sched.h:11,
                    from include/linux/utsname.h:5,
                    from drivers/char/random.c:238:
   include/linux/printk.h: At top level:
   include/linux/printk.h:172:5: error: conflicting types for 'printk'
    int printk(const char *fmt, ...);
        ^~~~~~
   include/linux/printk.h:172:1: note: a parameter list with an ellipsis can't match an empty parameter name list declaration
    int printk(const char *fmt, ...);
    ^~~
   In file included from arch/powerpc/include/asm/lppaca.h:36:0,
                    from arch/powerpc/include/asm/paca.h:21,
                    from arch/powerpc/include/asm/current.h:16,
                    from include/linux/sched.h:11,
                    from include/linux/utsname.h:5,
                    from drivers/char/random.c:238:
   arch/powerpc/include/asm/mmu.h:174:3: note: previous implicit declaration of 'printk' was here
      printk("Warning! mmu_has_feature() used prior to jump label init!\n");
      ^~~~~~
   In file included from include/linux/kernel.h:13:0,
                    from arch/powerpc/include/asm/page.h:15,
                    from arch/powerpc/include/asm/book3s/64/mmu-hash.h:16,
                    from arch/powerpc/include/asm/book3s/64/mmu.h:29,
                    from arch/powerpc/include/asm/mmu.h:304,
                    from arch/powerpc/include/asm/lppaca.h:36,
                    from arch/powerpc/include/asm/paca.h:21,
                    from arch/powerpc/include/asm/current.h:16,
                    from include/linux/sched.h:11,
                    from include/linux/utsname.h:5,
                    from drivers/char/random.c:238:
>> include/linux/printk.h:282:24: warning: conflicting types for 'dump_stack'
    extern asmlinkage void dump_stack(void) __cold;
                           ^~~~~~~~~~
   In file included from arch/powerpc/include/asm/lppaca.h:36:0,
                    from arch/powerpc/include/asm/paca.h:21,
                    from arch/powerpc/include/asm/current.h:16,
                    from include/linux/sched.h:11,
                    from include/linux/utsname.h:5,
                    from drivers/char/random.c:238:
   arch/powerpc/include/asm/mmu.h:175:3: note: previous implicit declaration of 'dump_stack' was here
      dump_stack();
      ^~~~~~~~~~
   cc1: some warnings being treated as errors

vim +/printk +174 arch/powerpc/include/asm/mmu.h

b5fa0f7f8 Michael Ellerman 2017-01-24  168  #ifndef __clang__ /* clang can't cope with this */
c12e6f24d Kevin Hao        2016-07-23  169  	BUILD_BUG_ON(!__builtin_constant_p(feature));
b5fa0f7f8 Michael Ellerman 2017-01-24  170  #endif
c12e6f24d Kevin Hao        2016-07-23  171  
c812c7d8f Aneesh Kumar K.V 2016-07-23  172  #ifdef CONFIG_JUMP_LABEL_FEATURE_CHECK_DEBUG
c812c7d8f Aneesh Kumar K.V 2016-07-23  173  	if (!static_key_initialized) {
c812c7d8f Aneesh Kumar K.V 2016-07-23 @174  		printk("Warning! mmu_has_feature() used prior to jump label init!\n");
c812c7d8f Aneesh Kumar K.V 2016-07-23 @175  		dump_stack();
c812c7d8f Aneesh Kumar K.V 2016-07-23  176  		return early_mmu_has_feature(feature);
c812c7d8f Aneesh Kumar K.V 2016-07-23  177  	}
c812c7d8f Aneesh Kumar K.V 2016-07-23  178  #endif

:::::: The code at line 174 was first introduced by commit
:::::: c812c7d8f1470ac9c8aa6d7e29b56e5845ee05fc powerpc/mm: Catch usage of cpu/mmu_has_feature() before jump label init

:::::: TO: Aneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com>
:::::: CC: Michael Ellerman <mpe@ellerman.id.au>

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 53757 bytes --]