[powerpc:next-test 40/58] WARNING: vmlinux.o(.text+0xb038): Section mismatch in reference from the function setup_kup() to the function .init.text:setup_kuep()

[powerpc:next-test 40/58] WARNING: vmlinux.o(.text+0xb038): Section mismatch in reference from the function setup_kup() to the function .init.text:setup_kuep()

[kbuild test robot]

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

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git next-test
head:   26ad26718dfaa7cf49d106d212ebf2370076c253
commit: 06fbe81b5909847aa13f9c86c2b6f9bbc5c2795b [40/58] powerpc/8xx: Add Kernel Userspace Execution Prevention
config: powerpc-mpc885_ads_defconfig (attached as .config)
compiler: powerpc-linux-gnu-gcc (Debian 7.2.0-11) 7.2.0
reproduce:
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        git checkout 06fbe81b5909847aa13f9c86c2b6f9bbc5c2795b
        # save the attached .config to linux build tree
        GCC_VERSION=7.2.0 make.cross ARCH=powerpc 

If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@intel.com>


All warnings (new ones prefixed by >>):

>> WARNING: vmlinux.o(.text+0xb038): Section mismatch in reference from the function setup_kup() to the function .init.text:setup_kuep()
   The function setup_kup() references
   the function __init setup_kuep().
   This is often because setup_kup lacks a __init
   annotation or the annotation of setup_kuep is wrong.

---
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: 10565 bytes --]

Re: [powerpc:next-test 40/58] WARNING: vmlinux.o(.text+0xb038): Section mismatch in reference from the function setup_kup() to the function .init.text:setup_kuep()

[Christophe Leroy]

Russel, Michael,

Looks like the reported problem comes from b28c97505eb1 ("powerpc/64: 
Setup KUP on secondary CPUs") 
(https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git/commit/?h=next&id=b28c97505eb1a5265e367c398c3406be6ce5e313)

Is it really necessary to remove the __init tag on setup_kup() ?

Christophe

Le 21/04/2019 à 18:23, kbuild test robot a écrit :
> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git next-test
> head:   26ad26718dfaa7cf49d106d212ebf2370076c253
> commit: 06fbe81b5909847aa13f9c86c2b6f9bbc5c2795b [40/58] powerpc/8xx: Add Kernel Userspace Execution Prevention
> config: powerpc-mpc885_ads_defconfig (attached as .config)
> compiler: powerpc-linux-gnu-gcc (Debian 7.2.0-11) 7.2.0
> reproduce:
>          wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
>          chmod +x ~/bin/make.cross
>          git checkout 06fbe81b5909847aa13f9c86c2b6f9bbc5c2795b
>          # save the attached .config to linux build tree
>          GCC_VERSION=7.2.0 make.cross ARCH=powerpc
> 
> If you fix the issue, kindly add following tag
> Reported-by: kbuild test robot <lkp@intel.com>
> 
> 
> All warnings (new ones prefixed by >>):
> 
>>> WARNING: vmlinux.o(.text+0xb038): Section mismatch in reference from the function setup_kup() to the function .init.text:setup_kuep()
>     The function setup_kup() references
>     the function __init setup_kuep().
>     This is often because setup_kup lacks a __init
>     annotation or the annotation of setup_kuep is wrong.
> 
> ---
> 0-DAY kernel test infrastructure                Open Source Technology Center
> https://lists.01.org/pipermail/kbuild-all                   Intel Corporation
> 

Re: [powerpc:next-test 40/58] WARNING: vmlinux.o(.text+0xb038): Section mismatch in reference from the function setup_kup() to the function .init.text:setup_kuep()

[Russell Currey]

On Tue, 2019-04-23 at 14:13 +0200, Christophe Leroy wrote:
> Russel, Michael,
> 
> Looks like the reported problem comes from b28c97505eb1
> ("powerpc/64: 
> Setup KUP on secondary CPUs") 
> (
> https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git/commit/?h=next&id=b28c97505eb1a5265e367c398c3406be6ce5e313
> )
> 
> Is it really necessary to remove the __init tag on setup_kup() ?

I don't think so, this was probably a mistake on my end, or something I
was testing and left in.  I'll put it back in and test to make sure it
doesn't break anything.

- Russell

> 
> Christophe
> 
> Le 21/04/2019 à 18:23, kbuild test robot a écrit :
> > tree:   
> > https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git
> > next-test
> > head:   26ad26718dfaa7cf49d106d212ebf2370076c253
> > commit: 06fbe81b5909847aa13f9c86c2b6f9bbc5c2795b [40/58]
> > powerpc/8xx: Add Kernel Userspace Execution Prevention
> > config: powerpc-mpc885_ads_defconfig (attached as .config)
> > compiler: powerpc-linux-gnu-gcc (Debian 7.2.0-11) 7.2.0
> > reproduce:
> >          wget 
> > https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross
> > -O ~/bin/make.cross
> >          chmod +x ~/bin/make.cross
> >          git checkout 06fbe81b5909847aa13f9c86c2b6f9bbc5c2795b
> >          # save the attached .config to linux build tree
> >          GCC_VERSION=7.2.0 make.cross ARCH=powerpc
> > 
> > If you fix the issue, kindly add following tag
> > Reported-by: kbuild test robot <lkp@intel.com>
> > 
> > 
> > All warnings (new ones prefixed by >>):
> > 
> > > > WARNING: vmlinux.o(.text+0xb038): Section mismatch in reference
> > > > from the function setup_kup() to the function
> > > > .init.text:setup_kuep()
> >     The function setup_kup() references
> >     the function __init setup_kuep().
> >     This is often because setup_kup lacks a __init
> >     annotation or the annotation of setup_kuep is wrong.
> > 
> > ---
> > 0-DAY kernel test infrastructure                Open Source
> > Technology Center
> > https://lists.01.org/pipermail/kbuild-all                   Intel
> > Corporation
> >