diff options
| author | Adhemerval Zanella <adhemerval.zanella@linaro.org> | 2021-06-15 16:59:50 -0300 |
|---|---|---|
| committer | Adhemerval Zanella <adhemerval.zanella@linaro.org> | 2021-06-22 12:09:52 -0300 |
| commit | 91cf411ad3ef10bd18ec053854fcb919be4f6789 (patch) | |
| tree | 9d3f802bd16bda2cee81aa61509384890b9f596f /sysdeps/unix/sysv/linux/pselect32.c | |
| parent | ecf2661281c71a9752c7238ab93bc12b16cfff23 (diff) | |
| download | glibc-91cf411ad3ef10bd18ec053854fcb919be4f6789.tar glibc-91cf411ad3ef10bd18ec053854fcb919be4f6789.tar.gz glibc-91cf411ad3ef10bd18ec053854fcb919be4f6789.tar.bz2 glibc-91cf411ad3ef10bd18ec053854fcb919be4f6789.zip | |
linux: Only use 64-bit syscall if required for pselect
For !__ASSUME_TIME64_SYSCALLS there is no need to issue a 64-bit syscall
if the provided timeout fits in a 32-bit one. The 64-bit usage should
be rare since the timeout is a relative one. This also avoids the need
to use supports_time64() (which breaks the usage case of live migration
like CRIU or similar).
Checked on i686-linux-gnu on a 4.15 kernel and on a 5.11 kernel
(with and without --enable-kernel=5.1) and on x86_64-linux-gnu.
Reviewed-by: Lukasz Majewski <lukma@denx.de>
Diffstat (limited to 'sysdeps/unix/sysv/linux/pselect32.c')
| -rw-r--r-- | sysdeps/unix/sysv/linux/pselect32.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/sysdeps/unix/sysv/linux/pselect32.c b/sysdeps/unix/sysv/linux/pselect32.c index eb59b51cdf..48724d8727 100644 --- a/sysdeps/unix/sysv/linux/pselect32.c +++ b/sysdeps/unix/sysv/linux/pselect32.c @@ -29,12 +29,6 @@ __pselect32 (int nfds, fd_set *readfds, fd_set *writefds, struct timespec ts32, *pts32 = NULL; if (timeout != NULL) { - if (! in_time_t_range (timeout->tv_sec)) - { - __set_errno (EINVAL); - return -1; - } - ts32 = valid_timespec64_to_timespec (*timeout); pts32 = &ts32; } |