LKML Archive on lore.kernel.org help / color / mirror / Atom feed
From: Daniel Walker <dwalker@mvista.com> To: akpm@osdl.org Cc: linux-kernel@vger.kernel.org, johnstul@us.ibm.com, tglx@linutronix.de, mingo@elte.hu Subject: [PATCH 03/23] clocksource: drop clocksource-remove-the-update-callback.patch Date: Tue, 30 Jan 2007 19:37:13 -0800 [thread overview] Message-ID: <20070131033803.909323479@mvista.com> (raw) In-Reply-To: 20070131033710.420168478@mvista.com [-- Attachment #1: drop-clocksource-remove-the-update-callback.patch --] [-- Type: text/plain, Size: 5390 bytes --] Drop. Signed-Off-By: Daniel Walker <dwalker@mvista.com> --- arch/i386/kernel/tsc.c | 51 +++++++++++++++++++++++++++++--------------- include/linux/clocksource.h | 11 ++++++--- kernel/timer.c | 3 ++ 3 files changed, 45 insertions(+), 20 deletions(-) Index: linux-2.6.19/arch/i386/kernel/tsc.c =================================================================== --- linux-2.6.19.orig/arch/i386/kernel/tsc.c +++ linux-2.6.19/arch/i386/kernel/tsc.c @@ -60,6 +60,12 @@ static inline int check_tsc_unstable(voi return tsc_unstable; } +void mark_tsc_unstable(void) +{ + tsc_unstable = 1; +} +EXPORT_SYMBOL_GPL(mark_tsc_unstable); + /* Accellerators for sched_clock() * convert from cycles(64bits) => nanoseconds (64bits) * basic equation: @@ -289,6 +295,7 @@ core_initcall(cpufreq_tsc); /* clock source code */ static unsigned long current_tsc_khz = 0; +static int tsc_update_callback(void); static cycle_t read_tsc(void) { @@ -306,28 +313,38 @@ static struct clocksource clocksource_ts .mask = CLOCKSOURCE_MASK(64), .mult = 0, /* to be set */ .shift = 22, + .update_callback = tsc_update_callback, .flags = CLOCK_SOURCE_IS_CONTINUOUS | CLOCK_SOURCE_MUST_VERIFY, }; -void mark_tsc_unstable(void) +static int tsc_update_callback(void) { - if (!tsc_unstable) { - tsc_unstable = 1; - /* Can be called before registration */ - if (clocksource_tsc.mult) - clocksource_change_rating(&clocksource_tsc, 0); - else - clocksource_tsc.rating = 0; + int change = 0; + + /* check to see if we should switch to the safe clocksource: */ + if (clocksource_tsc.rating != 0 && check_tsc_unstable()) { + clocksource_tsc.rating = 0; + clocksource_reselect(); + change = 1; + } + + /* only update if tsc_khz has changed: */ + if (current_tsc_khz != tsc_khz) { + current_tsc_khz = tsc_khz; + clocksource_tsc.mult = clocksource_khz2mult(current_tsc_khz, + clocksource_tsc.shift); + change = 1; } + + return change; } -EXPORT_SYMBOL_GPL(mark_tsc_unstable); static int __init dmi_mark_tsc_unstable(struct dmi_system_id *d) { printk(KERN_NOTICE "%s detected: marking TSC unstable.\n", d->ident); - tsc_unstable = 1; + mark_tsc_unstable(); return 0; } @@ -399,12 +416,11 @@ __cpuinit int unsynchronized_tsc(void) * Intel systems are normally all synchronized. * Exceptions must mark TSC as unstable: */ - if (boot_cpu_data.x86_vendor != X86_VENDOR_INTEL) { - /* assume multi socket systems are not synchronized: */ - if (num_possible_cpus() > 1) - tsc_unstable = 1; - } - return tsc_unstable; + if (boot_cpu_data.x86_vendor == X86_VENDOR_INTEL) + return 0; + + /* assume multi socket systems are not synchronized: */ + return num_possible_cpus() > 1; } /* @@ -433,7 +449,8 @@ static int __init init_tsc_clocksource(v /* check blacklist */ dmi_check_system(bad_tsc_dmi_table); - unsynchronized_tsc(); + if (unsynchronized_tsc()) /* mark unstable if unsynced */ + mark_tsc_unstable(); check_geode_tsc_reliable(); current_tsc_khz = tsc_khz; clocksource_tsc.mult = clocksource_khz2mult(current_tsc_khz, Index: linux-2.6.19/include/linux/clocksource.h =================================================================== --- linux-2.6.19.orig/include/linux/clocksource.h +++ linux-2.6.19/include/linux/clocksource.h @@ -44,6 +44,7 @@ typedef u64 cycle_t; * subtraction of non 64 bit counters * @mult: cycle to nanosecond multiplier * @shift: cycle to nanosecond divisor (power of two) + * @update_callback: called when safe to alter clocksource values * @flags: flags describing special properties * @vread: vsyscall based read * @cycle_interval: Used internally by timekeeping core, please ignore. @@ -57,6 +58,7 @@ struct clocksource { cycle_t mask; u32 mult; u32 shift; + int (*update_callback)(void); unsigned long flags; cycle_t (*vread)(void); @@ -184,9 +186,9 @@ static inline void clocksource_calculate /* used to install a new clocksource */ -extern int clocksource_register(struct clocksource*); -extern struct clocksource* clocksource_get_next(void); -extern void clocksource_change_rating(struct clocksource *cs, int rating); +int clocksource_register(struct clocksource*); +void clocksource_reselect(void); +struct clocksource* clocksource_get_next(void); #ifdef CONFIG_GENERIC_TIME_VSYSCALL extern void update_vsyscall(struct timespec *ts, struct clocksource *c); @@ -196,4 +198,7 @@ static inline void update_vsyscall(struc } #endif +#define clocksource_reselect() clocksource_change_rating(&clocksource_tsc, clocksource_tsc.rating) +extern void clocksource_change_rating(struct clocksource *cs, int rating); + #endif /* _LINUX_CLOCKSOURCE_H */ Index: linux-2.6.19/kernel/timer.c =================================================================== --- linux-2.6.19.orig/kernel/timer.c +++ linux-2.6.19/kernel/timer.c @@ -893,6 +893,8 @@ static int change_clocksource(void) printk(KERN_INFO "Time: %s clocksource has been installed.\n", clock->name); return 1; + } else if (clock->update_callback) { + return clock->update_callback(); } return 0; } @@ -959,6 +961,7 @@ void __init timekeeping_init(void) write_sequnlock_irqrestore(&xtime_lock, flags); } + /* flag for if timekeeping is suspended */ static int timekeeping_suspended; /* time in seconds when suspend began */ --
next prev parent reply other threads:[~2007-01-31 3:49 UTC|newest] Thread overview: 57+ messages / expand[flat|nested] mbox.gz Atom feed top 2007-01-31 3:37 [PATCH 00/23] clocksource update v12 Daniel Walker 2007-01-31 3:37 ` [PATCH 01/23] clocksource: drop clocksource-add-verification-watchdog-helper-fix.patch Daniel Walker 2007-01-31 12:47 ` Ingo Molnar 2007-01-31 3:37 ` [PATCH 02/23] clocksource: drop clocksource-add-verification-watchdog-helper.patch Daniel Walker 2007-01-31 3:37 ` Daniel Walker [this message] 2007-01-31 3:37 ` [PATCH 04/23] clocksource: drop time-x86_64-tsc-fixup-clocksource-changes.patch Daniel Walker 2007-01-31 3:37 ` [PATCH 05/23] clocksource: drop simplify-the-registration-of-clocksources.patch Daniel Walker 2007-01-31 3:37 ` [PATCH 06/23] timekeeping: create kernel/time/timekeeping.c Daniel Walker 2007-01-31 8:59 ` Ingo Molnar 2007-01-31 15:05 ` Daniel Walker 2007-01-31 3:37 ` [PATCH 07/23] clocksource: rating sorted list Daniel Walker 2007-01-31 9:34 ` Ingo Molnar 2007-01-31 15:07 ` Daniel Walker 2007-01-31 3:37 ` [PATCH 08/23] clocksource: drop duplicate register checking Daniel Walker 2007-01-31 9:59 ` Ingo Molnar 2007-01-31 15:13 ` Daniel Walker 2007-01-31 17:19 ` Ingo Molnar 2007-01-31 3:37 ` [PATCH 09/23] clocksource: add block notifier Daniel Walker 2007-01-31 10:17 ` Ingo Molnar 2007-01-31 15:25 ` Daniel Walker 2007-01-31 17:22 ` Ingo Molnar 2007-01-31 3:37 ` [PATCH 10/23] clocksource: remove update_callback Daniel Walker 2007-01-31 10:46 ` Ingo Molnar 2007-01-31 15:42 ` Daniel Walker 2007-01-31 17:18 ` Ingo Molnar 2007-01-31 3:37 ` [PATCH 11/23] clocksource: atomic signals Daniel Walker 2007-01-31 11:07 ` Ingo Molnar 2007-01-31 15:59 ` Daniel Walker 2007-01-31 17:15 ` Ingo Molnar 2007-01-31 3:37 ` [PATCH 12/23] clocksource: add clocksource_get_clock() Daniel Walker 2007-01-31 11:46 ` Ingo Molnar 2007-01-31 16:40 ` Daniel Walker 2007-01-31 3:37 ` [PATCH 13/23] timekeeping: move sysfs layer/drop API calls Daniel Walker 2007-01-31 11:49 ` Ingo Molnar 2007-01-31 3:37 ` [PATCH 14/23] clocksource: increase initcall priority Daniel Walker 2007-01-31 11:50 ` Ingo Molnar 2007-01-31 16:42 ` Daniel Walker 2007-01-31 17:10 ` Ingo Molnar 2007-01-31 17:20 ` Daniel Walker 2007-01-31 17:29 ` Thomas Gleixner 2007-01-31 3:37 ` [PATCH 15/23] clocksource: add new flags Daniel Walker 2007-01-31 3:37 ` [PATCH 16/23] clocksource: arm update for " Daniel Walker 2007-01-31 12:27 ` Ingo Molnar 2007-01-31 3:37 ` [PATCH 17/23] clocksource: avr32 " Daniel Walker 2007-01-31 3:37 ` [PATCH 18/23] clocksource: i386 " Daniel Walker 2007-01-31 3:37 ` [PATCH 19/23] clocksource: mips " Daniel Walker 2007-01-31 3:37 ` [PATCH 20/23] clocksource: x86_64 " Daniel Walker 2007-01-31 3:37 ` [PATCH 21/23] clocksource: drivers/ " Daniel Walker 2007-01-31 3:37 ` [PATCH 22/23] clocksource: new clock lookup method Daniel Walker [not found] ` <20070131122215.GE1847@elte.hu> [not found] ` <1170261439.9781.96.camel@imap.mvista.com> [not found] ` <20070131164918.GA4468@elte.hu> [not found] ` <1170265169.9781.145.camel@imap.mvista.com> 2007-01-31 17:55 ` Thomas Gleixner 2007-01-31 18:07 ` Daniel Walker 2007-01-31 21:09 ` Thomas Gleixner 2007-01-31 3:37 ` [PATCH 23/23] clocksource tsc: add verify routine Daniel Walker 2007-01-31 12:43 ` Ingo Molnar 2007-01-31 17:02 ` Daniel Walker 2007-01-31 17:22 ` Thomas Gleixner 2007-01-31 17:33 ` Ingo Molnar
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20070131033803.909323479@mvista.com \ --to=dwalker@mvista.com \ --cc=akpm@osdl.org \ --cc=johnstul@us.ibm.com \ --cc=linux-kernel@vger.kernel.org \ --cc=mingo@elte.hu \ --cc=tglx@linutronix.de \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).