linux-kernel - Re: arch/mips/vdso/elf.S:1:0: error: '-march=r3900' requires '-mfp32'

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Fri, 16 Dec 2016 00:57:14 +0000
From:   "Maciej W. Rozycki" <macro@...tec.com>
To:     Ralf Baechle <ralf@...ux-mips.org>
CC:     Matthew Fortune <Matthew.Fortune@...tec.com>,
        kbuild test robot <fengguang.wu@...el.com>,
        <kbuild-all@...org>, <linux-kernel@...r.kernel.org>,
        Markos Chandras <markos.chandras@...tec.com>
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