Re: [mmotm:master 506/522] frv-linux-ld: Warning: size of symbol `sys_membarrier' changed from 8 in kernel/sys_ni.o to 52 in kernel/membarrier.o

[Andrew Morton <akpm@linux-foundation.org>]

On Tue, 18 Aug 2015 09:43:48 +0800 kbuild test robot <fengguang.wu@intel.com> wrote:

> tree:   git://git.cmpxchg.org/linux-mmotm.git master
> head:   ef4ce68013dd78948da68460aab14b59563998d0
> commit: a91c5d8940ac0be6c80a796ddf6a2ddae5242c58 [506/522] sgi-xp: replace cpu_to_node() with cpu_to_mem() to support memoryless node
> config: frv-defconfig (attached as .config)
> reproduce:
>   wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
>   chmod +x ~/bin/make.cross
>   git checkout a91c5d8940ac0be6c80a796ddf6a2ddae5242c58
>   # save the attached .config to linux build tree
>   make.cross ARCH=frv 
> 
> All warnings (new ones prefixed by >>):
> 
>    frv-linux-ld: Warning: size of symbol `sys_setuid' changed from 272 in kernel/sys.o to 8 in kernel/sys_ni.o
>    frv-linux-ld: Warning: size of symbol `sys_setregid' changed from 328 in kernel/sys.o to 8 in kernel/sys_ni.o
>    frv-linux-ld: Warning: size of symbol `sys_setgid' changed from 212 in kernel/sys.o to 8 in kernel/sys_ni.o
> ...

Something appears to be screwed up in the FRV toolchain's handling of
weak symbols.

Possibly this is due to our implementation of cond_syscall().  I assume
this doesn't happen with C symbols which are declared __weak, so
perhaps someone who has an FRV compiler can look at the compiler's
assembly output for a __weak C symbol, compare that with the
cond_syscall() implementation and see if the difference suggests a fix.

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>