diff options
author | Roman Zippel <zippel@linux-m68k.org> | 2008-05-01 07:34:39 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2008-05-01 11:03:59 -0400 |
commit | 8383c42399f394a89bd6c2f03632c53689bdde7a (patch) | |
tree | c3f3a42a546a4afe3746e5894bcd425dc5a3d0bb | |
parent | 7fc5c78409479d826341b103bdf734cb4fb02436 (diff) |
ntp: remove current_tick_length()
current_tick_length used to do a little more, but now it just returns
tick_length, which we can also access directly at the few places, where it's
needed.
Signed-off-by: Roman Zippel <zippel@linux-m68k.org>
Cc: john stultz <johnstul@us.ibm.com>
Cc: Thomas Gleixner <tglx@linutronix.de>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | include/linux/timex.h | 4 | ||||
-rw-r--r-- | kernel/time/ntp.c | 16 | ||||
-rw-r--r-- | kernel/time/timekeeping.c | 5 |
3 files changed, 6 insertions, 19 deletions
diff --git a/include/linux/timex.h b/include/linux/timex.h index 5994d39de778..da431f8c5c6e 100644 --- a/include/linux/timex.h +++ b/include/linux/timex.h | |||
@@ -239,8 +239,8 @@ static inline int ntp_synced(void) | |||
239 | #endif | 239 | #endif |
240 | #define NTP_INTERVAL_LENGTH (NSEC_PER_SEC/NTP_INTERVAL_FREQ) | 240 | #define NTP_INTERVAL_LENGTH (NSEC_PER_SEC/NTP_INTERVAL_FREQ) |
241 | 241 | ||
242 | /* Returns how long ticks are at present, in ns / 2^(SHIFT_SCALE-10). */ | 242 | /* Returns how long ticks are at present, in ns / 2^NTP_SCALE_SHIFT. */ |
243 | extern u64 current_tick_length(void); | 243 | extern u64 tick_length; |
244 | 244 | ||
245 | extern void second_overflow(void); | 245 | extern void second_overflow(void); |
246 | extern void update_ntp_one_tick(void); | 246 | extern void update_ntp_one_tick(void); |
diff --git a/kernel/time/ntp.c b/kernel/time/ntp.c index a8fd1ba1ef19..df9718bac8d0 100644 --- a/kernel/time/ntp.c +++ b/kernel/time/ntp.c | |||
@@ -23,7 +23,8 @@ | |||
23 | */ | 23 | */ |
24 | unsigned long tick_usec = TICK_USEC; /* USER_HZ period (usec) */ | 24 | unsigned long tick_usec = TICK_USEC; /* USER_HZ period (usec) */ |
25 | unsigned long tick_nsec; /* ACTHZ period (nsec) */ | 25 | unsigned long tick_nsec; /* ACTHZ period (nsec) */ |
26 | static u64 tick_length, tick_length_base; | 26 | u64 tick_length; |
27 | static u64 tick_length_base; | ||
27 | 28 | ||
28 | #define MAX_TICKADJ 500 /* microsecs */ | 29 | #define MAX_TICKADJ 500 /* microsecs */ |
29 | #define MAX_TICKADJ_SCALED (((u64)(MAX_TICKADJ * NSEC_PER_USEC) << \ | 30 | #define MAX_TICKADJ_SCALED (((u64)(MAX_TICKADJ * NSEC_PER_USEC) << \ |
@@ -203,19 +204,6 @@ void second_overflow(void) | |||
203 | } | 204 | } |
204 | } | 205 | } |
205 | 206 | ||
206 | /* | ||
207 | * Return how long ticks are at the moment, that is, how much time | ||
208 | * update_wall_time_one_tick will add to xtime next time we call it | ||
209 | * (assuming no calls to do_adjtimex in the meantime). | ||
210 | * The return value is in fixed-point nanoseconds shifted by the | ||
211 | * specified number of bits to the right of the binary point. | ||
212 | * This function has no side-effects. | ||
213 | */ | ||
214 | u64 current_tick_length(void) | ||
215 | { | ||
216 | return tick_length; | ||
217 | } | ||
218 | |||
219 | #ifdef CONFIG_GENERIC_CMOS_UPDATE | 207 | #ifdef CONFIG_GENERIC_CMOS_UPDATE |
220 | 208 | ||
221 | /* Disable the cmos update - used by virtualization and embedded */ | 209 | /* Disable the cmos update - used by virtualization and embedded */ |
diff --git a/kernel/time/timekeeping.c b/kernel/time/timekeeping.c index a26429bc772a..7e74d8092067 100644 --- a/kernel/time/timekeeping.c +++ b/kernel/time/timekeeping.c | |||
@@ -380,8 +380,7 @@ static __always_inline int clocksource_bigadjust(s64 error, s64 *interval, | |||
380 | * Now calculate the error in (1 << look_ahead) ticks, but first | 380 | * Now calculate the error in (1 << look_ahead) ticks, but first |
381 | * remove the single look ahead already included in the error. | 381 | * remove the single look ahead already included in the error. |
382 | */ | 382 | */ |
383 | tick_error = current_tick_length() >> | 383 | tick_error = tick_length >> (NTP_SCALE_SHIFT - clock->shift + 1); |
384 | (NTP_SCALE_SHIFT - clock->shift + 1); | ||
385 | tick_error -= clock->xtime_interval >> 1; | 384 | tick_error -= clock->xtime_interval >> 1; |
386 | error = ((error - tick_error) >> look_ahead) + tick_error; | 385 | error = ((error - tick_error) >> look_ahead) + tick_error; |
387 | 386 | ||
@@ -473,7 +472,7 @@ void update_wall_time(void) | |||
473 | } | 472 | } |
474 | 473 | ||
475 | /* accumulate error between NTP and clock interval */ | 474 | /* accumulate error between NTP and clock interval */ |
476 | clock->error += current_tick_length(); | 475 | clock->error += tick_length; |
477 | clock->error -= clock->xtime_interval << (NTP_SCALE_SHIFT - clock->shift); | 476 | clock->error -= clock->xtime_interval << (NTP_SCALE_SHIFT - clock->shift); |
478 | } | 477 | } |
479 | 478 | ||