kpriv/linux-hardened
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

clocksource: keep track of original clocksource frequency

Browse source 
The clocksource frequency is represented by
clocksource->mult/2^(clocksource->shift).  Currently, when NTP makes
adjustments to the clock frequency, they are made directly to the mult
value.

This has the drawback that once changed, we cannot know what the orignal
mult value was, or how much adjustment has been applied.

This property causes problems in calculating proper ntp intervals when
switching back and forth between clocksources.

This patch separates the current mult value into a mult and mult_orig
pair.  The mult_orig value stays constant, while the ntp clocksource
adjustments are done only to the mult value.

This allows for correct ntp interval calculation and additionally lays the
groundwork for a new notion of time, what I'm calling the monotonic-raw
time, which is introduced in a following patch.

Signed-off-by: John Stultz <johnstul@us.ibm.com>
Signed-off-by: Roman Zippel <zippel@linux-m68k.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
This commit is contained in:
John Stultz 2008-08-20 16:37:28 -07:00 committed by Ingo Molnar
parent ee974e01e5
commit 1aa5dfb751
3 changed files with 11 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
include/linux/clocksource.h
View file

@ -45,7 +45,8 @@ struct clocksource;
* @read: returns a cycle value * @read: returns a cycle value
* @mask: bitmask for two's complement * @mask: bitmask for two's complement
* subtraction of non 64 bit counters * subtraction of non 64 bit counters
* @mult: cycle to nanosecond multiplier * @mult: cycle to nanosecond multiplier (adjusted by NTP)
* @mult_orig: cycle to nanosecond multiplier (unadjusted by NTP)
* @shift: cycle to nanosecond divisor (power of two) * @shift: cycle to nanosecond divisor (power of two)
* @flags: flags describing special properties * @flags: flags describing special properties
* @vread: vsyscall based read * @vread: vsyscall based read
@ -63,6 +64,7 @@ struct clocksource {
cycle_t (*read)(void); cycle_t (*read)(void);
cycle_t mask; cycle_t mask;
u32 mult; u32 mult;
u32 mult_orig;
u32 shift; u32 shift;
unsigned long flags; unsigned long flags;
cycle_t (*vread)(void); cycle_t (*vread)(void);
@ -201,16 +203,17 @@ static inline void clocksource_calculate_interval(struct clocksource *c,
{ {
u64 tmp; u64 tmp;
/* XXX - All of this could use a whole lot of optimization */ /* Do the ns -> cycle conversion first, using original mult */
tmp = length_nsec; tmp = length_nsec;
tmp <<= c->shift; tmp <<= c->shift;
tmp += c->mult/2; tmp += c->mult_orig/2;
do_div(tmp, c->mult); do_div(tmp, c->mult_orig);
c->cycle_interval = (cycle_t)tmp; c->cycle_interval = (cycle_t)tmp;
if (c->cycle_interval == 0) if (c->cycle_interval == 0)
c->cycle_interval = 1; c->cycle_interval = 1;
/* Go back from cycles -> shifted ns, this time use ntp adjused mult */
c->xtime_interval = (u64)c->cycle_interval * c->mult; c->xtime_interval = (u64)c->cycle_interval * c->mult;
} }

3
kernel/time/clocksource.c
View file

@ -325,6 +325,9 @@ int clocksource_register(struct clocksource *c)
unsigned long flags; unsigned long flags;
int ret; int ret;
/* save mult_orig on registration */
c->mult_orig = c->mult;
spin_lock_irqsave(&clocksource_lock, flags); spin_lock_irqsave(&clocksource_lock, flags);
ret = clocksource_enqueue(c); ret = clocksource_enqueue(c);
if (!ret) if (!ret)

1
kernel/time/jiffies.c
View file

@ -61,6 +61,7 @@ struct clocksource clocksource_jiffies = {
.read = jiffies_read, .read = jiffies_read,
.mask = 0xffffffff, /*32bits*/ .mask = 0xffffffff, /*32bits*/
.mult = NSEC_PER_JIFFY << JIFFIES_SHIFT, /* details above */ .mult = NSEC_PER_JIFFY << JIFFIES_SHIFT, /* details above */
.mult_orig = NSEC_PER_JIFFY << JIFFIES_SHIFT,
.shift = JIFFIES_SHIFT, .shift = JIFFIES_SHIFT,
}; };