Commit 3c9c12f4 authored by Deepa Dinamani's avatar Deepa Dinamani Committed by Thomas Gleixner

time: Change k_clock clock_get() 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.
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-4-git-send-email-deepa.kernel@gmail.comSigned-off-by: default avatarThomas Gleixner <tglx@linutronix.de>
parent d340266e
...@@ -478,13 +478,13 @@ static int sgi_clock_period; ...@@ -478,13 +478,13 @@ static int sgi_clock_period;
static struct timespec sgi_clock_offset; static struct timespec sgi_clock_offset;
static int sgi_clock_period; static int sgi_clock_period;
static int sgi_clock_get(clockid_t clockid, struct timespec *tp) static int sgi_clock_get(clockid_t clockid, struct timespec64 *tp)
{ {
u64 nsec; u64 nsec;
nsec = rtc_time() * sgi_clock_period nsec = rtc_time() * sgi_clock_period
+ sgi_clock_offset.tv_nsec; + sgi_clock_offset.tv_nsec;
*tp = ns_to_timespec(nsec); *tp = ns_to_timespec64(nsec);
tp->tv_sec += sgi_clock_offset.tv_sec; tp->tv_sec += sgi_clock_offset.tv_sec;
return 0; return 0;
}; };
......
...@@ -90,7 +90,7 @@ struct k_clock { ...@@ -90,7 +90,7 @@ struct k_clock {
int (*clock_getres) (const clockid_t which_clock, struct timespec *tp); int (*clock_getres) (const clockid_t which_clock, struct timespec *tp);
int (*clock_set) (const clockid_t which_clock, int (*clock_set) (const clockid_t which_clock,
const struct timespec *tp); const struct timespec *tp);
int (*clock_get) (const clockid_t which_clock, struct timespec * tp); int (*clock_get) (const clockid_t which_clock, struct timespec64 *tp);
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,
......
...@@ -258,6 +258,11 @@ static inline void timekeeping_clocktai(struct timespec *ts) ...@@ -258,6 +258,11 @@ static inline void timekeeping_clocktai(struct timespec *ts)
*ts = ktime_to_timespec(ktime_get_clocktai()); *ts = ktime_to_timespec(ktime_get_clocktai());
} }
static inline void timekeeping_clocktai64(struct timespec64 *ts)
{
*ts = ktime_to_timespec64(ktime_get_clocktai());
}
/* /*
* RTC specific * RTC specific
*/ */
......
...@@ -558,14 +558,14 @@ static int alarm_clock_getres(const clockid_t which_clock, struct timespec *tp) ...@@ -558,14 +558,14 @@ static int alarm_clock_getres(const clockid_t which_clock, struct timespec *tp)
* *
* Provides the underlying alarm base time. * Provides the underlying alarm base time.
*/ */
static int alarm_clock_get(clockid_t which_clock, struct timespec *tp) static int alarm_clock_get(clockid_t which_clock, struct timespec64 *tp)
{ {
struct alarm_base *base = &alarm_bases[clock2alarm(which_clock)]; struct alarm_base *base = &alarm_bases[clock2alarm(which_clock)];
if (!alarmtimer_get_rtcdev()) if (!alarmtimer_get_rtcdev())
return -EINVAL; return -EINVAL;
*tp = ktime_to_timespec(base->gettime()); *tp = ktime_to_timespec64(base->gettime());
return 0; return 0;
} }
......
...@@ -297,20 +297,17 @@ static int pc_clock_adjtime(clockid_t id, struct timex *tx) ...@@ -297,20 +297,17 @@ static int pc_clock_adjtime(clockid_t id, struct timex *tx)
return err; return err;
} }
static int pc_clock_gettime(clockid_t id, struct timespec *ts) static int pc_clock_gettime(clockid_t id, struct timespec64 *ts)
{ {
struct posix_clock_desc cd; struct posix_clock_desc cd;
struct timespec64 ts64;
int err; int err;
err = get_clock_desc(id, &cd); err = get_clock_desc(id, &cd);
if (err) if (err)
return err; return err;
if (cd.clk->ops.clock_gettime) { if (cd.clk->ops.clock_gettime)
err = cd.clk->ops.clock_gettime(cd.clk, &ts64); err = cd.clk->ops.clock_gettime(cd.clk, ts);
*ts = timespec64_to_timespec(ts64);
}
else else
err = -EOPNOTSUPP; err = -EOPNOTSUPP;
......
...@@ -261,7 +261,7 @@ static int cpu_clock_sample_group(const clockid_t which_clock, ...@@ -261,7 +261,7 @@ static int cpu_clock_sample_group(const clockid_t which_clock,
static int posix_cpu_clock_get_task(struct task_struct *tsk, static int posix_cpu_clock_get_task(struct task_struct *tsk,
const clockid_t which_clock, const clockid_t which_clock,
struct timespec *tp) struct timespec64 *tp)
{ {
int err = -EINVAL; int err = -EINVAL;
u64 rtn; u64 rtn;
...@@ -275,13 +275,13 @@ static int posix_cpu_clock_get_task(struct task_struct *tsk, ...@@ -275,13 +275,13 @@ static int posix_cpu_clock_get_task(struct task_struct *tsk,
} }
if (!err) if (!err)
*tp = ns_to_timespec(rtn); *tp = ns_to_timespec64(rtn);
return err; return err;
} }
static int posix_cpu_clock_get(const clockid_t which_clock, struct timespec *tp) static int posix_cpu_clock_get(const clockid_t which_clock, struct timespec64 *tp)
{ {
const pid_t pid = CPUCLOCK_PID(which_clock); const pid_t pid = CPUCLOCK_PID(which_clock);
int err = -EINVAL; int err = -EINVAL;
...@@ -1374,7 +1374,7 @@ static int process_cpu_clock_getres(const clockid_t which_clock, ...@@ -1374,7 +1374,7 @@ static int process_cpu_clock_getres(const clockid_t which_clock,
return posix_cpu_clock_getres(PROCESS_CLOCK, tp); return posix_cpu_clock_getres(PROCESS_CLOCK, tp);
} }
static int process_cpu_clock_get(const clockid_t which_clock, static int process_cpu_clock_get(const clockid_t which_clock,
struct timespec *tp) struct timespec64 *tp)
{ {
return posix_cpu_clock_get(PROCESS_CLOCK, tp); return posix_cpu_clock_get(PROCESS_CLOCK, tp);
} }
...@@ -1399,7 +1399,7 @@ static int thread_cpu_clock_getres(const clockid_t which_clock, ...@@ -1399,7 +1399,7 @@ static int thread_cpu_clock_getres(const clockid_t which_clock,
return posix_cpu_clock_getres(THREAD_CLOCK, tp); return posix_cpu_clock_getres(THREAD_CLOCK, tp);
} }
static int thread_cpu_clock_get(const clockid_t which_clock, static int thread_cpu_clock_get(const clockid_t which_clock,
struct timespec *tp) struct timespec64 *tp)
{ {
return posix_cpu_clock_get(THREAD_CLOCK, tp); return posix_cpu_clock_get(THREAD_CLOCK, tp);
} }
......
...@@ -64,14 +64,17 @@ SYSCALL_DEFINE2(clock_settime, const clockid_t, which_clock, ...@@ -64,14 +64,17 @@ SYSCALL_DEFINE2(clock_settime, const clockid_t, which_clock,
SYSCALL_DEFINE2(clock_gettime, const clockid_t, which_clock, SYSCALL_DEFINE2(clock_gettime, const clockid_t, which_clock,
struct timespec __user *,tp) struct timespec __user *,tp)
{ {
struct timespec64 kernel_tp64;
struct timespec kernel_tp; struct timespec kernel_tp;
switch (which_clock) { switch (which_clock) {
case CLOCK_REALTIME: ktime_get_real_ts(&kernel_tp); break; case CLOCK_REALTIME: ktime_get_real_ts64(&kernel_tp64); break;
case CLOCK_MONOTONIC: ktime_get_ts(&kernel_tp); break; case CLOCK_MONOTONIC: ktime_get_ts64(&kernel_tp64); break;
case CLOCK_BOOTTIME: get_monotonic_boottime(&kernel_tp); break; case CLOCK_BOOTTIME: get_monotonic_boottime64(&kernel_tp64); break;
default: return -EINVAL; default: return -EINVAL;
} }
kernel_tp = timespec64_to_timespec(kernel_tp64);
if (copy_to_user(tp, &kernel_tp, sizeof (kernel_tp))) if (copy_to_user(tp, &kernel_tp, sizeof (kernel_tp)))
return -EFAULT; return -EFAULT;
return 0; return 0;
......
...@@ -204,9 +204,9 @@ static inline void unlock_timer(struct k_itimer *timr, unsigned long flags) ...@@ -204,9 +204,9 @@ static inline void unlock_timer(struct k_itimer *timr, unsigned long flags)
} }
/* Get clock_realtime */ /* Get clock_realtime */
static int posix_clock_realtime_get(clockid_t which_clock, struct timespec *tp) static int posix_clock_realtime_get(clockid_t which_clock, struct timespec64 *tp)
{ {
ktime_get_real_ts(tp); ktime_get_real_ts64(tp);
return 0; return 0;
} }
...@@ -229,32 +229,32 @@ static int posix_clock_realtime_adj(const clockid_t which_clock, ...@@ -229,32 +229,32 @@ static int posix_clock_realtime_adj(const clockid_t which_clock,
/* /*
* Get monotonic time for posix timers * Get monotonic time for posix timers
*/ */
static int posix_ktime_get_ts(clockid_t which_clock, struct timespec *tp) static int posix_ktime_get_ts(clockid_t which_clock, struct timespec64 *tp)
{ {
ktime_get_ts(tp); ktime_get_ts64(tp);
return 0; return 0;
} }
/* /*
* Get monotonic-raw time for posix timers * Get monotonic-raw time for posix timers
*/ */
static int posix_get_monotonic_raw(clockid_t which_clock, struct timespec *tp) static int posix_get_monotonic_raw(clockid_t which_clock, struct timespec64 *tp)
{ {
getrawmonotonic(tp); getrawmonotonic64(tp);
return 0; return 0;
} }
static int posix_get_realtime_coarse(clockid_t which_clock, struct timespec *tp) static int posix_get_realtime_coarse(clockid_t which_clock, struct timespec64 *tp)
{ {
*tp = current_kernel_time(); *tp = current_kernel_time64();
return 0; return 0;
} }
static int posix_get_monotonic_coarse(clockid_t which_clock, static int posix_get_monotonic_coarse(clockid_t which_clock,
struct timespec *tp) struct timespec64 *tp)
{ {
*tp = get_monotonic_coarse(); *tp = get_monotonic_coarse64();
return 0; return 0;
} }
...@@ -264,15 +264,15 @@ static int posix_get_coarse_res(const clockid_t which_clock, struct timespec *tp ...@@ -264,15 +264,15 @@ static int posix_get_coarse_res(const clockid_t which_clock, struct timespec *tp
return 0; return 0;
} }
static int posix_get_boottime(const clockid_t which_clock, struct timespec *tp) static int posix_get_boottime(const clockid_t which_clock, struct timespec64 *tp)
{ {
get_monotonic_boottime(tp); get_monotonic_boottime64(tp);
return 0; return 0;
} }
static int posix_get_tai(clockid_t which_clock, struct timespec *tp) static int posix_get_tai(clockid_t which_clock, struct timespec64 *tp)
{ {
timekeeping_clocktai(tp); timekeeping_clocktai64(tp);
return 0; return 0;
} }
...@@ -1032,13 +1032,15 @@ SYSCALL_DEFINE2(clock_gettime, const clockid_t, which_clock, ...@@ -1032,13 +1032,15 @@ SYSCALL_DEFINE2(clock_gettime, const clockid_t, which_clock,
struct timespec __user *,tp) struct timespec __user *,tp)
{ {
struct k_clock *kc = clockid_to_kclock(which_clock); struct k_clock *kc = clockid_to_kclock(which_clock);
struct timespec64 kernel_tp64;
struct timespec kernel_tp; struct timespec kernel_tp;
int error; int error;
if (!kc) if (!kc)
return -EINVAL; return -EINVAL;
error = kc->clock_get(which_clock, &kernel_tp); error = kc->clock_get(which_clock, &kernel_tp64);
kernel_tp = timespec64_to_timespec(kernel_tp64);
if (!error && copy_to_user(tp, &kernel_tp, sizeof (kernel_tp))) if (!error && copy_to_user(tp, &kernel_tp, sizeof (kernel_tp)))
error = -EFAULT; error = -EFAULT;
......
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