sched_rr_get_interval: don't assume time_t is 32-bit on 32-bit archs
authorRich Felker <dalias@aerifal.cx>
Mon, 29 Jul 2019 03:26:38 +0000 (23:26 -0400)
committerRich Felker <dalias@aerifal.cx>
Mon, 29 Jul 2019 16:31:20 +0000 (12:31 -0400)
as with clock_getres, the time64 syscall for this is not necessary or
useful, this time since scheduling timeslices are not on the order 68
years. if there's a 32-bit syscall, use it and expand the result into
timespec; otherwise there is only one syscall and it does the right
thing to store to timespec directly.

on 64-bit archs, there is no change to the code after preprocessing.

src/sched/sched_rr_get_interval.c

index 4b01028..33a3d1a 100644 (file)
@@ -3,5 +3,19 @@
 
 int sched_rr_get_interval(pid_t pid, struct timespec *ts)
 {
+#ifdef SYS_sched_rr_get_interval_time64
+       /* On a 32-bit arch, use the old syscall if it exists. */
+       if (SYS_sched_rr_get_interval != SYS_sched_rr_get_interval_time64) {
+               long ts32[2];
+               int r = __syscall(SYS_sched_rr_get_interval, pid, ts32);
+               if (!r) {
+                       ts->tv_sec = ts32[0];
+                       ts->tv_nsec = ts32[1];
+               }
+               return __syscall_ret(r);
+       }
+#endif
+       /* If reaching this point, it's a 64-bit arch or time64-only
+        * 32-bit arch and we can get result directly into timespec. */
        return syscall(SYS_sched_rr_get_interval, pid, ts);
 }