Hi Alex,
FYI, the error/warning still remains.
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 045169816b31b10faed984b01c390db1b32ee4c1
commit: ebb5e78cc63417a35254a791de66e1cc84f963cc MIPS: Initial implementation of a VDSO
date: 1 year, 1 month ago
config: mips-jmr3927_defconfig (attached as .config)
compiler: mips-linux-gnu-gcc (Debian 6.1.1-9) 6.1.1 20160705
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 ebb5e78cc63417a35254a791de66e1cc84f963cc
# save the attached .config to linux build tree
make.cross ARCH=mips
All errors (new ones prefixed by >>):
>> arch/mips/vdso/elf.S:1:0: error: '-march=r3900' requires '-mfp32'
/*
--
>> arch/mips/vdso/sigreturn.S:1:0: error: '-march=r3900' requires '-mfp32'
/*
vim +1 arch/mips/vdso/elf.S
> 1 /*
2 * Copyright (C) 2015 Imagination Technologies
3 * Author: Alex Smith <[email protected]>
4 *
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
On Sun, Dec 11, 2016 at 09:04:48AM +0800, kbuild test robot wrote:
> FYI, the error/warning still remains.
>
> tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> head: 045169816b31b10faed984b01c390db1b32ee4c1
> commit: ebb5e78cc63417a35254a791de66e1cc84f963cc MIPS: Initial implementation of a VDSO
> date: 1 year, 1 month ago
> config: mips-jmr3927_defconfig (attached as .config)
> compiler: mips-linux-gnu-gcc (Debian 6.1.1-9) 6.1.1 20160705
> 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 ebb5e78cc63417a35254a791de66e1cc84f963cc
> # save the attached .config to linux build tree
> make.cross ARCH=mips
Which binutils are you using with this compiler?
Thanks,
Ralf
On Tue, Dec 13, 2016 at 05:02:40AM +0100, Ralf Baechle wrote:
>On Sun, Dec 11, 2016 at 09:04:48AM +0800, kbuild test robot wrote:
>
>> FYI, the error/warning still remains.
>>
>> tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
>> head: 045169816b31b10faed984b01c390db1b32ee4c1
>> commit: ebb5e78cc63417a35254a791de66e1cc84f963cc MIPS: Initial implementation of a VDSO
>> date: 1 year, 1 month ago
>> config: mips-jmr3927_defconfig (attached as .config)
>> compiler: mips-linux-gnu-gcc (Debian 6.1.1-9) 6.1.1 20160705
>> 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 ebb5e78cc63417a35254a791de66e1cc84f963cc
>> # save the attached .config to linux build tree
>> make.cross ARCH=mips
>
>Which binutils are you using with this compiler?
It's binutils debian version 2.27-8. Does this jmr3927_defconfig
require a special mips compiler? I see there are 4 available gcc
packages in debian:
gcc-mips64el-linux-gnuabi64
gcc-mips64-linux-gnuabi64
gcc-mipsel-linux-gnu
gcc-mips-linux-gnu
The last one is used here.
Thanks,
Fengguang
On Tue, 13 Dec 2016, Ralf Baechle wrote:
> > FYI, the error/warning still remains.
> >
> > tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> > head: 045169816b31b10faed984b01c390db1b32ee4c1
> > commit: ebb5e78cc63417a35254a791de66e1cc84f963cc MIPS: Initial implementation of a VDSO
> > date: 1 year, 1 month ago
> > config: mips-jmr3927_defconfig (attached as .config)
> > compiler: mips-linux-gnu-gcc (Debian 6.1.1-9) 6.1.1 20160705
> > 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 ebb5e78cc63417a35254a791de66e1cc84f963cc
> > # save the attached .config to linux build tree
> > make.cross ARCH=mips
>
> Which binutils are you using with this compiler?
FYI, see the discussion: <https://patchwork.linux-mips.org/patch/14505/>,
and the original patch: <https://patchwork.linux-mips.org/patch/14465/>.
My advice has been to stick `$(call cc-option,-mno-float,-msoft-float)'
there instead, although there might be a slight controversy around it, so
if you are happy with revisiting Guenter's change in the future, then you
may well pick the original patch.
Myself, I don't have resources available right now to submit a revised
patch, sorry.
Maciej