]> sourceware.org Git - glibc.git/log
glibc.git
4 years agoUpdate kernel version to 5.6 in tst-mman-consts.py.
Joseph Myers [Thu, 9 Apr 2020 21:19:19 +0000 (21:19 +0000)]
Update kernel version to 5.6 in tst-mman-consts.py.

This patch updates the kernel version in the test tst-mman-consts.py
to 5.6.  (There are no new constants covered by this test in 5.6 that
need any other header changes.)

Tested with build-many-glibcs.py.

4 years agoUpdate mips libm-test-ulps
Adhemerval Zanella [Wed, 8 Apr 2020 16:53:08 +0000 (13:53 -0300)]
Update mips libm-test-ulps

4 years agoUpdate alpha libm-test-ulps
Adhemerval Zanella [Wed, 8 Apr 2020 12:40:27 +0000 (09:40 -0300)]
Update alpha libm-test-ulps

4 years agoUpdate ia64 libm-test-ulps
Adhemerval Zanella [Wed, 8 Apr 2020 12:36:05 +0000 (09:36 -0300)]
Update ia64 libm-test-ulps

4 years agoUpdate sparc libm-test-ulps
Adhemerval Zanella [Wed, 8 Apr 2020 11:53:35 +0000 (14:53 +0300)]
Update sparc libm-test-ulps

4 years agoUpdate arm libm-test-ulps
Adhemerval Zanella [Wed, 8 Apr 2020 11:50:27 +0000 (11:50 +0000)]
Update arm libm-test-ulps

4 years agoUpdate aarch64 libm-test-ulps
Adhemerval Zanella [Wed, 8 Apr 2020 11:50:15 +0000 (11:50 +0000)]
Update aarch64 libm-test-ulps

