summaryrefslogtreecommitdiff
path: root/src/select/pselect.c
diff options
context:
space:
mode:
authorRich Felker <dalias@aerifal.cx>2019-07-28 18:15:22 -0400
committerRich Felker <dalias@aerifal.cx>2019-07-28 18:15:22 -0400
commit22276671d031639f1bd55d7dbf817290c321c7bf (patch)
tree31e824b8a7d6a37fe95ae0bcab406778d20edcfa /src/select/pselect.c
parent1492bdf53c70f436b0fc2a7238011f9786bf4cd5 (diff)
downloadmusl-22276671d031639f1bd55d7dbf817290c321c7bf.tar.gz
pselect, ppoll: add time64 syscall support, decouple 32-bit time_t
time64 syscall is used only if it's the only one defined for the arch, or if the requested timeout length does not fit in 32 bits. on current 32-bit archs where time_t is a 32-bit type, this makes it statically unreachable. on 64-bit archs, there are only superficial changes to the code after preprocessing. both before and after these changes, these functions copied their timeout arguments to avoid letting the kernel clobber the caller's copies. now, the copying also serves to change the type from userspace timespec to a pair of longs, which makes a difference only in the 32-bit fallback case, not on 64-bit.
Diffstat (limited to 'src/select/pselect.c')
-rw-r--r--src/select/pselect.c20
1 files changed, 17 insertions, 3 deletions
diff --git a/src/select/pselect.c b/src/select/pselect.c
index 762af37f..54cfb291 100644
--- a/src/select/pselect.c
+++ b/src/select/pselect.c
@@ -1,12 +1,26 @@
#include <sys/select.h>
#include <signal.h>
#include <stdint.h>
+#include <errno.h>
#include "syscall.h"
+#define IS32BIT(x) !((x)+0x80000000ULL>>32)
+#define CLAMP(x) (int)(IS32BIT(x) ? (x) : 0x7fffffffU+((0ULL+(x))>>63))
+
int pselect(int n, fd_set *restrict rfds, fd_set *restrict wfds, fd_set *restrict efds, const struct timespec *restrict ts, const sigset_t *restrict mask)
{
syscall_arg_t data[2] = { (uintptr_t)mask, _NSIG/8 };
- struct timespec ts_tmp;
- if (ts) ts_tmp = *ts;
- return syscall_cp(SYS_pselect6, n, rfds, wfds, efds, ts ? &ts_tmp : 0, data);
+ time_t s = ts ? ts->tv_sec : 0;
+ long ns = ts ? ts->tv_nsec : 0;
+#ifdef SYS_pselect6_time64
+ int r = -ENOSYS;
+ if (SYS_pselect6 == SYS_pselect6_time64 || !IS32BIT(s))
+ r = __syscall_cp(SYS_pselect6_time64, n, rfds, wfds, efds,
+ ts ? ((long long[]){s, ns}) : 0, data);
+ if (SYS_pselect6 == SYS_pselect6_time64 || r!=-ENOSYS)
+ return __syscall_ret(r);
+ s = CLAMP(s);
+#endif
+ return syscall_cp(SYS_pselect6, n, rfds, wfds, efds,
+ ts ? ((long[]){s, ns}) : 0, data);
}