diff options
| author | Xunlei Pang <pang.xunlei@linaro.org> | 2015-04-01 23:34:23 -0400 |
|---|---|---|
| committer | Ingo Molnar <mingo@kernel.org> | 2015-04-03 02:18:20 -0400 |
| commit | 3c00a1fe8496ff29ab62764bb3f4ce4b48089004 (patch) | |
| tree | 1d0ebcd277eb8e7bede0f7e4d8ef26a04285e30b | |
| parent | 2ee966320028ac846654eba5344540eeb4dc228d (diff) | |
time: Add y2038 safe update_persistent_clock64()
As part of addressing in-kernel y2038 issues, this patch adds
update_persistent_clock64() and replaces all the call sites of
update_persistent_clock() with this function. This is a __weak
implementation, which simply calls the existing y2038 unsafe
update_persistent_clock().
This allows architecture specific implementations to be
converted independently, and eventually y2038-unsafe
update_persistent_clock() can be removed after all its
architecture specific implementations have been converted to
update_persistent_clock64().
Suggested-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Xunlei Pang <pang.xunlei@linaro.org>
Signed-off-by: John Stultz <john.stultz@linaro.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Thomas Gleixner <tglx@linutronix.de>
Link: http://lkml.kernel.org/r/1427945681-29972-4-git-send-email-john.stultz@linaro.org
Signed-off-by: Ingo Molnar <mingo@kernel.org>
| -rw-r--r-- | drivers/rtc/systohc.c | 2 | ||||
| -rw-r--r-- | include/linux/timekeeping.h | 1 | ||||
| -rw-r--r-- | kernel/time/ntp.c | 13 |
3 files changed, 14 insertions, 2 deletions
diff --git a/drivers/rtc/systohc.c b/drivers/rtc/systohc.c index eb71872d0361..ef3c07a52c3a 100644 --- a/drivers/rtc/systohc.c +++ b/drivers/rtc/systohc.c | |||
| @@ -11,7 +11,7 @@ | |||
| 11 | * rtc_set_ntp_time - Save NTP synchronized time to the RTC | 11 | * rtc_set_ntp_time - Save NTP synchronized time to the RTC |
| 12 | * @now: Current time of day | 12 | * @now: Current time of day |
| 13 | * | 13 | * |
| 14 | * Replacement for the NTP platform function update_persistent_clock | 14 | * Replacement for the NTP platform function update_persistent_clock64 |
| 15 | * that stores time for later retrieval by rtc_hctosys. | 15 | * that stores time for later retrieval by rtc_hctosys. |
| 16 | * | 16 | * |
| 17 | * Returns 0 on successful RTC update, -ENODEV if a RTC update is not | 17 | * Returns 0 on successful RTC update, -ENODEV if a RTC update is not |
diff --git a/include/linux/timekeeping.h b/include/linux/timekeeping.h index 4c0f76f4616c..7a2369d5b3f4 100644 --- a/include/linux/timekeeping.h +++ b/include/linux/timekeeping.h | |||
| @@ -272,6 +272,7 @@ extern void read_persistent_clock64(struct timespec64 *ts); | |||
| 272 | extern void read_boot_clock(struct timespec *ts); | 272 | extern void read_boot_clock(struct timespec *ts); |
| 273 | extern void read_boot_clock64(struct timespec64 *ts); | 273 | extern void read_boot_clock64(struct timespec64 *ts); |
| 274 | extern int update_persistent_clock(struct timespec now); | 274 | extern int update_persistent_clock(struct timespec now); |
| 275 | extern int update_persistent_clock64(struct timespec64 now); | ||
| 275 | 276 | ||
| 276 | 277 | ||
| 277 | #endif | 278 | #endif |
diff --git a/kernel/time/ntp.c b/kernel/time/ntp.c index 9ad60d028508..7a681003001c 100644 --- a/kernel/time/ntp.c +++ b/kernel/time/ntp.c | |||
| @@ -458,6 +458,16 @@ out: | |||
| 458 | return leap; | 458 | return leap; |
| 459 | } | 459 | } |
| 460 | 460 | ||
| 461 | #ifdef CONFIG_GENERIC_CMOS_UPDATE | ||
| 462 | int __weak update_persistent_clock64(struct timespec64 now64) | ||
| 463 | { | ||
| 464 | struct timespec now; | ||
| 465 | |||
| 466 | now = timespec64_to_timespec(now64); | ||
| 467 | return update_persistent_clock(now); | ||
| 468 | } | ||
| 469 | #endif | ||
| 470 | |||
| 461 | #if defined(CONFIG_GENERIC_CMOS_UPDATE) || defined(CONFIG_RTC_SYSTOHC) | 471 | #if defined(CONFIG_GENERIC_CMOS_UPDATE) || defined(CONFIG_RTC_SYSTOHC) |
| 462 | static void sync_cmos_clock(struct work_struct *work); | 472 | static void sync_cmos_clock(struct work_struct *work); |
| 463 | 473 | ||
| @@ -493,8 +503,9 @@ static void sync_cmos_clock(struct work_struct *work) | |||
| 493 | if (persistent_clock_is_local) | 503 | if (persistent_clock_is_local) |
| 494 | adjust.tv_sec -= (sys_tz.tz_minuteswest * 60); | 504 | adjust.tv_sec -= (sys_tz.tz_minuteswest * 60); |
| 495 | #ifdef CONFIG_GENERIC_CMOS_UPDATE | 505 | #ifdef CONFIG_GENERIC_CMOS_UPDATE |
| 496 | fail = update_persistent_clock(timespec64_to_timespec(adjust)); | 506 | fail = update_persistent_clock64(adjust); |
| 497 | #endif | 507 | #endif |
| 508 | |||
| 498 | #ifdef CONFIG_RTC_SYSTOHC | 509 | #ifdef CONFIG_RTC_SYSTOHC |
| 499 | if (fail == -ENODEV) | 510 | if (fail == -ENODEV) |
| 500 | fail = rtc_set_ntp_time(adjust); | 511 | fail = rtc_set_ntp_time(adjust); |
