diff options
| author | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2008-03-09 13:06:49 -0400 |
|---|---|---|
| committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2008-03-09 13:06:49 -0400 |
| commit | bf5a25e1fff88a1066e20cc7263329405e4939f6 (patch) | |
| tree | 95822dc50801e21cf1ad1e6aa7b58af898156517 | |
| parent | 83f7a2c118833d3738b4d162ea3c17d0bd8ffa94 (diff) | |
| parent | 10a398d04c4a1fc395840f4d040493375f562302 (diff) | |
Merge git://git.kernel.org/pub/scm/linux/kernel/git/tglx/linux-2.6-hrt
* git://git.kernel.org/pub/scm/linux/kernel/git/tglx/linux-2.6-hrt:
time: remove obsolete CLOCK_TICK_ADJUST
time: don't touch an offlined CPU's ts->tick_stopped in tick_cancel_sched_timer()
time: prevent the loop in timespec_add_ns() from being optimised away
ntp: use unsigned input for do_div()
| -rw-r--r-- | include/linux/time.h | 4 | ||||
| -rw-r--r-- | include/linux/timex.h | 9 | ||||
| -rw-r--r-- | kernel/time/ntp.c | 23 | ||||
| -rw-r--r-- | kernel/time/tick-sched.c | 2 | ||||
| -rw-r--r-- | kernel/time/timekeeping.c | 6 |
5 files changed, 25 insertions, 19 deletions
diff --git a/include/linux/time.h b/include/linux/time.h index 2091a19f1655..d32ef0ad4c0a 100644 --- a/include/linux/time.h +++ b/include/linux/time.h | |||
| @@ -174,6 +174,10 @@ static inline void timespec_add_ns(struct timespec *a, u64 ns) | |||
| 174 | { | 174 | { |
| 175 | ns += a->tv_nsec; | 175 | ns += a->tv_nsec; |
| 176 | while(unlikely(ns >= NSEC_PER_SEC)) { | 176 | while(unlikely(ns >= NSEC_PER_SEC)) { |
| 177 | /* The following asm() prevents the compiler from | ||
| 178 | * optimising this loop into a modulo operation. */ | ||
| 179 | asm("" : "+r"(ns)); | ||
| 180 | |||
| 177 | ns -= NSEC_PER_SEC; | 181 | ns -= NSEC_PER_SEC; |
| 178 | a->tv_sec++; | 182 | a->tv_sec++; |
| 179 | } | 183 | } |
diff --git a/include/linux/timex.h b/include/linux/timex.h index c3f374786a43..8ea3e71ba7fa 100644 --- a/include/linux/timex.h +++ b/include/linux/timex.h | |||
| @@ -232,14 +232,7 @@ static inline int ntp_synced(void) | |||
| 232 | #else | 232 | #else |
| 233 | #define NTP_INTERVAL_FREQ (HZ) | 233 | #define NTP_INTERVAL_FREQ (HZ) |
| 234 | #endif | 234 | #endif |
| 235 | 235 | #define NTP_INTERVAL_LENGTH (NSEC_PER_SEC/NTP_INTERVAL_FREQ) | |
| 236 | #define CLOCK_TICK_OVERFLOW (LATCH * HZ - CLOCK_TICK_RATE) | ||
| 237 | #define CLOCK_TICK_ADJUST (((s64)CLOCK_TICK_OVERFLOW * NSEC_PER_SEC) / \ | ||
| 238 | (s64)CLOCK_TICK_RATE) | ||
| 239 | |||
| 240 | /* Because using NSEC_PER_SEC would be too easy */ | ||
| 241 | #define NTP_INTERVAL_LENGTH ((((s64)TICK_USEC * NSEC_PER_USEC * USER_HZ) + \ | ||
| 242 | CLOCK_TICK_ADJUST) / NTP_INTERVAL_FREQ) | ||
| 243 | 236 | ||
| 244 | /* Returns how long ticks are at present, in ns / 2^(SHIFT_SCALE-10). */ | 237 | /* Returns how long ticks are at present, in ns / 2^(SHIFT_SCALE-10). */ |
| 245 | extern u64 current_tick_length(void); | 238 | extern u64 current_tick_length(void); |
diff --git a/kernel/time/ntp.c b/kernel/time/ntp.c index c88b5910e7ab..5fd9b9469770 100644 --- a/kernel/time/ntp.c +++ b/kernel/time/ntp.c | |||
| @@ -42,12 +42,13 @@ long time_esterror = NTP_PHASE_LIMIT; /* estimated error (us) */ | |||
| 42 | long time_freq; /* frequency offset (scaled ppm)*/ | 42 | long time_freq; /* frequency offset (scaled ppm)*/ |
| 43 | static long time_reftime; /* time at last adjustment (s) */ | 43 | static long time_reftime; /* time at last adjustment (s) */ |
| 44 | long time_adjust; | 44 | long time_adjust; |
| 45 | static long ntp_tick_adj; | ||
| 45 | 46 | ||
| 46 | static void ntp_update_frequency(void) | 47 | static void ntp_update_frequency(void) |
| 47 | { | 48 | { |
| 48 | u64 second_length = (u64)(tick_usec * NSEC_PER_USEC * USER_HZ) | 49 | u64 second_length = (u64)(tick_usec * NSEC_PER_USEC * USER_HZ) |
| 49 | << TICK_LENGTH_SHIFT; | 50 | << TICK_LENGTH_SHIFT; |
| 50 | second_length += (s64)CLOCK_TICK_ADJUST << TICK_LENGTH_SHIFT; | 51 | second_length += (s64)ntp_tick_adj << TICK_LENGTH_SHIFT; |
| 51 | second_length += (s64)time_freq << (TICK_LENGTH_SHIFT - SHIFT_NSEC); | 52 | second_length += (s64)time_freq << (TICK_LENGTH_SHIFT - SHIFT_NSEC); |
| 52 | 53 | ||
| 53 | tick_length_base = second_length; | 54 | tick_length_base = second_length; |
| @@ -342,14 +343,16 @@ int do_adjtimex(struct timex *txc) | |||
| 342 | freq_adj = shift_right(freq_adj, time_constant * 2 + | 343 | freq_adj = shift_right(freq_adj, time_constant * 2 + |
| 343 | (SHIFT_PLL + 2) * 2 - SHIFT_NSEC); | 344 | (SHIFT_PLL + 2) * 2 - SHIFT_NSEC); |
| 344 | if (mtemp >= MINSEC && (time_status & STA_FLL || mtemp > MAXSEC)) { | 345 | if (mtemp >= MINSEC && (time_status & STA_FLL || mtemp > MAXSEC)) { |
| 346 | u64 utemp64; | ||
| 345 | temp64 = time_offset << (SHIFT_NSEC - SHIFT_FLL); | 347 | temp64 = time_offset << (SHIFT_NSEC - SHIFT_FLL); |
| 346 | if (time_offset < 0) { | 348 | if (time_offset < 0) { |
| 347 | temp64 = -temp64; | 349 | utemp64 = -temp64; |
| 348 | do_div(temp64, mtemp); | 350 | do_div(utemp64, mtemp); |
| 349 | freq_adj -= temp64; | 351 | freq_adj -= utemp64; |
| 350 | } else { | 352 | } else { |
| 351 | do_div(temp64, mtemp); | 353 | utemp64 = temp64; |
| 352 | freq_adj += temp64; | 354 | do_div(utemp64, mtemp); |
| 355 | freq_adj += utemp64; | ||
| 353 | } | 356 | } |
| 354 | } | 357 | } |
| 355 | freq_adj += time_freq; | 358 | freq_adj += time_freq; |
| @@ -400,3 +403,11 @@ leave: if ((time_status & (STA_UNSYNC|STA_CLOCKERR)) != 0) | |||
| 400 | notify_cmos_timer(); | 403 | notify_cmos_timer(); |
| 401 | return(result); | 404 | return(result); |
| 402 | } | 405 | } |
| 406 | |||
| 407 | static int __init ntp_tick_adj_setup(char *str) | ||
| 408 | { | ||
| 409 | ntp_tick_adj = simple_strtol(str, NULL, 0); | ||
| 410 | return 1; | ||
| 411 | } | ||
| 412 | |||
| 413 | __setup("ntp_tick_adj=", ntp_tick_adj_setup); | ||
diff --git a/kernel/time/tick-sched.c b/kernel/time/tick-sched.c index 2968298f8f36..686da821d376 100644 --- a/kernel/time/tick-sched.c +++ b/kernel/time/tick-sched.c | |||
| @@ -640,7 +640,7 @@ void tick_cancel_sched_timer(int cpu) | |||
| 640 | 640 | ||
| 641 | if (ts->sched_timer.base) | 641 | if (ts->sched_timer.base) |
| 642 | hrtimer_cancel(&ts->sched_timer); | 642 | hrtimer_cancel(&ts->sched_timer); |
| 643 | ts->tick_stopped = 0; | 643 | |
| 644 | ts->nohz_mode = NOHZ_MODE_INACTIVE; | 644 | ts->nohz_mode = NOHZ_MODE_INACTIVE; |
| 645 | } | 645 | } |
| 646 | #endif /* HIGH_RES_TIMERS */ | 646 | #endif /* HIGH_RES_TIMERS */ |
diff --git a/kernel/time/timekeeping.c b/kernel/time/timekeeping.c index 1af9fb050fe2..671af612b768 100644 --- a/kernel/time/timekeeping.c +++ b/kernel/time/timekeeping.c | |||
| @@ -187,8 +187,7 @@ static void change_clocksource(void) | |||
| 187 | 187 | ||
| 188 | clock->error = 0; | 188 | clock->error = 0; |
| 189 | clock->xtime_nsec = 0; | 189 | clock->xtime_nsec = 0; |
| 190 | clocksource_calculate_interval(clock, | 190 | clocksource_calculate_interval(clock, NTP_INTERVAL_LENGTH); |
| 191 | (unsigned long)(current_tick_length()>>TICK_LENGTH_SHIFT)); | ||
| 192 | 191 | ||
| 193 | tick_clock_notify(); | 192 | tick_clock_notify(); |
| 194 | 193 | ||
| @@ -245,8 +244,7 @@ void __init timekeeping_init(void) | |||
| 245 | ntp_clear(); | 244 | ntp_clear(); |
| 246 | 245 | ||
| 247 | clock = clocksource_get_next(); | 246 | clock = clocksource_get_next(); |
| 248 | clocksource_calculate_interval(clock, | 247 | clocksource_calculate_interval(clock, NTP_INTERVAL_LENGTH); |
| 249 | (unsigned long)(current_tick_length()>>TICK_LENGTH_SHIFT)); | ||
| 250 | clock->cycle_last = clocksource_read(clock); | 248 | clock->cycle_last = clocksource_read(clock); |
| 251 | 249 | ||
| 252 | xtime.tv_sec = sec; | 250 | xtime.tv_sec = sec; |
