aboutsummaryrefslogtreecommitdiffstats
path: root/kernel/time/ntp.c
diff options
context:
space:
mode:
authorjohn stultz <johnstul@us.ibm.com>2009-02-18 19:02:22 -0500
committerIngo Molnar <mingo@elte.hu>2009-02-19 04:10:08 -0500
commitfdcedf7b75808dd72c3cc0b931be11b04d75c60a (patch)
treec64743890ffa9722f8705ce0c214f0f08ffe4d9a /kernel/time/ntp.c
parentba95fd47d177d46743ad94055908d22840370e06 (diff)
time: apply NTP frequency/tick changes immediately
Since the GENERIC_TIME changes landed, the adjtimex behavior changed for struct timex.tick and .freq changed. When the tick or freq value is set, we adjust the tick_length_base in ntp_update_frequency(). However, this new value doesn't get applied to tick_length until the next second (via second_overflow). This means some applications that do quick time tweaking do not see the requested change made as quickly as expected. I've run a few tests with this change, and ntpd still functions fine. Signed-off-by: John Stultz <johnstul@us.ibm.com> Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'kernel/time/ntp.c')
-rw-r--r--kernel/time/ntp.c7
1 files changed, 7 insertions, 0 deletions
diff --git a/kernel/time/ntp.c b/kernel/time/ntp.c
index f5f793d92415..e1fa3689a903 100644
--- a/kernel/time/ntp.c
+++ b/kernel/time/ntp.c
@@ -51,6 +51,7 @@ static long ntp_tick_adj;
51 51
52static void ntp_update_frequency(void) 52static void ntp_update_frequency(void)
53{ 53{
54 u64 old_tick_length_base = tick_length_base;
54 u64 second_length = (u64)(tick_usec * NSEC_PER_USEC * USER_HZ) 55 u64 second_length = (u64)(tick_usec * NSEC_PER_USEC * USER_HZ)
55 << NTP_SCALE_SHIFT; 56 << NTP_SCALE_SHIFT;
56 second_length += (s64)ntp_tick_adj << NTP_SCALE_SHIFT; 57 second_length += (s64)ntp_tick_adj << NTP_SCALE_SHIFT;
@@ -60,6 +61,12 @@ static void ntp_update_frequency(void)
60 61
61 tick_nsec = div_u64(second_length, HZ) >> NTP_SCALE_SHIFT; 62 tick_nsec = div_u64(second_length, HZ) >> NTP_SCALE_SHIFT;
62 tick_length_base = div_u64(tick_length_base, NTP_INTERVAL_FREQ); 63 tick_length_base = div_u64(tick_length_base, NTP_INTERVAL_FREQ);
64
65 /*
66 * Don't wait for the next second_overflow, apply
67 * the change to the tick length immediately
68 */
69 tick_length += tick_length_base - old_tick_length_base;
63} 70}
64 71
65static void ntp_update_offset(long offset) 72static void ntp_update_offset(long offset)