4 years agoUpdates to the shn_MM locale [BZ #25532]
kokoye2007 [Tue, 11 Feb 2020 12:32:59 +0000 (13:32 +0100)]
Updates to the shn_MM locale [BZ #25532]

4 years agopowerpc: Update ULPs and xfail more ibm128 outputs
Tulio Magno Quites Machado Filho [Tue, 7 Apr 2020 14:41:29 +0000 (11:41 -0300)]
powerpc: Update ULPs and xfail more ibm128 outputs

There are 2 new input values that require to be marked as
xfail-rounding:ibm128-libgcc as they're known to fail because of libgcc
issues with different rounding modes.
Otherwise, the other tests just need an increase in ULP.

4 years agoi386: Remove build support for GCC older than GCC 6
H.J. Lu [Tue, 7 Apr 2020 13:44:37 +0000 (06:44 -0700)]
i386: Remove build support for GCC older than GCC 6

Since GCC 6.2 or later is required to build glibc, remove build support
for GCC older than GCC 6.

Testd with GCC 6.4 and GCC 9.3.

Reviewed-by: Adhemerval Zanella <adhemerval.zanella@linaro.org>
4 years agooc_FR locale: Fix spelling of April (bug 25639)
Rafał Lużyński [Mon, 6 Apr 2020 21:45:52 +0000 (23:45 +0200)]
oc_FR locale: Fix spelling of April (bug 25639)

Confirmed by CLDR and a native speaker: "abril" is more often used even
if "abrial" is also correct.  Both nominative (alt_mon) and genitive (mon)
cases are updated.

4 years agoUpdate hppa libm-test-ulps
John David Anglin [Mon, 6 Apr 2020 22:19:03 +0000 (22:19 +0000)]
Update hppa libm-test-ulps

4 years agoy2038: linux: Provide __mq_timedreceive_time64 implementation
Lukasz Majewski [Mon, 10 Feb 2020 16:36:33 +0000 (17:36 +0100)]
y2038: linux: Provide __mq_timedreceive_time64 implementation

This patch provides new __mq_timedreceive_time64 explicit 64 bit function for
receiving messages with absolute timeout.
Moreover, a 32 bit version - __mq_timedreceive has been refactored to
internally use __mq_timedreceive_time64.

The __mq_timedreceive is now supposed to be used on systems still supporting 32
bit time (__TIMESIZE != 64) - hence the necessary conversion to 64 bit struct
__timespec64 from struct timespec.

The new mq_timedsend_time64 syscall available from Linux 5.1+ has been used,
when applicable.

As this wrapper function is also used internally in the glibc, to e.g. provide
mq_receive implementation, an explicit check for abs_timeout being NULL has been
added due to conversions between struct timespec and struct __timespec64.
Before this change the Linux kernel handled this NULL pointer.

Build tests:
- ./src/scripts/build-many-glibcs.py glibcs

Run-time tests:
- Run specific tests on ARM/x86 32bit systems (qemu):
  https://github.com/lmajewski/meta-y2038 and run tests:
  https://github.com/lmajewski/y2038-tests/commits/master

Linux kernel, headers and minimal kernel version for glibc build test matrix:
- Linux v5.1 (with mq_timedreceive_time64) and glibc built with v5.1 as
  minimal kernel version (--enable-kernel="5.1.0")
  The __ASSUME_TIME64_SYSCALLS flag defined.

- Linux v5.1 and default minimal kernel version
  The __ASSUME_TIME64_SYSCALLS not defined, but kernel supports
  mq_timedreceive_time64 syscall.

- Linux v4.19 (no mq_timedreceive_time64 support) with default minimal kernel
  version for contemporary glibc (3.2.0)
  This kernel doesn't support mq_timedreceive_time64 syscall, so the fallback to
  mq_timedreceive is tested.

Above tests were performed with Y2038 redirection applied as well as without
(so the __TIMESIZE != 64 execution path is checked as well).

Reviewed-by: Adhemerval Zanella <adhemerval.zanella@linaro.org>
4 years agoy2038: linux: Provide __mq_timedsend_time64 implementation
Lukasz Majewski [Mon, 10 Feb 2020 09:55:24 +0000 (10:55 +0100)]
y2038: linux: Provide __mq_timedsend_time64 implementation

This patch provides new __mq_timedsend_time64 explicit 64 bit function for
sending messages with absolute timeout.
Moreover, a 32 bit version - __mq_timedsend has been refactored to internally
use __mq_timedsend_time64.

The __mq_timedsend is now supposed to be used on systems still supporting 32
bit time (__TIMESIZE != 64) - hence the necessary conversion to 64 bit struct
__timespec64 from struct timespec.

The new __mq_timedsend_time64 syscall available from Linux 5.1+ has been used,
when applicable.

As this wrapper function is also used internally in the glibc, to e.g. provide
mq_send implementation, an explicit check for abs_timeout being NULL has been
added due to conversions between struct timespec and struct __timespec64.
Before this change the Linux kernel handled this NULL pointer.

Build tests:
- ./src/scripts/build-many-glibcs.py glibcs

Run-time tests:
- Run specific tests on ARM/x86 32bit systems (qemu):
  https://github.com/lmajewski/meta-y2038 and run tests:
  https://github.com/lmajewski/y2038-tests/commits/master

Linux kernel, headers and minimal kernel version for glibc build test matrix:
- Linux v5.1 (with mq_timedsend_time64) and glibc built with v5.1 as a
  minimal kernel version (--enable-kernel="5.1.0")
  The __ASSUME_TIME64_SYSCALLS flag defined.

- Linux v5.1 and default minimal kernel version
  The __ASSUME_TIME64_SYSCALLS not defined, but kernel supports
  mq_timedsend_time64 syscall.

- Linux v4.19 (no mq_timedsend_time64 support) with default minimal kernel
  version for contemporary glibc (3.2.0)
  This kernel doesn't support mq_timedsend_time64 syscall, so the fallback to
  mq_timedsend is tested.

Above tests were performed with Y2038 redirection applied as well as without
(so the __TIMESIZE != 64 execution path is checked as well).

Reviewed-by: Adhemerval Zanella <adhemerval.zanella@linaro.org>
4 years agoy2038: include: Move struct __timespec64 definition to a separate file
Lukasz Majewski [Wed, 12 Feb 2020 09:42:49 +0000 (10:42 +0100)]
y2038: include: Move struct __timespec64 definition to a separate file

The struct __timespec64's definition has been moved from ./include/time.h to
./include/struct___timespec64.h.

This change would prevent from polluting other glibc namespaces (when
headers are modified to support 64 bit time on architectures with
__WORDSIZE==32).

Now it is possible to just include definition of this particular structure
when needed.

Reviewed-by: Adhemerval Zanella <adhemerval.zanella@linaro.org>
4 years agomalloc: ensure set_max_fast never stores zero [BZ #25733]
DJ Delorie [Sat, 4 Apr 2020 05:44:56 +0000 (01:44 -0400)]
malloc: ensure set_max_fast never stores zero [BZ #25733]

The code for set_max_fast() stores an "impossibly small value"
instead of zero, when the parameter is zero.  However, for
small values of the parameter (ex: 1 or 2) the computation
results in a zero being stored anyway.

This patch checks for the parameter being small enough for the
computation to result in zero instead, so that a zero is never
stored.

key values which result in zero being stored:

x86-64:  1..7  (or other 64-bit)
i686:    1..11
armhfp:  1..3  (or other 32-bit)

Reviewed-by: Carlos O'Donell <carlos@redhat.com>
4 years agopowerpc64le: enforce non-specific long double in .gnu.attributes section
Paul E. Murphy [Fri, 6 Mar 2020 15:40:27 +0000 (09:40 -0600)]
powerpc64le: enforce non-specific long double in .gnu.attributes section

We turn off this feature to avoid polluting our shared libary with
a specific value.  However, static libgcc is not under our control,
and has enabled this for ibm128 routines.  This pollutes the
resulting shared libraries with it.

Attach a post-linking hook to replace this section with one crafted
as hard-float + indeterminate ldbl.  This allows IEEE ldbl users to
avoid having to disable the gnu attributes feature which should
protect them from linking ibm ldbl libraries using the gnu attributes
feature.

Currently, this only replaces libc and libm which support both ldbl
formats and rely on application code to explicitly determine which
is to be used.

Strictly speaking, the section could be deleted with minimal lost value.
However correctly set attributes could prove useful for some future change,
and similarly missing attributes.

Reviewed-by: Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com>
4 years agopowerpc64le: workaround ieee long double / _Float128 stdc++ bug
Paul E. Murphy [Fri, 7 Feb 2020 20:08:20 +0000 (14:08 -0600)]
powerpc64le: workaround ieee long double / _Float128 stdc++ bug

-mabi=ieeelongdouble triggers the stdc++ libraries _Float128
support, which then breaks if algorithm is included.  For now,
explicitly disable _Float128 for such tests.

I have opened up GCC BZ 94080 to track this.

Reviewed-by: Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com>
4 years agopowerpc64le: Enforce -mabi=ibmlongdouble when -mfloat128 used
Paul E. Murphy [Fri, 7 Feb 2020 20:08:12 +0000 (14:08 -0600)]
powerpc64le: Enforce -mabi=ibmlongdouble when -mfloat128 used

I have observed a bug on 7.4.0 whereby __mulkc3 calls are
swapped with __multc3 depending on ABI selection.  For the
sake of being overly cautious, build all _Float128 files
with ibm128 to workaround these compilers.  This has been
noted in GCC BZ 84914, and will not be fixed for GCC 7.

Likewise, non-math files built with _Float128 are assumed
to have ibm long double.  Explicilty preserve this
assumption.

Finally, add some bootstrapping code to avoid applying
these options until IEEE long double is enabled as they
require GCC 7 and above.

Reviewed-by: Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com>
4 years agopowerpc64le/multiarch: don't generate strong aliases for fmaf128-ppc64
Paul E. Murphy [Tue, 31 Mar 2020 21:19:00 +0000 (16:19 -0500)]
powerpc64le/multiarch: don't generate strong aliases for fmaf128-ppc64

This prevents generating a second alias for __fmaieee128 when
compiling with ldouble == ieee128 redirects.

4 years agoldbl-128ibm: simplify iscanonical.h
Paul E. Murphy [Mon, 16 Mar 2020 15:49:04 +0000 (10:49 -0500)]
ldbl-128ibm: simplify iscanonical.h

The test for enabling _Float128 or IEEE 128 long double can be
greatly simplified knowing that there is no ibm128, thus we require
no special cases, and everything is canonical.

This reverts the changes to ldbl-128ibm iscanonical.h from commit
8dbfea3a2094798a52cebddde01d255483f49665 and extends the check
for __NO_LONG_DOUBLE_MATH to include a check for float128 redirects
to long double.

Reviewed-by: Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com>
4 years agoi386: Disable check_consistency for GCC 5 and above [BZ #25788]
H.J. Lu [Mon, 6 Apr 2020 13:02:11 +0000 (06:02 -0700)]
i386: Disable check_consistency for GCC 5 and above [BZ #25788]

check_consistency should be disabled for GCC 5 and above since there is
no fixed PIC register in GCC 5 and above.  Check __GNUC_PREREQ (5,0)
instead OPTIMIZE_FOR_GCC_5 since OPTIMIZE_FOR_GCC_5 is false with
-fno-omit-frame-pointer.

4 years agoAdd IPPROTO_ETHERNET and IPPROTO_MPTCP from Linux 5.6 to netinet/in.h.
Joseph Myers [Fri, 3 Apr 2020 18:08:28 +0000 (18:08 +0000)]
Add IPPROTO_ETHERNET and IPPROTO_MPTCP from Linux 5.6 to netinet/in.h.

This patch adds the IPPROTO_ETHERNET and IPPROTO_MPTCP constants from
Linux 5.6 to glibc's netinet/in.h.

Tested for x86_64.

4 years agoUpdate syscall lists for Linux 5.6.
Joseph Myers [Fri, 3 Apr 2020 18:07:55 +0000 (18:07 +0000)]
Update syscall lists for Linux 5.6.

Linux 5.6 has new openat2 and pidfd_getfd syscalls.  This patch adds
them to syscall-names.list and regenerates the arch-syscall.h files.

Tested with build-many-glibcs.py.

4 years agoelf: Implement DT_AUDIT, DT_DEPAUDIT support [BZ #24943]
Florian Weimer [Mon, 17 Feb 2020 15:49:40 +0000 (16:49 +0100)]
elf: Implement DT_AUDIT, DT_DEPAUDIT support [BZ #24943]

binutils ld has supported --audit, --depaudit for a long time,
only support in glibc has been missing.

Reviewed-by: Carlos O'Donell <carlos@redhat.com>
4 years agoelf: Simplify handling of lists of audit strings
Florian Weimer [Fri, 3 Apr 2020 11:17:48 +0000 (13:17 +0200)]
elf: Simplify handling of lists of audit strings

All list elements are colon-separated strings, and there is a hard
upper limit for the number of audit modules, so it is possible to
pre-allocate a fixed-size array of strings to which the LD_AUDIT
environment variable and --audit arguments are added.

Also eliminate the global variables for the audit list because
the list is only needed briefly during startup.

There is a slight behavior change: All duplicate LD_AUDIT environment
variables are now processed, not just the last one as before.  However,
such environment vectors are invalid anyway.

Reviewed-by: Carlos O'Donell <carlos@redhat.com>
4 years agosupport: Change xgetline to return 0 on EOF
Florian Weimer [Thu, 2 Apr 2020 15:09:36 +0000 (17:09 +0200)]
support: Change xgetline to return 0 on EOF

The advantage is that the buffer will always contain the number
of characters as returned from the function, which allows one to use
a sequence like

  /* No more audit module output.  */
  line_length = xgetline (&buffer, &buffer_length, fp);
  TEST_COMPARE_BLOB ("", 0, buffer, line_length);

to check for an expected EOF, while also reporting any unexpected
extra data encountered.

Reviewed-by: Carlos O'Donell <carlos@redhat.com>
4 years agonptl: Remove x86_64 cancellation assembly implementations [BZ #25765]
Adhemerval Zanella [Tue, 31 Mar 2020 17:59:28 +0000 (14:59 -0300)]
nptl: Remove x86_64 cancellation assembly implementations [BZ #25765]

All cancellable syscalls are done by C implementations, so there is no
no need to use a specialized implementation to optimize register usage.

It fixes BZ #25765.

Checked on x86_64-linux-gnu.

4 years agoaarch64: update bits/hwcap.h
Szabolcs Nagy [Tue, 17 Mar 2020 15:53:34 +0000 (15:53 +0000)]
aarch64: update bits/hwcap.h

Up to date with Linux 5.6. dl-procinfo.c is not updated because
HWCAP2 bits are not handled specially in glibc.

4 years agoAdd tests for Safe-Linking
Eyal Itkin [Thu, 2 Apr 2020 11:26:35 +0000 (07:26 -0400)]
Add tests for Safe-Linking

Adding the test "tst-safe-linking" for testing that Safe-Linking works
as expected. The test checks these 3 main flows:
 * tcache protection
 * fastbin protection
 * malloc_consolidate() correctness

As there is a random chance of 1/16 that of the alignment will remain
correct, the test checks each flow up to 10 times, using different random
values for the pointer corruption. As a result, the chance for a false
failure of a given tested flow is 2**(-40), thus highly unlikely.

Reviewed-by: Carlos O'Donell <carlos@redhat.com>
4 years agoS390: Regenerate ULPs.
Stefan Liebler [Fri, 3 Apr 2020 07:38:02 +0000 (09:38 +0200)]
S390: Regenerate ULPs.

Updates needed after recent commit
a9d42c09a327540a99f2eac25a98fd2ad6d0b540
math: Add inputs that yield larger errors for float type (x86_64)

4 years agosysv/alpha: Use generic __timeval32 and helpers
Alistair Francis [Tue, 11 Feb 2020 21:16:14 +0000 (13:16 -0800)]
sysv/alpha: Use generic __timeval32 and helpers

Now there is a generic __timeval32 and helpers we can use them for Alpha
instead of the Alpha specific ones.

Reviewed-by: Lukasz Majewski <lukma@denx.de>
Reviewed-by: Adhemerval Zanella <adhemerval.zanella@linaro.org>
4 years agolinux: Use long time_t for wait4/getrusage
Alistair Francis [Mon, 23 Dec 2019 21:26:50 +0000 (13:26 -0800)]
linux: Use long time_t for wait4/getrusage

The Linux kernel expects rusage to use a 32-bit time_t, even on archs
with a 64-bit time_t (like RV32). To address this let's convert
rusage to/from 32-bit and 64-bit to ensure the kernel always gets
a 32-bit time_t.

While we are converting these functions let's also convert them to be
the y2038 safe versions. This means there is a *64 function that is
called by a backwards compatible wrapper.

Reviewed-by: Lukasz Majewski <lukma@denx.de>
Reviewed-by: Adhemerval Zanella <adhemerval.zanella@linaro.org>
4 years agoresource: Add a __rusage64 struct
Alistair Francis [Sun, 19 Jan 2020 22:57:37 +0000 (14:57 -0800)]
resource: Add a __rusage64 struct

Add a __rusage64 struct which always uses a 64-bit time_t.

Reviewed-by: Lukasz Majewski <lukma@denx.de>
Reviewed-by: Adhemerval Zanella <adhemerval.zanella@linaro.org>
4 years agolinux: Use long time_t __getitimer/__setitimer
Alistair Francis [Mon, 23 Dec 2019 19:45:01 +0000 (11:45 -0800)]
linux: Use long time_t __getitimer/__setitimer

The Linux kernel expects itimerval to use a 32-bit time_t, even on archs
with a 64-bit time_t (like RV32). To address this let's convert
itimerval to/from 32-bit and 64-bit to ensure the kernel always gets
a 32-bit time_t.

While we are converting these functions let's also convert them to be
the y2038 safe versions. This means there is a *64 function that is
called by a backwards compatible wrapper.

Tested-by: Lukasz Majewski <lukma@denx.de>
Reviewed-by: Adhemerval Zanella <adhemerval.zanella@linaro.org>
4 years agosysv: Define __KERNEL_OLD_TIMEVAL_MATCHES_TIMEVAL64
Alistair Francis [Tue, 11 Feb 2020 20:56:50 +0000 (12:56 -0800)]
sysv: Define __KERNEL_OLD_TIMEVAL_MATCHES_TIMEVAL64

On y2038 safe 32-bit systems the Linux kernel expects itimerval
and rusage to use a 32-bit time_t, even though the other time_t's
are 64-bit. There are currently no plans to make 64-bit time_t versions
of these structs.

There are also other occurrences where the time passed to the kernel via
timeval doesn't match the wordsize.

To handle these cases let's define a new macro
__KERNEL_OLD_TIMEVAL_MATCHES_TIMEVAL64. This macro specifies if the
kernel's old_timeval matches the new timeval64. This should be 1 for
64-bit architectures except for Alpha's osf syscalls. The define should
be 0 for 32-bit architectures and Alpha's osf syscalls.

Reviewed-by: Lukasz Majewski <lukma@denx.de>
Reviewed-by: Adhemerval Zanella <adhemerval.zanella@linaro.org>
4 years agomath: Add inputs that yield larger errors for float type (x86_64)
Paul Zimmermann [Mon, 30 Mar 2020 13:55:55 +0000 (15:55 +0200)]
math: Add inputs that yield larger errors for float type (x86_64)

The corner cases included were generated using exhaustive search
for all float/binary32 values on x86_64 (comparing to MPFR for
correct rounding to nearest).

For the j0/j1/y0 functions, only cases with ulp error <= 9 were
included.

Reviewed-by: Carlos O'Donell <carlos@redhat.com>
4 years agoFix alignment bug in Safe-Linking
Eyal Itkin [Tue, 31 Mar 2020 06:00:14 +0000 (02:00 -0400)]
Fix alignment bug in Safe-Linking

Alignment checks should be performed on the user's buffer and NOT
on the mchunkptr as was done before. This caused bugs in 32 bit
versions, because: 2*sizeof(t) != MALLOC_ALIGNMENT.

As the tcache works on users' buffers it uses the aligned_OK()
check, and the rest work on mchunkptr and therefore check using
misaligned_chunk().

Reviewed-by: Carlos O'Donell <carlos@redhat.com>
4 years agoTypo fixes and CR cleanup in Safe-Linking
Eyal Itkin [Tue, 31 Mar 2020 05:55:13 +0000 (01:55 -0400)]
Typo fixes and CR cleanup in Safe-Linking

Removed unneeded '\' chars from end of lines and fixed some
indentation issues that were introduced in the original
Safe-Linking patch.

Reviewed-by: Carlos O'Donell <carlos@redhat.com>
4 years agoUse Linux 5.6 and GMP 6.2.0 in build-many-glibcs.py.
Joseph Myers [Tue, 31 Mar 2020 00:22:22 +0000 (00:22 +0000)]
Use Linux 5.6 and GMP 6.2.0 in build-many-glibcs.py.

This patch makes build-many-glibcs.py use the current versions of
Linux (5.6) and GMP (6.2.0).

Tested with build-many-glibcs.py (host-libraries, compilers and glibcs
builds).

4 years agoAdd new file missed in previous hppa commit.
John David Anglin [Mon, 30 Mar 2020 21:58:06 +0000 (21:58 +0000)]
Add new file missed in previous hppa commit.

4 years agopowerpc: Add support for fmaf128() in hardware
Raphael Moreira Zinsly [Fri, 20 Mar 2020 20:52:28 +0000 (17:52 -0300)]
powerpc: Add support for fmaf128() in hardware

Adds a POWER9 version of fmaf128 that uses the xsmaddqp
instruction.

Co-authored-by: Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com>
Reviewed-by: Adhemerval Zanella <adhemerval.zanella@linaro.org>
4 years agoFix data race in setting function descriptors during lazy binding on hppa.
John David Anglin [Mon, 30 Mar 2020 20:36:49 +0000 (20:36 +0000)]
Fix data race in setting function descriptors during lazy binding on hppa.

This addresses an issue that is present mainly on SMP machines running
threaded code.  In a typical indirect call or PLT import stub, the
target address is loaded first.  Then the global pointer is loaded into
the PIC register in the delay slot of a branch to the target address.
During lazy binding, the target address is a trampoline which transfers
to _dl_runtime_resolve().

_dl_runtime_resolve() uses the relocation offset stored in the global
pointer and the linkage map stored in the trampoline to find the
relocation.  Then, the function descriptor is updated.

In a multi-threaded application, it is possible for the global pointer
to be updated between the load of the target address and the global
pointer.  When this happens, the relocation offset has been replaced
by the new global pointer.  The function pointer has probably been
updated as well but there is no way to find the address of the function
descriptor and to transfer to the target.  So, _dl_runtime_resolve()
typically crashes.

HP-UX addressed this problem by adding an extra pc-relative branch to
the trampoline.  The descriptor is initially setup to point to the
branch.  The branch then transfers to the trampoline.  This allowed
the trampoline code to figure out which descriptor was being used
without any modification to user code.  I didn't use this approach
as it is more complex and changes function pointer canonicalization.

The order of loading the target address and global pointer in
indirect calls was not consistent with the order used in import stubs.
In particular, $$dyncall and some inline versions of it loaded the
global pointer first.  This was inconsistent with the global pointer
being updated first in dl-machine.h.  Assuming the accesses are
ordered, we want elf_machine_fixup_plt() to store the global pointer
first and calls to load it last.  Then, the global pointer will be
correct when the target function is entered.

However, just to make things more fun, HP added support for
out-of-order execution of accesses in PA 2.0.  The accesses used by
calls are weakly ordered. So, it's possibly under some circumstances
that a function might be entered with the wrong global pointer.
However, HP uses weakly ordered accesses in 64-bit HP-UX, so I assume
that loading the global pointer in the delay slot of the branch must
work consistently.

The basic fix for the race is a combination of modifying user code to
preserve the address of the function descriptor in register %r22 and
setting the least-significant bit in the relocation offset.  The
latter was suggested by Carlos as a way to distinguish relocation
offsets from global pointer values.  Conventionally, %r22 is used
as the address of the function descriptor in calls to $$dyncall.
So, it wasn't hard to preserve the address in %r22.

I have updated gcc trunk and gcc-9 branch to not clobber %r22 in
$$dyncall and inline indirect calls.  I have also modified the import
stubs in binutils trunk and the 2.33 branch to preserve %r22.  This
required making the stubs one instruction longer but we save one
relocation.  I also modified binutils to align the .plt section on
a 8-byte boundary.  This allows descriptors to be updated atomically
with a floting-point store.

With these changes, _dl_runtime_resolve() can fallback to an alternate
mechanism to find the relocation offset when it has been clobbered.
There's just one additional instruction in the fast path. I tested
the fallback function, _dl_fix_reloc_arg(), by changing the branch to
always use the fallback.  Old code still runs as it did before.

Fixes bug 23296.

Reviewed-by: Carlos O'Donell <carlos@redhat.com>
4 years agosparc: Move __fenv_{ld,st}fsr to fenv-private.h
Adhemerval Zanella [Mon, 9 Mar 2020 17:31:03 +0000 (20:31 +0300)]
sparc: Move __fenv_{ld,st}fsr to fenv-private.h

These should not be exported on installed headers.

Checked on sparc64-linux-gnu and sparcv9-linux-gnu.

4 years agox86: Remove feraiseexcept optimization
Adhemerval Zanella [Mon, 9 Mar 2020 13:21:51 +0000 (10:21 -0300)]
x86: Remove feraiseexcept optimization

Similar to fenvinline.h removal, this kind of optimization is better
implemented by the compiler.  Also newer code avoid setting exceptions
directly (for instance the code to make new logf, log2f and powf
implementatation to now support SVID compat).

The BZ#94194 [1] the corresponding GCC bug for adding replacements
for these on x86.

Checked on x86_64-linux-gnu and i686-linux-gnu.

[1] https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94194

4 years agomath: Remove fenvinline.h
Adhemerval Zanella [Fri, 6 Mar 2020 16:42:07 +0000 (13:42 -0300)]
math: Remove fenvinline.h

Similar to string2.h (18b10de7ce) and string3.h (09a596cc2c) this
patch removes the fenvinline.h on all architectures.  Currently
only powerpc implements some optimizations.  This kind of optimization
is better implemented by the compiler (which handles the architecture
ISA transparently).

Also, for the specific optimized powerpc implementation the code is
becoming convoluted and these micro-optimization are hardly wildly
used, even more being a possible hotspot in realword cases
(non-default rounding are used only on specific cases and exception
handling are done most likely only on errors path).  Only x86
implements similar optimization (on fenv.h) also indicates that
these should no be on libc.

The math/test-fenv already covers all math/test-fenvinline tests,
so it is safe to remove it.

The powerpc fegetround optimization is moved to internal
fenv_libc.h.

The BZ#94193 [1] the corresponding GCC bug for adding replacements
for these on powerpc.

Checked on x86_64-linux-gnu and powerpc64le-linux-gnu.

[1] https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94193

4 years agohurd: Make O_TRUNC update mtime/ctime
Samuel Thibault [Sun, 29 Mar 2020 17:42:55 +0000 (19:42 +0200)]
hurd: Make O_TRUNC update mtime/ctime

* hurd/lookup-retry.c (__hurd_file_name_lookup_retry): Call
__file_utimens after __file_set_size.

4 years agoAdd Safe-Linking to fastbins and tcache
Eyal Itkin [Fri, 20 Mar 2020 19:19:17 +0000 (21:19 +0200)]
Add Safe-Linking to fastbins and tcache

Safe-Linking is a security mechanism that protects single-linked
lists (such as the fastbin and tcache) from being tampered by attackers.
The mechanism makes use of randomness from ASLR (mmap_base), and when
combined with chunk alignment integrity checks, it protects the "next"
pointers from being hijacked by an attacker.

While Safe-Unlinking protects double-linked lists (such as the small
bins), there wasn't any similar protection for attacks against
single-linked lists. This solution protects against 3 common attacks:
  * Partial pointer override: modifies the lower bytes (Little Endian)
  * Full pointer override: hijacks the pointer to an attacker's location
  * Unaligned chunks: pointing the list to an unaligned address

The design assumes an attacker doesn't know where the heap is located,
and uses the ASLR randomness to "sign" the single-linked pointers. We
mark the pointer as P and the location in which it is stored as L, and
the calculation will be:
  * PROTECT(P) := (L >> PAGE_SHIFT) XOR (P)
  * *L = PROTECT(P)

This way, the random bits from the address L (which start at the bit
in the PAGE_SHIFT position), will be merged with LSB of the stored
protected pointer. This protection layer prevents an attacker from
modifying the pointer into a controlled value.

An additional check that the chunks are MALLOC_ALIGNed adds an
important layer:
  * Attackers can't point to illegal (unaligned) memory addresses
  * Attackers must guess correctly the alignment bits

On standard 32 bit Linux machines, an attack will directly fail 7
out of 8 times, and on 64 bit machines it will fail 15 out of 16
times.

This proposed patch was benchmarked and it's effect on the overall
performance of the heap was negligible and couldn't be distinguished
from the default variance between tests on the vanilla version. A
similar protection was added to Chromium's version of TCMalloc
in 2012, and according to their documentation it had an overhead of
less than 2%.

Reviewed-by: DJ Delorie <dj@redhat.com>
Reviewed-by: Carlos O'Donell <carlos@redhat.com>
Reviewed-by: Adhemerval Zacnella <adhemerval.zanella@linaro.org>
4 years agoAdd benchtests for roundeven and roundevenf.
Shen-Ta Hsieh [Fri, 27 Mar 2020 23:24:02 +0000 (23:24 +0000)]
Add benchtests for roundeven and roundevenf.

This patch adds benchtests for the roundeven and roundevenf functions.
The inputs are copied from trunc-inputs.

4 years agotime: Add a __itimerval64 struct
Alistair Francis [Sat, 18 Jan 2020 23:17:03 +0000 (15:17 -0800)]
time: Add a __itimerval64 struct

Add a __itimerval64 which always uses a 64-bit time_t.

Reviewed-by: Lukasz Majewski <lukma@denx.de>
Reviewed-by: Adhemerval Zanella <adhemerval.zanella@linaro.org>
4 years agotime: Add a timeval with a 32-bit tv_sec and tv_usec
Alistair Francis [Fri, 27 Dec 2019 17:07:40 +0000 (09:07 -0800)]
time: Add a timeval with a 32-bit tv_sec and tv_usec

On y2038 safe 32-bit systems the Linux kernel expects itimerval to
use a 32-bit time_t, even though the other time_t's are 64-bit. To
address this let's add a __timeval32 struct to be used internally.

Reviewed-by: Lukasz Majewski <lukma@denx.de>
Reviewed-by: Adhemerval Zanella <adhemerval.zanella@linaro.org>
4 years agosysv/linux: Rename alpha functions to be alpha specific
Alistair Francis [Tue, 21 Jan 2020 07:54:35 +0000 (23:54 -0800)]
sysv/linux: Rename alpha functions to be alpha specific

These functions are alpha specifc, rename them to be clear.

Let's also rename the header file from tv32-compat.h to
alpha-tv32-compat.h. This is to avoid conflicts with the one we will
introduce later.

Reviewed-by: Lukasz Majewski <lukma@denx.de>
Reviewed-by: Adhemerval Zanella <adhemerval.zanella@linaro.org>
4 years agoARC: add definitions to elf/elf.h
Vineet Gupta [Wed, 30 Mar 2016 12:52:22 +0000 (18:22 +0530)]
ARC: add definitions to elf/elf.h

Signed-off-by: Vineet Gupta <vgupta@synopsys.com>
4 years agopowerpc64: apply -mabi=ibmlongdouble to special files
Paul E. Murphy [Fri, 7 Feb 2020 20:08:08 +0000 (14:08 -0600)]
powerpc64: apply -mabi=ibmlongdouble to special files

Some of these files depend on the avoidance of using the various
register sets of POWER.  When enabling the IEEE 128 long double,
we must be sure to disable this ABI as some compilers will
refuse to compile if -mno-vsx and -mabi=ieeelongdouble are both
present.

Reviewed-by: Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com>
4 years agopowerpc64le: add -mno-gnu-attribute to *f128 objects and difftime
Paul E. Murphy [Fri, 7 Feb 2020 20:08:01 +0000 (14:08 -0600)]
powerpc64le: add -mno-gnu-attribute to *f128 objects and difftime

In practice, this flag should be applied globally, but it makes a good
sanity check to ensure ibm128 and ieee128 long double files are not
getting mismatched.  _Float128 files use no long double, thus are
always safe to use this option.

Similarly, when investigating the linker complaints, difftime
makes trivial, self contained, usage of long double, so thus it
is also explicitly marked as such.

Reviewed-by: Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com>
4 years agoMakeconfig: sandwich gnulib-tests between libc/ld linking of tests
Paul E. Murphy [Fri, 7 Feb 2020 20:08:01 +0000 (14:08 -0600)]
Makeconfig: sandwich gnulib-tests between libc/ld linking of tests

This better resembles the default linking process with the gnulibs,
and also resolves the increasingly difficult to maintain
f128-loader-link usage on powerpc64le as some libgcc symbols are
dependent on those found in the loader (ld).

4 years agopowerpc64le: Ensure correct ldouble compiler flags are used
Gabriel F. T. Gomes [Thu, 28 Dec 2017 19:30:06 +0000 (17:30 -0200)]
powerpc64le: Ensure correct ldouble compiler flags are used

Ensure the correct ldouble abi flags are applied to ibm128 files and
nldbl files.  Remove the IEEE options if used, and apply the flags
used to build ldouble files which are ibm128 abi.

nldbl tests are a little tricky.  To use the support, we must remove
all ldouble abi flags, and ensure -mlong-double-64 is used.

Co-authored-by: Rajalakshmi Srinivasaraghavan <raji@linux.vnet.ibm.com>
Co-authored-by: Tulio Magno Quites Machado Filho <tuliom@linux.vnet.ibm.com>
Co-authored-by: Paul E. Murphy <murphyp@linux.vnet.ibm.com>
4 years agoFix tests which expose ldbl -> _Float128 redirects
Paul E. Murphy [Fri, 7 Feb 2020 20:08:16 +0000 (14:08 -0600)]
Fix tests which expose ldbl -> _Float128 redirects

The ldbl redirects for ieee128 have some jagged edges when
inspecting and manipulating symbols directly.

e.g asprintf is unconditionally redirected to __asprintfieee128
thus any tests relying on GCC's redirect behavior will encounter
problems if they inspect the symbol names too closely.

I've mitigated tests which expose the limitations of the
ldbl -> f128 redirects by giving them knowledge about the
redirected symbol names.

Hopefully there isn't much user code which depends on this
implementation specific behavior.

Reviewed-by: Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com>
4 years agoldbl-128ibm-compat: PLT redirects for using ldbl redirects internally
Paul E. Murphy [Fri, 7 Feb 2020 20:07:55 +0000 (14:07 -0600)]
ldbl-128ibm-compat: PLT redirects for using ldbl redirects internally

Tweak the PLT bypass magic when building glibc with long double
redirects.  This is made more difficult by the fact we only get
one chance to redirect functions.  This happens via the public
headers.

There are roughly three classes of redirect we need to attend to
today:

 1. Simple redirects, redirected via cdef macro overrides and
    and new libc_hidden_ldbl_proto macro.
 2. Internal usage of internal API, e.g __snprintf, which has
    no direct analogue.  This is bypassed directly on case-by-
    case basis.
 3. Double redirects, e.g sscanf and related.  These require
    a heavier handed approach of macro renaming to existing
    symbols.

Most simple redirects are handled via 1.  Ideally, the libc_*
macro would live in libc-symbols.h, but in practice the macros
needed for it to do anything useful live in cdefs.h, so they
are defined in the local override.

Notably, the internal name of the asprintf generated for ieee ldbl
redirects is renamed to work with internal prefixed usage.

This resolves the local plt usage introduced when building glibc
with ldbl == ieee128 on ppc64le.

Reviewed-by: Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com>
4 years agostdlib: Move tst-system to tests-container
Adhemerval Zanella [Tue, 24 Mar 2020 18:48:34 +0000 (15:48 -0300)]
stdlib: Move tst-system to tests-container

Fix some issues with different shell and error messages.

Checked on x86_64-linux-gnu and i686-linux-gnu.

4 years agosupport/shell-container.c: Add builtin kill
Adhemerval Zanella [Tue, 24 Mar 2020 18:47:13 +0000 (15:47 -0300)]
support/shell-container.c: Add builtin kill

No options supported.

Reviewed-by: DJ Delorie <dj@redhat.com>
4 years agosupport/shell-container.c: Add builtin exit
Adhemerval Zanella [Tue, 24 Mar 2020 18:40:36 +0000 (15:40 -0300)]
support/shell-container.c: Add builtin exit

Reviewed-by: DJ Delorie <dj@redhat.com>
4 years agosupport/shell-container.c: Return 127 if execve fails
Adhemerval Zanella [Tue, 24 Mar 2020 18:36:23 +0000 (15:36 -0300)]
support/shell-container.c: Return 127 if execve fails

Reviewed-by: DJ Delorie <dj@redhat.com>
4 years agoAdd NEWS entry for CVE-2020-1751 (bug 25423)
Aurelien Jarno [Tue, 24 Mar 2020 21:49:10 +0000 (22:49 +0100)]
Add NEWS entry for CVE-2020-1751 (bug 25423)

Reviewed-by: Carlos O'Donell <carlos@redhat.com>
4 years agoposix: Fix system error return value [BZ #25715]
Adhemerval Zanella [Mon, 23 Mar 2020 18:23:20 +0000 (15:23 -0300)]
posix: Fix system error return value [BZ #25715]

It fixes 5fb7fc9635 when posix_spawn fails.

Checked on x86_64-linux-gnu and i686-linux-gnu.

Reviewed-by: Carlos O'Donell <carlos@redhat.com>
4 years agoy2038: fix: Add missing libc_hidden_def attribute for some syscall wrappers
Lukasz Majewski [Sat, 14 Mar 2020 23:18:36 +0000 (00:18 +0100)]
y2038: fix: Add missing libc_hidden_def attribute for some syscall wrappers

During the conversion to support 64 bit time on some architectures with
__WORDSIZE == 32 && __TIMESIZE != 64 the libc_hidden_def attribute for
eligible functions was by mistake omitted.

This patch fixes this issue and exports (and allows using) those
functions when Y2038 support is enabled in glibc.

4 years agoExtended Char Intro: Use getwc in example (Bug 25626)
Carlos O'Donell [Thu, 5 Mar 2020 02:19:36 +0000 (21:19 -0500)]
Extended Char Intro: Use getwc in example (Bug 25626)

In the "Extended Char Intro" the example incorrectly uses a function
called wgetc which doesn't exist.  The example is corrected to use
getwc, which is correct for the use in this case.

Reported-by: Toomas Rosin <toomas@rosin.ee>
4 years agostdio: Add tests for printf multibyte convertion leak [BZ#25691]
Adhemerval Zanella [Thu, 19 Mar 2020 21:35:46 +0000 (18:35 -0300)]
stdio: Add tests for printf multibyte convertion leak [BZ#25691]

Checked on x86_64-linux-gnu and i686-linux-gnu.

4 years agostdio: Remove memory leak from multibyte convertion [BZ#25691]
Florian Weimer [Thu, 19 Mar 2020 21:32:28 +0000 (18:32 -0300)]
stdio: Remove memory leak from multibyte convertion [BZ#25691]

This is an updated version of a previous patch [1] with the
following changes:

  - Use compiler overflow builtins on done_add_func function.
  - Define the scratch +utstring_converted_wide_string using
    CHAR_T.
  - Added a testcase and mention the bug report.

Both default and wide printf functions might leak memory when
manipulate multibyte characters conversion depending of the size
of the input (whether __libc_use_alloca trigger or not the fallback
heap allocation).

This patch fixes it by removing the extra memory allocation on
string formatting with conversion parts.

The testcase uses input argument size that trigger memory leaks
on unpatched code (using a scratch buffer the threashold to use
heap allocation is lower).

Checked on x86_64-linux-gnu and i686-linux-gnu.

Reviewed-by: Adhemerval Zanella <adhemerval.zanella@linaro.org>
[1] https://sourceware.org/pipermail/libc-alpha/2017-June/082098.html

4 years agoAdd NEWS entry for CVE-2020-1752 (bug 25414)
Aurelien Jarno [Thu, 19 Mar 2020 21:53:00 +0000 (22:53 +0100)]
Add NEWS entry for CVE-2020-1752 (bug 25414)

4 years agomath: Remove inline math tests
Adhemerval Zanella [Tue, 17 Mar 2020 18:46:29 +0000 (15:46 -0300)]
math: Remove inline math tests

With mathinline removal there is no need to keep building and testing
inline math tests.

The gen-libm-tests.py support to generate ULP_I_* is removed and all
libm-test-ulps files are updated to longer have the
i{float,double,ldouble} entries.  The support for no-test-inline is
also removed from both gen-auto-libm-tests and the
auto-libm-test-out-* were regenerated.

Checked on x86_64-linux-gnu and i686-linux-gnu.

4 years agoRemove __LIBC_INTERNAL_MATH_INLINES
Adhemerval Zanella [Tue, 17 Mar 2020 16:58:10 +0000 (13:58 -0300)]
Remove __LIBC_INTERNAL_MATH_INLINES

With m68k mathinline.h removal the flag is not used anymore.

Checked with a m68k-linux-gnu build/check.

4 years agomath: Remove mathinline
Adhemerval Zanella [Mon, 16 Mar 2020 19:28:39 +0000 (16:28 -0300)]
math: Remove mathinline

With m68k bits moved to internal headers, no architectures export
additional optimizations on mathinline.

4 years agom68k: Remove mathinline.h
Adhemerval Zanella [Mon, 16 Mar 2020 18:58:45 +0000 (15:58 -0300)]
m68k: Remove mathinline.h

This is similar to x86 (da75c1b180f9355a) and powerpc (32ea72999693b98e)
mathinline.h removal.  The required macros to build the fpu routines
are moved to mathimpl.h, while the inline optimization macros for
atan, tanh, rint, log1p, significand, trunc, floor, ceil, isinf,
finite, scalbn, isnan, scalbln, nearbyint, lrint, and sincos are removed.

The gcc bug https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94204 was
created to track builtin support.

Checked with a build against m68k-linux-gnu, resulting binaries
are similar with and without the patch.

4 years agooc_FR locale: Fix spelling of Thursday (bug 25639)
Rafał Lużyński [Wed, 18 Mar 2020 23:09:29 +0000 (00:09 +0100)]
oc_FR locale: Fix spelling of Thursday (bug 25639)

As reported by a native speaker:

Thursday: "dijóus" -> "dijòus" (also confirmed by CLDR)

4 years agox86: Remove ARCH_CET_LEGACY_BITMAP [BZ #25397]
H.J. Lu [Wed, 18 Mar 2020 11:35:54 +0000 (04:35 -0700)]
x86: Remove ARCH_CET_LEGACY_BITMAP [BZ #25397]

Since legacy bitmap doesn't cover jitted code generated by legacy JIT
engine, it isn't very useful.  This patch removes ARCH_CET_LEGACY_BITMAP
and treats indirect branch tracking similar to shadow stack by removing
legacy bitmap support.

Tested on CET Linux/x86-64 and non-CET Linux/x86-64.

Reviewed-by: Carlos O'Donell <carlos@redhat.com>
4 years agoFix build with GCC 10 when long double = double.
Joseph Myers [Tue, 17 Mar 2020 22:57:42 +0000 (22:57 +0000)]
Fix build with GCC 10 when long double = double.

On platforms where long double has the same ABI as double, glibc
defines long double functions as aliases for the corresponding double
functions.  The declarations of those functions in <math.h> are
disabled to avoid problems with aliases having incompatible types, but
GCC 10 now gives errors for incompatible types when the long double
function is known to GCC as a built-in function, not just when there
is an incompatible header declaration.

This patch fixes those errors by using appropriate
-fno-builtin-<function> options to compile the double functions.  The
list of CFLAGS-* settings is an appropriately adapted version of that
in sysdeps/ieee754/ldbl-opt/Makefile used there for building nldbl-*.c
files; in particular, the options are used even if GCC does not
currently have a built-in function of a given function, so that adding
such a built-in function in future will not break the glibc build.
Thus, various of the CFLAGS-* settings are only for future-proofing
and may not currently be needed (and it's possible some could be
irrelevant for other reasons).

Tested with build-many-glibcs.py for arm-linux-gnueabi (compilers and
glibcs builds), where it fixes the build that previously failed.

4 years agonscd/cachedumper.c : fix whitespace
DJ Delorie [Tue, 17 Mar 2020 18:03:43 +0000 (14:03 -0400)]
nscd/cachedumper.c : fix whitespace

4 years agoFix nscd/cachedumper.c compile errors
DJ Delorie [Sat, 14 Mar 2020 04:21:40 +0000 (00:21 -0400)]
Fix nscd/cachedumper.c compile errors

4 years agomanual: Fix inconsistent declaration of wcsrchr [BZ #24655]
Girish Joshi [Fri, 6 Mar 2020 19:47:01 +0000 (01:17 +0530)]
manual: Fix inconsistent declaration of wcsrchr [BZ #24655]

Reviewed-by: DJ Delorie <dj@redhat.com>
4 years agonscd: add cache dumper
DJ Delorie [Wed, 19 Feb 2020 17:31:09 +0000 (12:31 -0500)]
nscd: add cache dumper

4 years agoparse_tunables: Fix typo in comment
Jonathan Wakely [Fri, 13 Mar 2020 10:27:23 +0000 (10:27 +0000)]
parse_tunables: Fix typo in comment

4 years agoldconfig: trace origin paths with -v
DJ Delorie [Wed, 19 Feb 2020 17:31:38 +0000 (12:31 -0500)]
ldconfig: trace origin paths with -v

With this patch, -v turns on a "from" trace for each directory
searched, that tells you WHY that directory is being searched -
is it a builtin, from the command line, or from some config file?

4 years agotest-container: print errno when execvp fails
Michael Hudson-Doyle [Wed, 11 Mar 2020 00:05:25 +0000 (13:05 +1300)]
test-container: print errno when execvp fails

I'm debugging a situation where lots of tests using test-container fail
and it's possible knowing errno would help understand why.

Reviewed-by: DJ Delorie <dj@redhat.com>
4 years ago[AArch64] Improve integer memcpy
Wilco Dijkstra [Wed, 11 Mar 2020 17:15:25 +0000 (17:15 +0000)]
[AArch64] Improve integer memcpy

Further optimize integer memcpy.  Small cases now include copies up
to 32 bytes.  64-128 byte copies are split into two cases to improve
performance of 64-96 byte copies.  Comments have been rewritten.

4 years agoAdd NEWS entry for CVE-2020-10029 (bug 25487)
Aurelien Jarno [Wed, 4 Mar 2020 21:32:32 +0000 (22:32 +0100)]
Add NEWS entry for CVE-2020-10029 (bug 25487)

4 years agogcc PR 89877: miscompilation due to missing cc clobber in longlong.h macros
Vineet Gupta [Thu, 28 Mar 2019 22:24:35 +0000 (15:24 -0700)]
gcc PR 89877: miscompilation due to missing cc clobber in longlong.h macros

simple test such as below was failing.

| void main(int argc, char *argv[])
| {
|    size_t total_time = 115424;                       // expected 115.424
|    double secs = (double)total_time/(double)1000;
|    printf("%s %d %lf\n", "secs", total_time, secs);  // prints 113.504
|    printf("%d\n", (size_t)secs);
| }

The printf eventually called into glibc stdlib/divrem.c:__mpn_divrem()
which uses the __arc__ specific inline asm macros from longlong.h which
were causing miscompilation.

include/
2019-03-28  Vineet Gupta <vgupta@synopsys.com>

PR 89877

* longlong.h [__arc__] (add_ssaaaa): Add cc clobber
(sub_ddmmss): Likewise.

Signed-off-by: Vineet Gupta <vgupta@synopsys.com>
4 years agomips: Fix wrong INTERNAL_SYSCALL_ERROR_P check from bc2eb9321e
Adhemerval Zanella [Tue, 10 Mar 2020 20:24:14 +0000 (17:24 -0300)]
mips: Fix wrong INTERNAL_SYSCALL_ERROR_P check from bc2eb9321e

Checked on mips64-linux-gnu.

4 years agoelf: Fix wrong indentation from commit eb447b7b4b
Adhemerval Zanella [Tue, 10 Mar 2020 19:11:29 +0000 (16:11 -0300)]
elf: Fix wrong indentation from commit eb447b7b4b

4 years agoy2038: linux: Provide __futimesat64 implementation
Lukasz Majewski [Sun, 16 Feb 2020 22:42:14 +0000 (23:42 +0100)]
y2038: linux: Provide __futimesat64 implementation

This conversion patch for supporting 64 bit time for futimesat only differs
from the work performed for futimes (when providing __futimes64) with passing
also the file name (and path) to utimensat.

All the design and conversion decisions are exactly the same as for futimens
conversion.

4 years agoy2038: linux: Provide __lutimes64 implementation
Lukasz Majewski [Sun, 16 Feb 2020 22:02:05 +0000 (23:02 +0100)]
y2038: linux: Provide __lutimes64 implementation

This conversion patch for supporting 64 bit time for lutimes mostly differs from
the work performed for futimes (when providing __futimes64) with adding the
AT_SYMLINK_NOFOLLOW flag to utimensat.
It also supports passing file name instead of file descriptor number, but this
is not relevant for utimensat used to implement it.

All the design and conversion decisions are exactly the same as for futimens
conversion.

4 years agoy2038: linux: Provide __futimes64 implementation
Lukasz Majewski [Sun, 16 Feb 2020 08:32:42 +0000 (09:32 +0100)]
y2038: linux: Provide __futimes64 implementation

This patch provides new __futimes64 explicit 64 bit function for setting file's
64 bit attributes for access and modification time (by specifying file
descriptor number).

Internally, the __utimensat64_helper function is used. This patch is necessary
for having architectures with __WORDSIZE == 32 Y2038 safe.

Moreover, a 32 bit version - __futimes has been refactored to internally use
__futimes64.

The __futimes is now supposed to be used on systems still supporting 32
bit time (__TIMESIZE != 64) - hence the necessary conversion of struct timeval
to 64 bit struct __timeval64.

The check if struct timevals' usec fields are in the range between 0 and 1000000
has been removed as Linux kernel performs it internally in the implementation
of utimensat (the conversion between struct __timeval64 and __timespec64 is not
relevant for this particular check).

Last but not least, checks for tvp{64} not being NULL have been preserved from
the original code as some legacy user space programs may rely on it.

Build tests:
./src/scripts/build-many-glibcs.py glibcs

Run-time tests:
- Run specific tests on ARM/x86 32bit systems (qemu):
  https://github.com/lmajewski/meta-y2038 and run tests:
  https://github.com/lmajewski/y2038-tests/commits/master

Above tests were performed with Y2038 redirection applied as well as without to
test the proper usage of both __futimes64 and __futimes.

4 years agoy2038: fix: Add missing libc_hidden_def for __futimens64
Lukasz Majewski [Tue, 3 Mar 2020 10:21:11 +0000 (11:21 +0100)]
y2038: fix: Add missing libc_hidden_def for __futimens64

The libc_hidden_def () declaration for __futimens64 function was missing,
so it is added in this patch.

4 years agosparc: Move sigreturn stub to assembly
Adhemerval Zanella [Thu, 5 Mar 2020 16:38:24 +0000 (13:38 -0300)]
sparc: Move sigreturn stub to assembly

It seems that some gcc versions might generates a stack frame for the
sigreturn stub requires on sparc signal handling.  For instance:

  $ cat test.c
  #define _GNU_SOURCE
  #include <sys/syscall.h>

  __attribute__ ((__optimize__ ("-fno-stack-protector")))
  void
  __sigreturn_stub (void)
  {
    __asm__ ("mov %0, %%g1\n\t"
            "ta  0x10\n\t"
            : /* no outputs */
            : "i" (SYS_rt_sigreturn));
  }
  $ gcc -v
  [...]
  gcc version 9.2.1 20200224 (Debian 9.2.1-30)
  $ gcc -O2 -m64 test.c -S -o -
  [...]
    __sigreturn_stub:
          save    %sp, -176, %sp
  #APP
  ! 9 "t.c" 1
          mov 101, %g1
          ta  0x10

  ! 0 "" 2
  #NO_APP
          .size   __sigreturn_stub, .-__sigreturn_stub

As indicated by kernel developers [1], the sigreturn stub can not change
the register window or the stack pointer since the kernel has setup the
restore frame at a precise location relative to the stack pointer when
the stub is invoked.

I tried to play with some compiler flags and even with _Noreturn and
__builtin_unreachable after the asm does not help (and Sparc does not
support naked functions).

To avoid similar issues, as the stack-protector support also have
stumbled, this patch moves the implementation of the sigreturn stubs to
assembly.

Checked on sparcv9-linux-gnu and sparc64-linux-gnu with gcc 9.2.1
and gcc 7.5.0.

[1] https://lkml.org/lkml/2016/5/27/465

4 years agoldbl-128ibm: Let long double files have specific compiler flags
Tulio Magno Quites Machado Filho [Thu, 15 Feb 2018 17:44:47 +0000 (15:44 -0200)]
ldbl-128ibm: Let long double files have specific compiler flags

Soon, powerpc64le will need to provide extra compiler flags to the long
double files in order to continue to build using the IBM 128-bit
extended floating point type as long double.

4 years agoldbl-128ibm-compat: Add tests for IBM long double functions
Rajalakshmi Srinivasaraghavan [Mon, 28 May 2018 21:56:35 +0000 (18:56 -0300)]
ldbl-128ibm-compat: Add tests for IBM long double functions

This patch creates test-ibm128* tests from the long double function tests.
In order to explicitly test IBM long double functions -mabi=ibmlongdouble is
added to CFLAGS.

Likewise, update the test headers to correct choose ULPs when redirects
are enabled.

Co-authored-by: Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com>
Co-authored-by: Paul E. Murphy <murphyp@linux.vnet.ibm.com>
4 years agopowerpc: Fix feraiseexcept and feclearexcept macros
Matheus Castanho [Tue, 3 Mar 2020 18:20:38 +0000 (15:20 -0300)]
powerpc: Fix feraiseexcept and feclearexcept macros

A recent change to fenvinline.h modified the check if __e is a
a power of 2 inside feraiseexcept and feclearexcept macros.  It
introduced the use of the powerof2 macro but also removed the
if statement checking whether __e != 0 before issuing an mtfsb*
instruction.  This is problematic because powerof2 (0) evaluates
to 1 and without the removed if __e is allowed to be 0 when
__builtin_clz is called.  In that case the value 32 is passed
to __MTFSB*, which is invalid.

This commit uses __builtin_popcount instead of powerof2 to fix this
issue and avoid the extra check for __e != 0.  This was the approach
used by the initial versions of that previous patch.

Reviewed-by: Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com>
4 years agoarm: Fix softp-fp Implies (BZ #25635)
Adhemerval Zanella [Thu, 5 Mar 2020 13:14:41 +0000 (13:14 +0000)]
arm: Fix softp-fp Implies (BZ #25635)

The commit "arm: Split BE/LE abilist"
(1673ba87fefe019c834c09d33673d1d453ea698d) changed the soft-fp order for
ARM selection when __SOFTFP__ is defined by the compiler.

On 2.30 the sysdeps order is:

2.30
sysdeps/unix/sysv/linux/arm
sysdeps/arm/nptl
sysdeps/unix/sysv/linux
sysdeps/nptl
sysdeps/pthread
sysdeps/gnu
sysdeps/unix/inet
sysdeps/unix/sysv
sysdeps/unix/arm
sysdeps/unix
sysdeps/posix
sysdeps/arm/nofpu
sysdeps/ieee754/soft-fp
sysdeps/arm
sysdeps/wordsize-32
sysdeps/ieee754/flt-32
sysdeps/ieee754/dbl-64
sysdeps/ieee754
sysdeps/generic

While on master is:

sysdeps/unix/sysv/linux/arm/le
sysdeps/unix/sysv/linux/arm
sysdeps/arm/nptl
sysdeps/unix/sysv/linux
sysdeps/nptl
sysdeps/pthread
sysdeps/gnu
sysdeps/unix/inet
sysdeps/unix/sysv
sysdeps/unix/arm
sysdeps/unix
sysdeps/posix
sysdeps/arm/le
sysdeps/arm
sysdeps/wordsize-32
sysdeps/ieee754/flt-32
sysdeps/ieee754/dbl-64
sysdeps/arm/nofpu
sysdeps/ieee754/soft-fp
sysdeps/ieee754
sysdeps/generic

It make the build select some routines (fadd, fdiv, fmul, fsub, and fma)
on ieee754/flt-32 and ieee754/dbl-64 that requires fenv support to be
correctly rounded which in turns lead to math failures since the
__SOFTFP__ does not have fenv support.

With this patch the order is now:

sysdeps/unix/sysv/linux/arm/le
sysdeps/unix/sysv/linux/arm
sysdeps/arm/nptl
sysdeps/unix/sysv/linux
sysdeps/nptlsysdeps/pthread
sysdeps/gnu
sysdeps/unix/inet
sysdeps/unix/sysv
sysdeps/unix/arm
sysdeps/unix
sysdeps/posix
sysdeps/arm/le/nofpu
sysdeps/arm/nofpu
sysdeps/ieee754/soft-fp
sysdeps/arm/le
sysdeps/arm
sysdeps/wordsize-32
sysdeps/ieee754/flt-32
sysdeps/ieee754/dbl-64
sysdeps/ieee754
sysdeps/generic

Checked on arm-linux-gnuaebi.

4 years agoRemove reference of --without-fp on configure
Adhemerval Zanella [Thu, 5 Mar 2020 12:41:42 +0000 (12:41 +0000)]
Remove reference of --without-fp on configure

4 years agolinux/sysipc: Include linux/posix_types.h for __kernel_mode_t
Adhemerval Zanella [Fri, 6 Mar 2020 12:25:32 +0000 (09:25 -0300)]
linux/sysipc: Include linux/posix_types.h for __kernel_mode_t

The posix_types.h (where __kernel_mode_t is defined) is included
implicitly, which might not happen on older kernels.

This page took 0.100009 seconds and 5 git commands to generate.