Re: svn commit: r333461 - head/sys/amd64/amd64

2018-05-11 Thread Conrad Meyer
PR here, for the curious: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=228161 On Fri, May 11, 2018 at 11:00 AM, Conrad Meyer wrote: > On Fri, May 11, 2018 at 3:07 AM, Konstantin Belousov > wrote: >> On FreeBSD, gcc configuration requires explicit >> --enable-gnu-indirect-function >> option

Re: svn commit: r333461 - head/sys/amd64/amd64

2018-05-11 Thread Conrad Meyer
On Fri, May 11, 2018 at 3:07 AM, Konstantin Belousov wrote: > On FreeBSD, gcc configuration requires explicit --enable-gnu-indirect-function > option. I see it in e.g. lang/gcc7 port Makefile. > > On the other hand, I do not understand how devel/amd64-xtoolchain-gcc > and devel/powerpc64-xtoolcha

Re: svn commit: r333461 - head/sys/amd64/amd64

2018-05-11 Thread Konstantin Belousov
On Thu, May 10, 2018 at 10:17:06PM -0400, Ed Maste wrote: > On 10 May 2018 at 15:38, Konstantin Belousov wrote: > > > > Yes, I already noted and mjg noted that ifuncs are directed through PLT. > > I remember that it was not the case when I did it the first time, but then > > both compiler and link

Re: svn commit: r333461 - head/sys/amd64/amd64

2018-05-11 Thread Konstantin Belousov
On Thu, May 10, 2018 at 07:57:03PM -0700, Conrad Meyer wrote: > On Thu, May 10, 2018 at 8:01 AM, Konstantin Belousov wrote: > > Author: kib > > Date: Thu May 10 15:01:43 2018 > > New Revision: 333461 > > URL: https://svnweb.freebsd.org/changeset/base/333461 > > > > Log: > > Make fpusave() and fp

Re: svn commit: r333461 - head/sys/amd64/amd64

2018-05-10 Thread Conrad Meyer
On Thu, May 10, 2018 at 8:01 AM, Konstantin Belousov wrote: > Author: kib > Date: Thu May 10 15:01:43 2018 > New Revision: 333461 > URL: https://svnweb.freebsd.org/changeset/base/333461 > > Log: > Make fpusave() and fpurestore() on amd64 ifuncs. > > From now on, linking amd64 kernel requires e

Re: svn commit: r333461 - head/sys/amd64/amd64

2018-05-10 Thread Ed Maste
On 10 May 2018 at 15:38, Konstantin Belousov wrote: > > Yes, I already noted and mjg noted that ifuncs are directed through PLT. > I remember that it was not the case when I did it the first time, but then > both compiler and linker were different. I'm trying to find evidence of non-PLT ifuncs, b

Re: svn commit: r333461 - head/sys/amd64/amd64

2018-05-10 Thread Bruce Evans
On Thu, 10 May 2018, Konstantin Belousov wrote: On Fri, May 11, 2018 at 03:31:46AM +1000, Bruce Evans wrote: On Thu, 10 May 2018, Konstantin Belousov wrote: Log: Make fpusave() and fpurestore() on amd64 ifuncs. From now on, linking amd64 kernel requires either lld or newer ld.bfd. This b

Re: svn commit: r333461 - head/sys/amd64/amd64

2018-05-10 Thread Konstantin Belousov
On Fri, May 11, 2018 at 03:31:46AM +1000, Bruce Evans wrote: > On Thu, 10 May 2018, Konstantin Belousov wrote: > > > Log: > > Make fpusave() and fpurestore() on amd64 ifuncs. > > > > From now on, linking amd64 kernel requires either lld or newer ld.bfd. > > This breaks building with gcc: > > X

Re: svn commit: r333461 - head/sys/amd64/amd64

2018-05-10 Thread Bruce Evans
On Thu, 10 May 2018, Konstantin Belousov wrote: Log: Make fpusave() and fpurestore() on amd64 ifuncs. From now on, linking amd64 kernel requires either lld or newer ld.bfd. This breaks building with gcc: XX cc1: warnings being treated as errors XX ../../../amd64/amd64/fpu.c:195: warning: '