Commit ad196384 authored by Deepa Dinamani's avatar Deepa Dinamani Committed by Thomas Gleixner

time: Change k_clock nsleep() to use timespec64

struct timespec is not y2038 safe on 32 bit machines.  Replace uses of
struct timespec with struct timespec64 in the kernel.

The syscall interfaces themselves will be changed in a separate series.

Note that the restart_block parameter for nanosleep has also been left
unchanged and will be part of syscall series noted above.
Signed-off-by: default avatarDeepa Dinamani <deepa.kernel@gmail.com>
Cc: y2038@lists.linaro.org
Cc: john.stultz@linaro.org
Cc: arnd@arndb.de
Link: http://lkml.kernel.org/r/1490555058-4603-8-git-send-email-deepa.kernel@gmail.comSigned-off-by: default avatarThomas Gleixner <tglx@linutronix.de>
parent 5f252b32
...@@ -452,7 +452,7 @@ static inline u64 hrtimer_forward_now(struct hrtimer *timer, ...@@ -452,7 +452,7 @@ static inline u64 hrtimer_forward_now(struct hrtimer *timer,
} }
/* Precise sleep: */ /* Precise sleep: */
extern long hrtimer_nanosleep(struct timespec *rqtp, extern long hrtimer_nanosleep(struct timespec64 *rqtp,
struct timespec __user *rmtp, struct timespec __user *rmtp,
const enum hrtimer_mode mode, const enum hrtimer_mode mode,
const clockid_t clockid); const clockid_t clockid);
......
...@@ -94,7 +94,7 @@ struct k_clock { ...@@ -94,7 +94,7 @@ struct k_clock {
int (*clock_adj) (const clockid_t which_clock, struct timex *tx); int (*clock_adj) (const clockid_t which_clock, struct timex *tx);
int (*timer_create) (struct k_itimer *timer); int (*timer_create) (struct k_itimer *timer);
int (*nsleep) (const clockid_t which_clock, int flags, int (*nsleep) (const clockid_t which_clock, int flags,
struct timespec *, struct timespec __user *); struct timespec64 *, struct timespec __user *);
long (*nsleep_restart) (struct restart_block *restart_block); long (*nsleep_restart) (struct restart_block *restart_block);
int (*timer_set) (struct k_itimer *timr, int flags, int (*timer_set) (struct k_itimer *timr, int flags,
struct itimerspec64 *new_setting, struct itimerspec64 *new_setting,
......
...@@ -240,18 +240,20 @@ COMPAT_SYSCALL_DEFINE2(nanosleep, struct compat_timespec __user *, rqtp, ...@@ -240,18 +240,20 @@ COMPAT_SYSCALL_DEFINE2(nanosleep, struct compat_timespec __user *, rqtp,
struct compat_timespec __user *, rmtp) struct compat_timespec __user *, rmtp)
{ {
struct timespec tu, rmt; struct timespec tu, rmt;
struct timespec64 tu64;
mm_segment_t oldfs; mm_segment_t oldfs;
long ret; long ret;
if (compat_get_timespec(&tu, rqtp)) if (compat_get_timespec(&tu, rqtp))
return -EFAULT; return -EFAULT;
if (!timespec_valid(&tu)) tu64 = timespec_to_timespec64(tu);
if (!timespec64_valid(&tu64))
return -EINVAL; return -EINVAL;
oldfs = get_fs(); oldfs = get_fs();
set_fs(KERNEL_DS); set_fs(KERNEL_DS);
ret = hrtimer_nanosleep(&tu, ret = hrtimer_nanosleep(&tu64,
rmtp ? (struct timespec __user *)&rmt : NULL, rmtp ? (struct timespec __user *)&rmt : NULL,
HRTIMER_MODE_REL, CLOCK_MONOTONIC); HRTIMER_MODE_REL, CLOCK_MONOTONIC);
set_fs(oldfs); set_fs(oldfs);
......
...@@ -790,13 +790,14 @@ static long __sched alarm_timer_nsleep_restart(struct restart_block *restart) ...@@ -790,13 +790,14 @@ static long __sched alarm_timer_nsleep_restart(struct restart_block *restart)
* Handles clock_nanosleep calls against _ALARM clockids * Handles clock_nanosleep calls against _ALARM clockids
*/ */
static int alarm_timer_nsleep(const clockid_t which_clock, int flags, static int alarm_timer_nsleep(const clockid_t which_clock, int flags,
struct timespec *tsreq, struct timespec __user *rmtp) struct timespec64 *tsreq,
struct timespec __user *rmtp)
{ {
enum alarmtimer_type type = clock2alarm(which_clock); enum alarmtimer_type type = clock2alarm(which_clock);
struct restart_block *restart;
struct alarm alarm; struct alarm alarm;
ktime_t exp; ktime_t exp;
int ret = 0; int ret = 0;
struct restart_block *restart;
if (!alarmtimer_get_rtcdev()) if (!alarmtimer_get_rtcdev())
return -ENOTSUPP; return -ENOTSUPP;
...@@ -809,7 +810,7 @@ static int alarm_timer_nsleep(const clockid_t which_clock, int flags, ...@@ -809,7 +810,7 @@ static int alarm_timer_nsleep(const clockid_t which_clock, int flags,
alarm_init(&alarm, type, alarmtimer_nsleep_wakeup); alarm_init(&alarm, type, alarmtimer_nsleep_wakeup);
exp = timespec_to_ktime(*tsreq); exp = timespec64_to_ktime(*tsreq);
/* Convert (if necessary) to absolute time */ /* Convert (if necessary) to absolute time */
if (flags != TIMER_ABSTIME) { if (flags != TIMER_ABSTIME) {
ktime_t now = alarm_bases[type].gettime(); ktime_t now = alarm_bases[type].gettime();
......
...@@ -1503,7 +1503,7 @@ long __sched hrtimer_nanosleep_restart(struct restart_block *restart) ...@@ -1503,7 +1503,7 @@ long __sched hrtimer_nanosleep_restart(struct restart_block *restart)
return ret; return ret;
} }
long hrtimer_nanosleep(struct timespec *rqtp, struct timespec __user *rmtp, long hrtimer_nanosleep(struct timespec64 *rqtp, struct timespec __user *rmtp,
const enum hrtimer_mode mode, const clockid_t clockid) const enum hrtimer_mode mode, const clockid_t clockid)
{ {
struct restart_block *restart; struct restart_block *restart;
...@@ -1516,7 +1516,7 @@ long hrtimer_nanosleep(struct timespec *rqtp, struct timespec __user *rmtp, ...@@ -1516,7 +1516,7 @@ long hrtimer_nanosleep(struct timespec *rqtp, struct timespec __user *rmtp,
slack = 0; slack = 0;
hrtimer_init_on_stack(&t.timer, clockid, mode); hrtimer_init_on_stack(&t.timer, clockid, mode);
hrtimer_set_expires_range_ns(&t.timer, timespec_to_ktime(*rqtp), slack); hrtimer_set_expires_range_ns(&t.timer, timespec64_to_ktime(*rqtp), slack);
if (do_nanosleep(&t, mode)) if (do_nanosleep(&t, mode))
goto out; goto out;
...@@ -1547,15 +1547,17 @@ long hrtimer_nanosleep(struct timespec *rqtp, struct timespec __user *rmtp, ...@@ -1547,15 +1547,17 @@ long hrtimer_nanosleep(struct timespec *rqtp, struct timespec __user *rmtp,
SYSCALL_DEFINE2(nanosleep, struct timespec __user *, rqtp, SYSCALL_DEFINE2(nanosleep, struct timespec __user *, rqtp,
struct timespec __user *, rmtp) struct timespec __user *, rmtp)
{ {
struct timespec64 tu64;
struct timespec tu; struct timespec tu;
if (copy_from_user(&tu, rqtp, sizeof(tu))) if (copy_from_user(&tu, rqtp, sizeof(tu)))
return -EFAULT; return -EFAULT;
if (!timespec_valid(&tu)) tu64 = timespec_to_timespec64(tu);
if (!timespec64_valid(&tu64))
return -EINVAL; return -EINVAL;
return hrtimer_nanosleep(&tu, rmtp, HRTIMER_MODE_REL, CLOCK_MONOTONIC); return hrtimer_nanosleep(&tu64, rmtp, HRTIMER_MODE_REL, CLOCK_MONOTONIC);
} }
/* /*
......
...@@ -1219,9 +1219,8 @@ void set_process_cpu_timer(struct task_struct *tsk, unsigned int clock_idx, ...@@ -1219,9 +1219,8 @@ void set_process_cpu_timer(struct task_struct *tsk, unsigned int clock_idx,
} }
static int do_cpu_nanosleep(const clockid_t which_clock, int flags, static int do_cpu_nanosleep(const clockid_t which_clock, int flags,
struct timespec *rqtp, struct itimerspec *it) struct timespec64 *rqtp, struct itimerspec64 *it)
{ {
struct itimerspec64 it64;
struct k_itimer timer; struct k_itimer timer;
int error; int error;
...@@ -1241,8 +1240,7 @@ static int do_cpu_nanosleep(const clockid_t which_clock, int flags, ...@@ -1241,8 +1240,7 @@ static int do_cpu_nanosleep(const clockid_t which_clock, int flags,
it->it_value = *rqtp; it->it_value = *rqtp;
spin_lock_irq(&timer.it_lock); spin_lock_irq(&timer.it_lock);
it64 = itimerspec_to_itimerspec64(it); error = posix_cpu_timer_set(&timer, flags, it, NULL);
error = posix_cpu_timer_set(&timer, flags, &it64, NULL);
if (error) { if (error) {
spin_unlock_irq(&timer.it_lock); spin_unlock_irq(&timer.it_lock);
return error; return error;
...@@ -1271,10 +1269,8 @@ static int do_cpu_nanosleep(const clockid_t which_clock, int flags, ...@@ -1271,10 +1269,8 @@ static int do_cpu_nanosleep(const clockid_t which_clock, int flags,
/* /*
* We were interrupted by a signal. * We were interrupted by a signal.
*/ */
*rqtp = ns_to_timespec(timer.it.cpu.expires); *rqtp = ns_to_timespec64(timer.it.cpu.expires);
it64 = itimerspec_to_itimerspec64(it); error = posix_cpu_timer_set(&timer, 0, &zero_it, it);
error = posix_cpu_timer_set(&timer, 0, &zero_it, &it64);
*it = itimerspec64_to_itimerspec(&it64);
if (!error) { if (!error) {
/* /*
* Timer is now unarmed, deletion can not fail. * Timer is now unarmed, deletion can not fail.
...@@ -1310,10 +1306,11 @@ static int do_cpu_nanosleep(const clockid_t which_clock, int flags, ...@@ -1310,10 +1306,11 @@ static int do_cpu_nanosleep(const clockid_t which_clock, int flags,
static long posix_cpu_nsleep_restart(struct restart_block *restart_block); static long posix_cpu_nsleep_restart(struct restart_block *restart_block);
static int posix_cpu_nsleep(const clockid_t which_clock, int flags, static int posix_cpu_nsleep(const clockid_t which_clock, int flags,
struct timespec *rqtp, struct timespec __user *rmtp) struct timespec64 *rqtp, struct timespec __user *rmtp)
{ {
struct restart_block *restart_block = &current->restart_block; struct restart_block *restart_block = &current->restart_block;
struct itimerspec it; struct itimerspec64 it;
struct timespec ts;
int error; int error;
/* /*
...@@ -1333,13 +1330,14 @@ static int posix_cpu_nsleep(const clockid_t which_clock, int flags, ...@@ -1333,13 +1330,14 @@ static int posix_cpu_nsleep(const clockid_t which_clock, int flags,
/* /*
* Report back to the user the time still remaining. * Report back to the user the time still remaining.
*/ */
if (rmtp && copy_to_user(rmtp, &it.it_value, sizeof *rmtp)) ts = timespec64_to_timespec(it.it_value);
if (rmtp && copy_to_user(rmtp, &ts, sizeof(*rmtp)))
return -EFAULT; return -EFAULT;
restart_block->fn = posix_cpu_nsleep_restart; restart_block->fn = posix_cpu_nsleep_restart;
restart_block->nanosleep.clockid = which_clock; restart_block->nanosleep.clockid = which_clock;
restart_block->nanosleep.rmtp = rmtp; restart_block->nanosleep.rmtp = rmtp;
restart_block->nanosleep.expires = timespec_to_ns(rqtp); restart_block->nanosleep.expires = timespec64_to_ns(rqtp);
} }
return error; return error;
} }
...@@ -1347,11 +1345,12 @@ static int posix_cpu_nsleep(const clockid_t which_clock, int flags, ...@@ -1347,11 +1345,12 @@ static int posix_cpu_nsleep(const clockid_t which_clock, int flags,
static long posix_cpu_nsleep_restart(struct restart_block *restart_block) static long posix_cpu_nsleep_restart(struct restart_block *restart_block)
{ {
clockid_t which_clock = restart_block->nanosleep.clockid; clockid_t which_clock = restart_block->nanosleep.clockid;
struct timespec t; struct itimerspec64 it;
struct itimerspec it; struct timespec64 t;
struct timespec tmp;
int error; int error;
t = ns_to_timespec(restart_block->nanosleep.expires); t = ns_to_timespec64(restart_block->nanosleep.expires);
error = do_cpu_nanosleep(which_clock, TIMER_ABSTIME, &t, &it); error = do_cpu_nanosleep(which_clock, TIMER_ABSTIME, &t, &it);
...@@ -1360,10 +1359,11 @@ static long posix_cpu_nsleep_restart(struct restart_block *restart_block) ...@@ -1360,10 +1359,11 @@ static long posix_cpu_nsleep_restart(struct restart_block *restart_block)
/* /*
* Report back to the user the time still remaining. * Report back to the user the time still remaining.
*/ */
if (rmtp && copy_to_user(rmtp, &it.it_value, sizeof *rmtp)) tmp = timespec64_to_timespec(it.it_value);
if (rmtp && copy_to_user(rmtp, &tmp, sizeof(*rmtp)))
return -EFAULT; return -EFAULT;
restart_block->nanosleep.expires = timespec_to_ns(&t); restart_block->nanosleep.expires = timespec64_to_ns(&t);
} }
return error; return error;
...@@ -1388,7 +1388,7 @@ static int process_cpu_timer_create(struct k_itimer *timer) ...@@ -1388,7 +1388,7 @@ static int process_cpu_timer_create(struct k_itimer *timer)
return posix_cpu_timer_create(timer); return posix_cpu_timer_create(timer);
} }
static int process_cpu_nsleep(const clockid_t which_clock, int flags, static int process_cpu_nsleep(const clockid_t which_clock, int flags,
struct timespec *rqtp, struct timespec64 *rqtp,
struct timespec __user *rmtp) struct timespec __user *rmtp)
{ {
return posix_cpu_nsleep(PROCESS_CLOCK, flags, rqtp, rmtp); return posix_cpu_nsleep(PROCESS_CLOCK, flags, rqtp, rmtp);
......
...@@ -103,6 +103,7 @@ SYSCALL_DEFINE4(clock_nanosleep, const clockid_t, which_clock, int, flags, ...@@ -103,6 +103,7 @@ SYSCALL_DEFINE4(clock_nanosleep, const clockid_t, which_clock, int, flags,
const struct timespec __user *, rqtp, const struct timespec __user *, rqtp,
struct timespec __user *, rmtp) struct timespec __user *, rmtp)
{ {
struct timespec64 t64;
struct timespec t; struct timespec t;
switch (which_clock) { switch (which_clock) {
...@@ -111,9 +112,10 @@ SYSCALL_DEFINE4(clock_nanosleep, const clockid_t, which_clock, int, flags, ...@@ -111,9 +112,10 @@ SYSCALL_DEFINE4(clock_nanosleep, const clockid_t, which_clock, int, flags,
case CLOCK_BOOTTIME: case CLOCK_BOOTTIME:
if (copy_from_user(&t, rqtp, sizeof (struct timespec))) if (copy_from_user(&t, rqtp, sizeof (struct timespec)))
return -EFAULT; return -EFAULT;
if (!timespec_valid(&t)) t64 = timespec_to_timespec64(t);
if (!timespec64_valid(&t64))
return -EINVAL; return -EINVAL;
return hrtimer_nanosleep(&t, rmtp, flags & TIMER_ABSTIME ? return hrtimer_nanosleep(&t64, rmtp, flags & TIMER_ABSTIME ?
HRTIMER_MODE_ABS : HRTIMER_MODE_REL, HRTIMER_MODE_ABS : HRTIMER_MODE_REL,
which_clock); which_clock);
default: default:
......
...@@ -130,7 +130,7 @@ static struct k_clock posix_clocks[MAX_CLOCKS]; ...@@ -130,7 +130,7 @@ static struct k_clock posix_clocks[MAX_CLOCKS];
/* /*
* These ones are defined below. * These ones are defined below.
*/ */
static int common_nsleep(const clockid_t, int flags, struct timespec *t, static int common_nsleep(const clockid_t, int flags, struct timespec64 *t,
struct timespec __user *rmtp); struct timespec __user *rmtp);
static int common_timer_create(struct k_itimer *new_timer); static int common_timer_create(struct k_itimer *new_timer);
static void common_timer_get(struct k_itimer *, struct itimerspec64 *); static void common_timer_get(struct k_itimer *, struct itimerspec64 *);
...@@ -1099,7 +1099,7 @@ SYSCALL_DEFINE2(clock_getres, const clockid_t, which_clock, ...@@ -1099,7 +1099,7 @@ SYSCALL_DEFINE2(clock_getres, const clockid_t, which_clock,
* nanosleep for monotonic and realtime clocks * nanosleep for monotonic and realtime clocks
*/ */
static int common_nsleep(const clockid_t which_clock, int flags, static int common_nsleep(const clockid_t which_clock, int flags,
struct timespec *tsave, struct timespec __user *rmtp) struct timespec64 *tsave, struct timespec __user *rmtp)
{ {
return hrtimer_nanosleep(tsave, rmtp, flags & TIMER_ABSTIME ? return hrtimer_nanosleep(tsave, rmtp, flags & TIMER_ABSTIME ?
HRTIMER_MODE_ABS : HRTIMER_MODE_REL, HRTIMER_MODE_ABS : HRTIMER_MODE_REL,
...@@ -1111,6 +1111,7 @@ SYSCALL_DEFINE4(clock_nanosleep, const clockid_t, which_clock, int, flags, ...@@ -1111,6 +1111,7 @@ SYSCALL_DEFINE4(clock_nanosleep, const clockid_t, which_clock, int, flags,
struct timespec __user *, rmtp) struct timespec __user *, rmtp)
{ {
struct k_clock *kc = clockid_to_kclock(which_clock); struct k_clock *kc = clockid_to_kclock(which_clock);
struct timespec64 t64;
struct timespec t; struct timespec t;
if (!kc) if (!kc)
...@@ -1121,10 +1122,11 @@ SYSCALL_DEFINE4(clock_nanosleep, const clockid_t, which_clock, int, flags, ...@@ -1121,10 +1122,11 @@ SYSCALL_DEFINE4(clock_nanosleep, const clockid_t, which_clock, int, flags,
if (copy_from_user(&t, rqtp, sizeof (struct timespec))) if (copy_from_user(&t, rqtp, sizeof (struct timespec)))
return -EFAULT; return -EFAULT;
if (!timespec_valid(&t)) t64 = timespec_to_timespec64(t);
if (!timespec64_valid(&t64))
return -EINVAL; return -EINVAL;
return kc->nsleep(which_clock, flags, &t, rmtp); return kc->nsleep(which_clock, flags, &t64, rmtp);
} }
/* /*
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment