Include SSE state in i386 fenv_t (bug 16064).
This patch fixes bug 16064, i386 fenv_t not including SSE state, using
the technique suggested there of storing the state in the existing
__eip field of fenv_t to avoid needing to increase the size of fenv_t
and add new symbol versions. The included testcase, which previously
failed for i386 (but passed for x86_64), illustrates how the previous
state was buggy.
This patch causes the SSE state to be included *to the extent it is on
x86_64*. Where some state should logically be included but isn't for
x86_64 (see bug 16068), this patch does not cause it to be included
for i386 either. The idea is that any patch fixing that bug should
fix it for both x86_64 and i386 at once.
Tested i386 and x86_64. (I haven't tested the case of a CPU without
SSE2 disabling the test.)
[BZ #16064]
* sysdeps/i386/fpu/fegetenv.c: Include <unistd.h>, <ldsodefs.h>
and <dl-procinfo.h>.
(__fegetenv): Save SSE state in envp->__eip if supported.
* sysdeps/i386/fpu/feholdexcpt.c (feholdexcept): Save SSE state in
envp->__eip if supported.
* sysdeps/i386/fpu/fesetenv.c: Include <unistd.h>, <ldsodefs.h>
and <dl-procinfo.h>.
(__fesetenv): Always set __eip, __cs_selector, __opcode,
__data_offset and __data_selector in environment to 0. Set SSE
state if supported.
* sysdeps/x86/fpu/Makefile [$(subdir) = math] (tests): Add
test-fenv-sse.
[$(subdir) = math] (CFLAGS-test-fenv-sse.c): Add -msse2
-mfpmath=sse.
* sysdeps/x86/fpu/test-fenv-sse.c: New file.
2014-05-09 16:59:56 +00:00
|
|
|
/* Test floating-point environment includes SSE state (bug 16064).
|
2020-01-01 00:14:33 +00:00
|
|
|
Copyright (C) 2014-2020 Free Software Foundation, Inc.
|
Include SSE state in i386 fenv_t (bug 16064).
This patch fixes bug 16064, i386 fenv_t not including SSE state, using
the technique suggested there of storing the state in the existing
__eip field of fenv_t to avoid needing to increase the size of fenv_t
and add new symbol versions. The included testcase, which previously
failed for i386 (but passed for x86_64), illustrates how the previous
state was buggy.
This patch causes the SSE state to be included *to the extent it is on
x86_64*. Where some state should logically be included but isn't for
x86_64 (see bug 16068), this patch does not cause it to be included
for i386 either. The idea is that any patch fixing that bug should
fix it for both x86_64 and i386 at once.
Tested i386 and x86_64. (I haven't tested the case of a CPU without
SSE2 disabling the test.)
[BZ #16064]
* sysdeps/i386/fpu/fegetenv.c: Include <unistd.h>, <ldsodefs.h>
and <dl-procinfo.h>.
(__fegetenv): Save SSE state in envp->__eip if supported.
* sysdeps/i386/fpu/feholdexcpt.c (feholdexcept): Save SSE state in
envp->__eip if supported.
* sysdeps/i386/fpu/fesetenv.c: Include <unistd.h>, <ldsodefs.h>
and <dl-procinfo.h>.
(__fesetenv): Always set __eip, __cs_selector, __opcode,
__data_offset and __data_selector in environment to 0. Set SSE
state if supported.
* sysdeps/x86/fpu/Makefile [$(subdir) = math] (tests): Add
test-fenv-sse.
[$(subdir) = math] (CFLAGS-test-fenv-sse.c): Add -msse2
-mfpmath=sse.
* sysdeps/x86/fpu/test-fenv-sse.c: New file.
2014-05-09 16:59:56 +00:00
|
|
|
This file is part of the GNU C Library.
|
|
|
|
|
|
|
|
The GNU C Library is free software; you can redistribute it and/or
|
|
|
|
modify it under the terms of the GNU Lesser General Public
|
|
|
|
License as published by the Free Software Foundation; either
|
|
|
|
version 2.1 of the License, or (at your option) any later version.
|
|
|
|
|
|
|
|
The GNU C Library is distributed in the hope that it will be useful,
|
|
|
|
but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
|
|
MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
|
|
|
|
Lesser General Public License for more details.
|
|
|
|
|
|
|
|
You should have received a copy of the GNU Lesser General Public
|
|
|
|
License along with the GNU C Library; if not, see
|
Prefer https to http for gnu.org and fsf.org URLs
Also, change sources.redhat.com to sourceware.org.
This patch was automatically generated by running the following shell
script, which uses GNU sed, and which avoids modifying files imported
from upstream:
sed -ri '
s,(http|ftp)(://(.*\.)?(gnu|fsf|sourceware)\.org($|[^.]|\.[^a-z])),https\2,g
s,(http|ftp)(://(.*\.)?)sources\.redhat\.com($|[^.]|\.[^a-z]),https\2sourceware.org\4,g
' \
$(find $(git ls-files) -prune -type f \
! -name '*.po' \
! -name 'ChangeLog*' \
! -path COPYING ! -path COPYING.LIB \
! -path manual/fdl-1.3.texi ! -path manual/lgpl-2.1.texi \
! -path manual/texinfo.tex ! -path scripts/config.guess \
! -path scripts/config.sub ! -path scripts/install-sh \
! -path scripts/mkinstalldirs ! -path scripts/move-if-change \
! -path INSTALL ! -path locale/programs/charmap-kw.h \
! -path po/libc.pot ! -path sysdeps/gnu/errlist.c \
! '(' -name configure \
-execdir test -f configure.ac -o -f configure.in ';' ')' \
! '(' -name preconfigure \
-execdir test -f preconfigure.ac ';' ')' \
-print)
and then by running 'make dist-prepare' to regenerate files built
from the altered files, and then executing the following to cleanup:
chmod a+x sysdeps/unix/sysv/linux/riscv/configure
# Omit irrelevant whitespace and comment-only changes,
# perhaps from a slightly-different Autoconf version.
git checkout -f \
sysdeps/csky/configure \
sysdeps/hppa/configure \
sysdeps/riscv/configure \
sysdeps/unix/sysv/linux/csky/configure
# Omit changes that caused a pre-commit check to fail like this:
# remote: *** error: sysdeps/powerpc/powerpc64/ppc-mcount.S: trailing lines
git checkout -f \
sysdeps/powerpc/powerpc64/ppc-mcount.S \
sysdeps/unix/sysv/linux/s390/s390-64/syscall.S
# Omit change that caused a pre-commit check to fail like this:
# remote: *** error: sysdeps/sparc/sparc64/multiarch/memcpy-ultra3.S: last line does not end in newline
git checkout -f sysdeps/sparc/sparc64/multiarch/memcpy-ultra3.S
2019-09-07 05:40:42 +00:00
|
|
|
<https://www.gnu.org/licenses/>. */
|
Include SSE state in i386 fenv_t (bug 16064).
This patch fixes bug 16064, i386 fenv_t not including SSE state, using
the technique suggested there of storing the state in the existing
__eip field of fenv_t to avoid needing to increase the size of fenv_t
and add new symbol versions. The included testcase, which previously
failed for i386 (but passed for x86_64), illustrates how the previous
state was buggy.
This patch causes the SSE state to be included *to the extent it is on
x86_64*. Where some state should logically be included but isn't for
x86_64 (see bug 16068), this patch does not cause it to be included
for i386 either. The idea is that any patch fixing that bug should
fix it for both x86_64 and i386 at once.
Tested i386 and x86_64. (I haven't tested the case of a CPU without
SSE2 disabling the test.)
[BZ #16064]
* sysdeps/i386/fpu/fegetenv.c: Include <unistd.h>, <ldsodefs.h>
and <dl-procinfo.h>.
(__fegetenv): Save SSE state in envp->__eip if supported.
* sysdeps/i386/fpu/feholdexcpt.c (feholdexcept): Save SSE state in
envp->__eip if supported.
* sysdeps/i386/fpu/fesetenv.c: Include <unistd.h>, <ldsodefs.h>
and <dl-procinfo.h>.
(__fesetenv): Always set __eip, __cs_selector, __opcode,
__data_offset and __data_selector in environment to 0. Set SSE
state if supported.
* sysdeps/x86/fpu/Makefile [$(subdir) = math] (tests): Add
test-fenv-sse.
[$(subdir) = math] (CFLAGS-test-fenv-sse.c): Add -msse2
-mfpmath=sse.
* sysdeps/x86/fpu/test-fenv-sse.c: New file.
2014-05-09 16:59:56 +00:00
|
|
|
|
|
|
|
#include <cpuid.h>
|
|
|
|
#include <fenv.h>
|
|
|
|
#include <float.h>
|
|
|
|
#include <stdbool.h>
|
|
|
|
#include <stdio.h>
|
|
|
|
|
|
|
|
static bool
|
|
|
|
have_sse2 (void)
|
|
|
|
{
|
|
|
|
unsigned int eax, ebx, ecx, edx;
|
|
|
|
|
|
|
|
if (!__get_cpuid (1, &eax, &ebx, &ecx, &edx))
|
|
|
|
return false;
|
|
|
|
|
|
|
|
return (edx & bit_SSE2) != 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
static __attribute__ ((noinline)) int
|
|
|
|
sse_tests (void)
|
|
|
|
{
|
|
|
|
int ret = 0;
|
|
|
|
fenv_t base_env;
|
|
|
|
if (fegetenv (&base_env) != 0)
|
|
|
|
{
|
|
|
|
puts ("fegetenv (&base_env) failed");
|
|
|
|
return 1;
|
|
|
|
}
|
|
|
|
if (fesetround (FE_UPWARD) != 0)
|
|
|
|
{
|
|
|
|
puts ("fesetround (FE_UPWARD) failed");
|
|
|
|
return 1;
|
|
|
|
}
|
|
|
|
if (fesetenv (&base_env) != 0)
|
|
|
|
{
|
|
|
|
puts ("fesetenv (&base_env) failed");
|
|
|
|
return 1;
|
|
|
|
}
|
|
|
|
volatile float a = 1.0f, b = FLT_MIN, c;
|
|
|
|
c = a + b;
|
|
|
|
if (c != 1.0f)
|
|
|
|
{
|
|
|
|
puts ("fesetenv did not restore rounding mode");
|
|
|
|
ret = 1;
|
|
|
|
}
|
|
|
|
if (fesetround (FE_DOWNWARD) != 0)
|
|
|
|
{
|
|
|
|
puts ("fesetround (FE_DOWNWARD) failed");
|
|
|
|
return 1;
|
|
|
|
}
|
|
|
|
if (feupdateenv (&base_env) != 0)
|
|
|
|
{
|
|
|
|
puts ("feupdateenv (&base_env) failed");
|
|
|
|
return 1;
|
|
|
|
}
|
|
|
|
volatile float d = -FLT_MIN, e;
|
|
|
|
e = a + d;
|
|
|
|
if (e != 1.0f)
|
|
|
|
{
|
|
|
|
puts ("feupdateenv did not restore rounding mode");
|
|
|
|
ret = 1;
|
|
|
|
}
|
|
|
|
if (fesetround (FE_UPWARD) != 0)
|
|
|
|
{
|
|
|
|
puts ("fesetround (FE_UPWARD) failed");
|
|
|
|
return 1;
|
|
|
|
}
|
|
|
|
fenv_t upward_env;
|
|
|
|
if (feholdexcept (&upward_env) != 0)
|
|
|
|
{
|
|
|
|
puts ("feholdexcept (&upward_env) failed");
|
|
|
|
return 1;
|
|
|
|
}
|
|
|
|
if (fesetround (FE_DOWNWARD) != 0)
|
|
|
|
{
|
|
|
|
puts ("fesetround (FE_DOWNWARD) failed");
|
|
|
|
return 1;
|
|
|
|
}
|
|
|
|
if (fesetenv (&upward_env) != 0)
|
|
|
|
{
|
|
|
|
puts ("fesetenv (&upward_env) failed");
|
|
|
|
return 1;
|
|
|
|
}
|
|
|
|
e = a + d;
|
|
|
|
if (e != 1.0f)
|
|
|
|
{
|
|
|
|
puts ("fesetenv did not restore rounding mode from feholdexcept");
|
|
|
|
ret = 1;
|
|
|
|
}
|
|
|
|
if (fesetround (FE_UPWARD) != 0)
|
|
|
|
{
|
|
|
|
puts ("fesetround (FE_UPWARD) failed");
|
|
|
|
return 1;
|
|
|
|
}
|
|
|
|
if (fesetenv (FE_DFL_ENV) != 0)
|
|
|
|
{
|
|
|
|
puts ("fesetenv (FE_DFL_ENV) failed");
|
|
|
|
return 1;
|
|
|
|
}
|
|
|
|
c = a + b;
|
|
|
|
if (c != 1.0f)
|
|
|
|
{
|
|
|
|
puts ("fesetenv (FE_DFL_ENV) did not restore rounding mode");
|
|
|
|
ret = 1;
|
|
|
|
}
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
|
|
|
static int
|
|
|
|
do_test (void)
|
|
|
|
{
|
|
|
|
if (!have_sse2 ())
|
|
|
|
{
|
|
|
|
puts ("CPU does not support SSE2, cannot test");
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
return sse_tests ();
|
|
|
|
}
|
|
|
|
|
|
|
|
#define TEST_FUNCTION do_test ()
|
|
|
|
#include <test-skeleton.c>
|