]> git.baikalelectronics.ru Git - kernel.git/commit
time: remove obsolete CLOCK_TICK_ADJUST
authorRoman Zippel <zippel@linux-m68k.org>
Tue, 4 Mar 2008 23:14:26 +0000 (15:14 -0800)
committerThomas Gleixner <tglx@linutronix.de>
Sun, 9 Mar 2008 07:42:57 +0000 (08:42 +0100)
commit9e41c5f408c543d63ea85674e9c6be1ca4c838c6
tree02096fe81b70e2baf8db35ebc9d303a1b13029c3
parent4dfa19902c2517ebaf19d6367d4dffa4ed43fb40
time: remove obsolete CLOCK_TICK_ADJUST

The first version of the ntp_interval/tick_length inconsistent usage patch was
recently merged as 05cd3d4516709f5af58bca55049652ba22670ae4

http://git.kernel.org/gitweb.cgi?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=05cd3d4516709f5af58bca55049652ba22670ae4

While the fix did greatly improve the situation, it was correctly pointed out
by Roman that it does have a small bug: If the users change clocksources after
the system has been running and NTP has made corrections, the correctoins made
against the old clocksource will be applied against the new clocksource,
causing error.

The second attempt, which corrects the issue in the NTP_INTERVAL_LENGTH
definition has also made it up-stream as commit
97b9621c6dcb2084f821c0b342038344f288ebe8

http://git.kernel.org/gitweb.cgi?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=97b9621c6dcb2084f821c0b342038344f288ebe8

Roman has correctly pointed out that CLOCK_TICK_ADJUST is calculated
based on the PIT's frequency, and isn't really relevant to non-PIT
driven clocksources (that is, clocksources other then jiffies and pit).

This patch reverts both of those changes, and simply removes
CLOCK_TICK_ADJUST.

This does remove the granularity error correction for users of PIT and Jiffies
clocksource users, but the granularity error but for the majority of users, it
should be within the 500ppm range NTP can accommodate for.

For systems that have granularity errors greater then 500ppm, the
"ntp_tick_adj=" boot option can be used to compensate.

[johnstul@us.ibm.com: provided changelog]
[mattilinnanvuori@yahoo.com: maek ntp_tick_adj static]
Signed-off-by: Roman Zippel <zippel@linux-m68k.org>
Acked-by: john stultz <johnstul@us.ibm.com>
Signed-off-by: Matti Linnanvuori <mattilinnanvuori@yahoo.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Cc: mingo@elte.hu
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
include/linux/timex.h
kernel/time/ntp.c
kernel/time/timekeeping.c