2016-12-11 01:05:20

by Fengguang Wu

[permalink] [raw]
Subject: arch/mips/vdso/elf.S:1:0: error: '-march=r3900' requires '-mfp32'

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


Attachments:
(No filename) (1.22 kB)
.config.gz (9.88 kB)
Download all attachments

2016-12-13 04:02:48

by Ralf Baechle

[permalink] [raw]
Subject: Re: arch/mips/vdso/elf.S:1:0: error: '-march=r3900' requires '-mfp32'

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

2016-12-14 04:00:37

by Fengguang Wu

[permalink] [raw]
Subject: Re: [kbuild-all] arch/mips/vdso/elf.S:1:0: error: '-march=r3900' requires '-mfp32'

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

2016-12-16 00:58:16

by Maciej W. Rozycki

[permalink] [raw]
Subject: Re: arch/mips/vdso/elf.S:1:0: error: '-march=r3900' requires '-mfp32'

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